Minor Changes in Terminal Program

Author: Max 'MaxMax' Mönikes

README.md

@@ -1,4 +1,4 @@
-# A Arduino Library for the Infineon TLE9012 Battery Management IC
+# An Arduino Library for the Infineon TLE9012 Battery Management IC
 
 This Library aims to be an easy to use solution to experiment with the TLE9012 BMS IC from Infineon. There are a couple examples included in the examples folder to get you started using the basic functions of the library. More complex examples are planed to be added in the future. Examples are written for a ESP32 Node MCU but can be adapted to other boards by passing an appropriate HardwareSerial instance to the tle9012.init() function. If RX and TX pins are fixed for the architecture just pass 0 for the rxpin and txpin arguments. 
 

TLE9012.cpp

@@ -998,6 +998,52 @@ iso_uart_status_t TLE9012::readRegisterSingle(uint8_t nodeID, uint16_t regaddres
   return status;
 }
 
+iso_uart_status_t TLE9012::readRegisterSingle_ext(uint8_t* nodeID, uint16_t* regaddress, uint16_t* result)
+{
+  ISOUART_LOCK();
+  iso_uart_status_t status;
+  uint8_t response_buffer[9];
+  
+  status = isoUART_TIMEOUT;
+
+  isoUARTClearRXBUffer();
+  isoUARTReadRequest(*nodeID, *regaddress);
+
+  uint32_t starttime = millis();
+  while((millis()-starttime) < ISOUART_TIMEOUT)
+  {
+    if(hisoUART->available() > 8)
+    {
+      hisoUART->readBytes(response_buffer,9);
+      status = isoUART_OK;
+      break;
+    }
+  }
+
+  //Check if Timeout occured
+  if(status != isoUART_OK)
+  {
+    status = isoUART_TIMEOUT;
+    ISOUART_UNLOCK();
+    return status;
+  }
+
+  #ifdef SOFT_MSB_FIRST
+  msb_first_converter(&(response_buffer[4]),5);
+  #endif
+  uint8_t crc = crc8(&response_buffer[4],4);
+
+  *result = (((uint16_t) response_buffer[6])<<8) | ((uint16_t) response_buffer[7]);
+  *regaddress = (uint16_t) response_buffer[5];
+  *nodeID = (uint8_t) response_buffer[4];
+
+  if(crc != response_buffer[8])
+    status = isoUART_CRC_ERROR;
+
+  ISOUART_UNLOCK();
+  return status;
+}
+
 /**
  * @brief isoUART function to write a value to a single register on a single node in the daisy chain
  * 

TLE9012.h

@@ -391,6 +391,7 @@ typedef struct
 
       //Low Level Routines for direct register Access
       iso_uart_status_t readRegisterSingle(uint8_t nodeID, uint16_t regaddress, uint16_t* result);  //Read data from a single register
+      iso_uart_status_t readRegisterSingle_ext(uint8_t* nodeID, uint16_t* regaddress, uint16_t* result); //Extended Read Function that reads back all parameters (data, return frame address and returned node ID)
       iso_uart_status_t writeRegisterSingle(uint8_t nodeID, uint16_t regaddress, uint16_t databuffer); //Write data to a single register
 
       iso_uart_status_t readRegisterBroadcast(uint16_t regaddress, uint16_t* result); //Write a broadcast to all devices in the daisy chain

examples/Terminal_Interface/Terminal_Interface.ino

@@ -53,9 +53,12 @@ The following commands are implemented:\n \
  -K update_rate : Reset Watchdog every update_rate interval in milliseconds\n\
  -? Show Manual\n";
 
+HardwareSerial Console = Serial0;
+
 void setup() { 
-  Serial.begin(115200); //Start Console interface
-  tle9012.init(&Serial1, 1000000,0,0); //Initialize driver with 2Mbit
+  Console.begin(115200); //Start Console interface
+  Console.println("Start completed");
+  tle9012.init(&Serial1, 2000000,RXPIN,TXPIN); //Initialize driver with 2Mbit
 }
 
 void loop() {
@@ -88,7 +91,7 @@ void handleTerminal()
 {
 
   //Check if new data was received
-  uint8_t rx_len = Serial.available();
+  uint8_t rx_len = Console.available();
 
   //Handle overflow situations by resetting the buffer
   if((rx_len+terminalbufferrecvlen) > 64)
@@ -100,7 +103,7 @@ void handleTerminal()
   }
 
   //Read Bytes from Serial RX Buffer to terminalbuffer
-  Serial.readBytes(&terminalbuffer[terminalbufferrecvlen], rx_len);
+  Console.readBytes(&terminalbuffer[terminalbufferrecvlen], rx_len);
 
   //Check newly received data for \n end of line character
   uint8_t eol_char_found = 0;
@@ -136,15 +139,15 @@ void interpretCommand(const char commandbuffer[], uint8_t bufferlength)
 
     if(sscanf(commandbuffer,"%s",cmd)  == 1) //Read command
     {
-      Serial.write(commandbuffer,bufferlength);
+      Console.write(commandbuffer,bufferlength);
       if(!strcmp(cmd,"IL")) //Emulate IL Command (no function because no ring bus mode is supported by this library)
       {
-        Serial.println("IL OK");
+        Console.println("IL OK");
       }
 
       else if(!strcmp(cmd,"IH")) //Emulate IH Command (no function because no ring bus mode is supported by this library)
       {
-        Serial.println("IH OK");
+        Console.println("IH OK");
       }
 
       else if((strcmp(cmd,"WH") == 0) | (strcmp(cmd,"WL") == 0)) //Write data to a single register of a single device on the bus
@@ -157,61 +160,61 @@ void interpretCommand(const char commandbuffer[], uint8_t bufferlength)
           iso_uart_status_t status = tle9012.writeRegisterSingle(dev_address, reg_address, data);
           if(status == isoUART_OK)
           {
-            Serial.print("WL ");
-            Serial.print(" ");
-            Serial.print(dev_address,HEX);
-            Serial.print(" ");
-            Serial.print(reg_address, HEX);
-            Serial.println(" OK 8000");
+            Console.print("WL ");
+            Console.print(" ");
+            Console.print(dev_address,HEX);
+            Console.print(" ");
+            Console.print(reg_address, HEX);
+            Console.println(" OK 8000");
           }
           if(status == isoUART_TIMEOUT)
           {
-            Serial.println("TIMEOUT");
+            Console.println("TIMEOUT");
           }
           if(status == isoUART_CRC_ERROR)
           {
-            Serial.println("CRC ERROR");
+            Console.println("CRC ERROR");
           }
         }
         else
         {
-          Serial.println("INVALID COMMAND FORMAT");
+          Console.println("INVALID COMMAND FORMAT");
         }
       }
 
       else if((strcmp(cmd,"RH") == 0) | (strcmp(cmd,"RL") == 0)) //Read data from a single register of a single device on the bus
       {
 
         uint8_t dev_address;
-        uint8_t reg_address;
+        uint16_t reg_address;
         uint16_t data;
 
         if(sscanf(commandbuffer, "%s %d %x", cmd, &dev_address, &reg_address) == 3)
         {
-          iso_uart_status_t status = tle9012.readRegisterSingle(dev_address, reg_address, &data);
+          iso_uart_status_t status = tle9012.readRegisterSingle_ext(&dev_address, &reg_address, &data);
           if(status == isoUART_OK)
           {
-            Serial.print("RL ");
-            Serial.print(dev_address,HEX);
-            Serial.print(" ");
-            Serial.print(reg_address,HEX);
-            Serial.print(" ");
-            Serial.print(data,HEX);
-            Serial.print(" OK ");
-            Serial.println("8000");
+            Console.print("RL ");
+            Console.print(dev_address,HEX);
+            Console.print(" ");
+            Console.print(reg_address,HEX);
+            Console.print(" ");
+            Console.print(data,HEX);
+            Console.print(" OK ");
+            Console.println("8000");
           }
           if(status == isoUART_TIMEOUT)
           {
-            Serial.println("TIMEOUT");
+            Console.println("TIMEOUT");
           }
           if(status == isoUART_CRC_ERROR)
           {
-            Serial.println("CRC ERROR");
+            Console.println("CRC ERROR");
           }
         }
         else
         {
-          Serial.println("INVALID COMMAND FORMAT");
+          Console.println("INVALID COMMAND FORMAT");
         }        
       }
 
@@ -224,26 +227,26 @@ void interpretCommand(const char commandbuffer[], uint8_t bufferlength)
           {
             watchdog_time = wd_time;
             watchdog_active = 1;
-            Serial.print("Watchdog kicking time change to ");
-            Serial.print(watchdog_time);
-            Serial.println(" ms");
+            Console.print("Watchdog kicking time change to ");
+            Console.print(watchdog_time);
+            Console.println(" ms");
           }
           else
           {
              watchdog_active = 0;
-             Serial.println("Watchdog kicker deactivated");
+             Console.println("Watchdog kicker deactivated");
           }
         }
       }
 
       else if(!strcmp(cmd,"?")) //Print Helppage
       {
-        Serial.print(helppage);
+        Console.print(helppage);
       }
 
       else //No valid command was detected
       {
-        Serial.println("INVALID COMMAND");
+        Console.println("INVALID COMMAND");
       }
 
     }