Sercom: partially revert error handling change

kaysievers · kaysievers · commit 34d15d0f9986 · 2025-10-13T18:39:54.000+02:00
Revert a part of ccfc7db. It breaks I2C communcation.
diff --git a/cores/arduino/SERCOM.cpp b/cores/arduino/SERCOM.cpp
@@ -605,9 +605,7 @@ bool SERCOM::sendDataMasterWIRE(uint8_t data)
   while(!sercom->I2CM.INTFLAG.bit.MB) {
     // If a data transfer error occurs, the MB bit may never be set.
     // Check the error bit and bail if it's set.
-    // The data transfer errors that can occur (including BUSERR) are all
-    // rolled up into INTFLAG.bit.ERROR from STATUS.reg
-    if (sercom->I2CM.INTFLAG.bit.ERROR) {
+    if (sercom->I2CM.STATUS.bit.BUSERR) {
       return false;
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -605,9 +605,7 @@ bool SERCOM::sendDataMasterWIRE(uint8_t data)`
`605`	`605`	`while(!sercom->I2CM.INTFLAG.bit.MB) {`
`606`	`606`	`// If a data transfer error occurs, the MB bit may never be set.`
`607`	`607`	`// Check the error bit and bail if it's set.`
`608`		`- // The data transfer errors that can occur (including BUSERR) are all`
`609`		`- // rolled up into INTFLAG.bit.ERROR from STATUS.reg`
`610`		`- if (sercom->I2CM.INTFLAG.bit.ERROR) {`
	`608`	`+ if (sercom->I2CM.STATUS.bit.BUSERR) {`
`611`	`609`	`return false;`
`612`	`610`	`}`
`613`	`611`	`}`