peci: Only write-clear Upstream Done

crawfxrd · crawfxrd · commit 67bbcd9cb9a8 · 2025-08-18T10:16:24.000-06:00
Signed-off-by: Tim Crawford &lt;tcrawford@system76.com&gt;
diff --git a/src/board/system76/common/peci.c b/src/board/system76/common/peci.c
@@ -105,7 +105,7 @@ bool peci_get_temp(int16_t *const data) {
         }
     }
     // Clear upstream done status
-    ESUCTRL0 = ESUCTRL0_DONE;
+    ESUCTRL0 |= ESUCTRL0_DONE;
 
     // Wait for response
     //TODO: do this asynchronously to avoid delays?
@@ -202,7 +202,7 @@ int16_t peci_wr_pkg_config(uint8_t index, uint16_t param, uint32_t data) {
         }
     }
     // Clear upstream done status
-    ESUCTRL0 = ESUCTRL0_DONE;
+    ESUCTRL0 |= ESUCTRL0_DONE;
 
     // Wait for response
     //TODO: do this asynchronously to avoid delays?

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ bool peci_get_temp(int16_t *const data) {`
`105`	`105`	`}`
`106`	`106`	`}`
`107`	`107`	`// Clear upstream done status`
`108`		`- ESUCTRL0 = ESUCTRL0_DONE;`
	`108`	`+ ESUCTRL0 \|= ESUCTRL0_DONE;`
`109`	`109`
`110`	`110`	`// Wait for response`
`111`	`111`	`//TODO: do this asynchronously to avoid delays?`
`@@ -202,7 +202,7 @@ int16_t peci_wr_pkg_config(uint8_t index, uint16_t param, uint32_t data) {`
`202`	`202`	`}`
`203`	`203`	`}`
`204`	`204`	`// Clear upstream done status`
`205`		`- ESUCTRL0 = ESUCTRL0_DONE;`
	`205`	`+ ESUCTRL0 \|= ESUCTRL0_DONE;`
`206`	`206`
`207`	`207`	`// Wait for response`
`208`	`208`	`//TODO: do this asynchronously to avoid delays?`