maintainability: openblt text status #6981

only:proteus_f7

firmware/bootloader/openblt_chibios/openblt_usb.cpp

@@ -51,6 +51,17 @@ void Rs232TransmitPacket(blt_int8u *data, blt_int8u len)
   }
 } /*** end of Rs232TransmitPacket ***/
 
+PUBLIC_API_WEAK void openBltUnexpectedByte(blt_int8u firstByte) {
+#if defined(OPEN_BLT_TEST_COMMAND)
+// 'z' is right at the end of 128 ascii range
+static_assert(BOOT_COM_RS232_RX_MAX_DATA < 'z');
+  if (firstByte == 'z') {
+  const char * bltTest = "openblt\n";
+    chnWriteTimeout(&SDU1, (const uint8_t*)bltTest, sizeof(bltTest), TIME_INFINITE);
+  }
+#endif // OPEN_BLT_TEST_COMMAND
+}
+
 /************************************************************************************//**
 ** \brief     Receives a communication interface packet if one is present.
 ** \param     data Pointer to byte array where the data is to be stored.
@@ -80,6 +91,8 @@ blt_bool Rs232ReceivePacket(blt_int8u *data, blt_int8u *len)
         xcpCtoRxLength = 0;
         /* indicate that a cto packet is being received */
         xcpCtoRxInProgress = BLT_TRUE;
+      } else {
+        openBltUnexpectedByte(xcpCtoReqPacket[0]);
       }
     }
   }

firmware/config/boards/proteus/board.mk

@@ -16,6 +16,9 @@ DDEFS += -DEFI_MAX_31855=TRUE
 DDEFS += -DSTM32_SPI_USE_SPI5=TRUE
 DDEFS += -DEFI_TCU=TRUE
 
+DDEFS += -DBOOT_COM_RS232_RX_MAX_DATA=120
+DDEFS += -DOPEN_BLT_TEST_COMMAND=1
+
 # let's start asap
 DDEFS += -DBOOT_BACKDOOR_ENTRY_TIMEOUT_MS=0
 

firmware/config/boards/proteus/compile_proteus_f7_bootloader.sh

@@ -0,0 +1,2 @@
+#!/usr/bin/env bash
+cd ../../.. && bash bin/compile.sh config/boards/proteus/meta-info-proteus_f7.env bootloader

firmware/hw_layer/openblt/blt_conf.h

@@ -114,7 +114,9 @@
 /** \brief Configure number of bytes in the target->host data packet. */
 #define BOOT_COM_RS232_TX_MAX_DATA       (200)
 /** \brief Configure number of bytes in the host->target data packet. */
+#ifndef BOOT_COM_RS232_RX_MAX_DATA
 #define BOOT_COM_RS232_RX_MAX_DATA       (200)
+#endif
 
 /** only USB supported, this is ignored but required */
 #define BOOT_COM_RS232_CHANNEL_INDEX 0