lincomatic / open_evse

Firmware for Open EVSE
GNU General Public License v3.0
116 stars 166 forks source link

missing I2C temperature sensors #163

Closed KipK closed 1 year ago

KipK commented 1 year ago

Latest compiled master seems to miss i2c T° sensor from LCD ( or led extension ) . Switching back to previous version ( 8.2.2 ) is ok.

I'm trying to track down what could introduce this issue but haven't found yet.

I've added a msg on 8.2.3 pre-release to prevent people upgrading until it's solved.

glynhudson commented 1 year ago

I can confirm I'm able to replicate the issue,the latest version does not seem to read from a MCP9808 sensor.

I'm testing with an older LCD display with a MCP9808 sensor (temp2) and a MCP9808 external sensor connected to the OpenEVSE WiFi V1 ESP32 (temp4)

Running 8.2.2.EU (V8.2.0 Lincomatic) Temp1 and Temp4 are available.

8 2 2 0 with ESP32 I2c

Flashing 8.2.3.EU (V8.2.1 Lincomatic) only Temp4 is available, the controller is no longer reading from the MCP9808:

8 2 3

KipK commented 1 year ago

Ok I have found it. I2C was reinitialised after temperature sensor init.

164 fix this