ina238: use I2C resets

PX4 · Jan 22, 2025 · 4a81e65 · 4a81e65
1 parent 15b94a3
commit 4a81e65
Showing 1 changed file with 8 additions and 10 deletions.
diff --git a/src/drivers/power_monitor/ina238/ina238.cpp b/src/drivers/power_monitor/ina238/ina238.cpp
@@ -98,17 +98,13 @@ int INA238::read(uint8_t address, uint16_t &data)
 	uint16_t received_bytes;
 	int ret = PX4_ERROR;
 
-	for (size_t i = 0; i < 6; i++) {
-		ret = transfer(&address, 1, (uint8_t *)&received_bytes, sizeof(received_bytes));
+	ret = transfer(&address, 1, (uint8_t *)&received_bytes, sizeof(received_bytes));
 
-		if (ret == PX4_OK) {
-			data = swap16(received_bytes);
-			break;
-
-		} else {
-			perf_count(_comms_errors);
-			PX4_DEBUG("i2c::transfer returned %d", ret);
-		}
+	if (ret == PX4_OK) {
+		data = swap16(received_bytes);
+	} else {
+		perf_count(_comms_errors);
+		PX4_DEBUG("i2c::transfer returned %d", ret);
 	}
 
 	return ret;
@@ -165,6 +161,8 @@ int INA238::Reset()
 
 	int ret = PX4_ERROR;
 
+	_retries = 6;
+
 	if (RegisterWrite(Register::CONFIG, (uint16_t)(ADC_RESET_BIT)) != PX4_OK) {
 		return ret;
 	}