Fix wrong i2c id range check

augustofg · augustofg · commit 2869e9b1c45c · 2021-01-15T10:05:41.000-03:00
i2c_take_by_chipid should check chip_id against I2C_CHIP_CNT. This
fixes the PCIe link never coming up and other problems related to I2C
access.
diff --git a/modules/i2c.c b/modules/i2c.c
@@ -85,7 +85,7 @@ bool i2c_take_by_busid( uint8_t bus_id, uint8_t *i2c_interface, TickType_t timeo
 
 bool i2c_take_by_chipid( uint8_t chip_id, uint8_t *i2c_address, uint8_t *i2c_interface,  uint32_t timeout )
 {
-    if ( chip_id > I2C_BUS_CNT ) {
+    if ( chip_id > I2C_CHIP_CNT ) {
         return false;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ bool i2c_take_by_busid( uint8_t bus_id, uint8_t *i2c_interface, TickType_t timeo`
`85`	`85`
`86`	`86`	`bool i2c_take_by_chipid( uint8_t chip_id, uint8_t i2c_address, uint8_t i2c_interface, uint32_t timeout )`
`87`	`87`	`{`
`88`		`- if ( chip_id > I2C_BUS_CNT ) {`
	`88`	`+ if ( chip_id > I2C_CHIP_CNT ) {`
`89`	`89`	`return false;`
`90`	`90`	`}`
`91`	`91`