Return false if we already have the lock

tannewt · tannewt · commit 367d3800fc50 · 2020-07-01T14:35:25.000-07:00
diff --git a/ports/esp32s2/common-hal/busio/I2C.c b/ports/esp32s2/common-hal/busio/I2C.c
@@ -163,6 +163,9 @@ bool common_hal_busio_i2c_probe(busio_i2c_obj_t *self, uint8_t addr) {
 }
 
 bool common_hal_busio_i2c_try_lock(busio_i2c_obj_t *self) {
+    if (self->has_lock) {
+        return false;
+    }
     self->has_lock = xSemaphoreTake(&self->semaphore, 0) == pdTRUE;
     return self->has_lock;
 }

Original file line number	Diff line number	Diff line change
`@@ -163,6 +163,9 @@ bool common_hal_busio_i2c_probe(busio_i2c_obj_t *self, uint8_t addr) {`
`163`	`163`	`}`
`164`	`164`
`165`	`165`	`bool common_hal_busio_i2c_try_lock(busio_i2c_obj_t *self) {`
	`166`	`+ if (self->has_lock) {`
	`167`	`+ return false;`
	`168`	`+ }`
`166`	`169`	`self->has_lock = xSemaphoreTake(&self->semaphore, 0) == pdTRUE;`
`167`	`170`	`return self->has_lock;`
`168`	`171`	`}`