Merge pull request #8084 from breakersun/fix-spi

tannewt · web-flow · commit 1f2032806835 · 2023-06-14T17:27:24.000-07:00
fix `invalid pin` error when create `busio.SPI` on MIMXRT10XX port
diff --git a/ports/mimxrt10xx/common-hal/busio/SPI.c b/ports/mimxrt10xx/common-hal/busio/SPI.c
@@ -109,7 +109,7 @@ void common_hal_busio_spi_construct(busio_spi_obj_t *self,
         // if both MOSI and MISO exist, loop search normally
         if ((mosi != NULL) && (miso != NULL)) {
             for (uint j = 0; j < mosi_count; j++) {
-                if ((mcu_spi_sdo_list[i].pin != mosi)
+                if ((mcu_spi_sdo_list[j].pin != mosi)
                     || (mcu_spi_sck_list[i].bank_idx != mcu_spi_sdo_list[j].bank_idx)) {
                     continue;
                 }

Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ void common_hal_busio_spi_construct(busio_spi_obj_t *self,`
`109`	`109`	`// if both MOSI and MISO exist, loop search normally`
`110`	`110`	`if ((mosi != NULL) && (miso != NULL)) {`
`111`	`111`	`for (uint j = 0; j < mosi_count; j++) {`
`112`		`- if ((mcu_spi_sdo_list[i].pin != mosi)`
	`112`	`+ if ((mcu_spi_sdo_list[j].pin != mosi)`
`113`	`113`	`\|\| (mcu_spi_sck_list[i].bank_idx != mcu_spi_sdo_list[j].bank_idx)) {`
`114`	`114`	`continue;`
`115`	`115`	`}`