From 42c7b4a116352a880e2cd960b14e467638a50d03 Mon Sep 17 00:00:00 2001 From: Randy Mackay Date: Fri, 7 Aug 2015 15:08:23 +0900 Subject: [PATCH] BattMon_SMBus: remove unnecessary I2C semaphore give --- libraries/AP_BattMonitor/AP_BattMonitor_SMBus_I2C.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/libraries/AP_BattMonitor/AP_BattMonitor_SMBus_I2C.cpp b/libraries/AP_BattMonitor/AP_BattMonitor_SMBus_I2C.cpp index fa065db92a..af8be9c5ba 100644 --- a/libraries/AP_BattMonitor/AP_BattMonitor_SMBus_I2C.cpp +++ b/libraries/AP_BattMonitor/AP_BattMonitor_SMBus_I2C.cpp @@ -67,7 +67,6 @@ bool AP_BattMonitor_SMBus_I2C::read_word(uint8_t reg, uint16_t& data) const // take i2c bus semaphore if (!i2c_sem->take_nonblocking()) { - i2c_sem->give(); return false; } @@ -106,7 +105,6 @@ uint8_t AP_BattMonitor_SMBus_I2C::read_block(uint8_t reg, uint8_t* data, uint8_t // take i2c bus semaphore if (!i2c_sem->take_nonblocking()) { - i2c_sem->give(); return 0; } @@ -130,7 +128,6 @@ uint8_t AP_BattMonitor_SMBus_I2C::read_block(uint8_t reg, uint8_t* data, uint8_t // check PEC uint8_t pec = get_PEC(BATTMONITOR_SMBUS_I2C_ADDR, reg, true, buff, bufflen+1); if (pec != buff[bufflen+1]) { - i2c_sem->give(); return 0; }