From 67b8238b298b3204762d5f448fd3d7231b6d1a18 Mon Sep 17 00:00:00 2001
From: Carsten Schmiemann <carsten.schmiemann@pfalz-mail.de>
Date: Wed, 13 Apr 2022 22:37:45 +0200
Subject: [PATCH] Add battery average temperature

---
 .../src/vehicle_renaultzoe_ph2.cpp            | 20 ++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/OVMS.V3/components/vehicle_renaultzoe_ph2/src/vehicle_renaultzoe_ph2.cpp b/OVMS.V3/components/vehicle_renaultzoe_ph2/src/vehicle_renaultzoe_ph2.cpp
index a28a7a2..ab82df9 100644
--- a/OVMS.V3/components/vehicle_renaultzoe_ph2/src/vehicle_renaultzoe_ph2.cpp
+++ b/OVMS.V3/components/vehicle_renaultzoe_ph2/src/vehicle_renaultzoe_ph2.cpp
@@ -67,6 +67,7 @@ static const OvmsVehicle::poll_pid_t renault_zoe_polls[] = {
   { 0x18dadbf1, 0x18daf1db, VEHICLE_POLL_TYPE_OBDIIGROUP, 0x9062, { 0, 60, 600, 60 }, 0, ISOTP_EXTFRAME },  // Cell Bat Module 63-96
   { 0x18dadbf1, 0x18daf1db, VEHICLE_POLL_TYPE_OBDIIEXTENDED, 0x9005, { 0, 10, 3, 5 }, 0, ISOTP_EXTFRAME },  // Battery Voltage
   { 0x18dadbf1, 0x18daf1db, VEHICLE_POLL_TYPE_OBDIIEXTENDED, 0x925D, { 0, 10, 3, 5 }, 0, ISOTP_EXTFRAME },  // Battery Current
+  { 0x18dadbf1, 0x18daf1db, VEHICLE_POLL_TYPE_OBDIIEXTENDED, 0x9012, { 0, 10, 3, 5 }, 0, ISOTP_EXTFRAME },  // Battery Average Temperature
   { 0x18dadbf1, 0x18daf1db, VEHICLE_POLL_TYPE_OBDIIEXTENDED, 0x91C8, { 0, 10, 3, 5 }, 0, ISOTP_EXTFRAME },  // Battery Available Energy kWh
   { 0x18dadbf1, 0x18daf1db, VEHICLE_POLL_TYPE_OBDIIEXTENDED, 0x9243, { 0, 10, 10, 10 }, 0, ISOTP_EXTFRAME },  // Energy charged kWh
   { 0x18dadbf1, 0x18daf1db, VEHICLE_POLL_TYPE_OBDIIEXTENDED, 0x9244, { 0, 10, 10, 10 }, 0, ISOTP_EXTFRAME },  // Energy discharged kWh
@@ -361,38 +362,43 @@ void OvmsVehicleRenaultZoePh2::IncomingBCB(uint16_t type, uint16_t pid, const ch
  */
 void OvmsVehicleRenaultZoePh2::IncomingLBC(uint16_t type, uint16_t pid, const char* data, uint16_t len) {
 	switch (pid) {
-    case 0x9005: {
+    case 0x9005: { //Battery voltage
       StandardMetrics.ms_v_bat_voltage->SetValue((float) (CAN_UINT(0) * 0.1));
       ESP_LOGD(TAG, "9005 LBC ms_v_bat_voltage: %f", CAN_UINT(0) * 0.1);
       break;
     }
-    case 0x925D: {
+    case 0x925D: { //Battery current
       StandardMetrics.ms_v_bat_current->SetValue((float) ((CAN_UINT(0) - 1020) * 0.03125));
       ESP_LOGD(TAG, "925D LBC ms_v_bat_current: %d", CAN_UINT(0));
       ESP_LOGD(TAG, "925D LBC ms_v_bat_current: %f", ((CAN_UINT(0) - 1020) * 0.03125));
       break;
     }
-    case 0x9002: {
+    case 0x9012: { //Battery average temperature
+      StandardMetrics.ms_v_bat_temp->SetValue((float) ((CAN_UINT(0) - 40));
+      ESP_LOGD(TAG, "9012 LBC ms_v_bat_temp: %f", ((CAN_UINT(0) - 40));
+      break;
+    }
+    case 0x9002: { //Battery SOC
       StandardMetrics.ms_v_bat_soc->SetValue((float) (CAN_UINT(0)) * 0.01);
       ESP_LOGD(TAG, "9002 LBC ms_v_bat_soc: %f", CAN_UINT(0) * 0.01);
       break;
     }
-    case 0x9003: {
+    case 0x9003: { //Battery SOH
       StandardMetrics.ms_v_bat_soh->SetValue((float) (CAN_UINT(0) * 0.01));
       ESP_LOGD(TAG, "9003 LBC ms_v_bat_soh: %f", CAN_UINT(0) * 0.01);
       break;
     }
-    case 0x9243: {
+    case 0x9243: { //Battery energy charged kWh
       StandardMetrics.ms_v_charge_kwh_grid_total->SetValue((float) (CAN_UINT24(0)));
       ESP_LOGD(TAG, "9243 LBC ms_v_charge_kwh_grid_total: %d", CAN_UINT24(0));
       break;
     }
-    case 0x9244: {
+    case 0x9244: { //Battery energy discharged kWh
       StandardMetrics.ms_v_bat_energy_used->SetValue((float) (CAN_UINT24(0)));
       ESP_LOGD(TAG, "9244 LBC ms_v_bat_energy_used: %d", CAN_UINT24(0));
       break;
     }
-    case 0x9246: {
+    case 0x9246: { //Battery energy regenerated kWh
       StandardMetrics.ms_v_bat_energy_recd->SetValue((float) (CAN_UINT24(0)));
       ESP_LOGD(TAG, "9246 LBC ms_v_bat_energy_recd: %d", CAN_UINT24(0));
       break;