uart cosmetic fixes

* fx

* tttttttttttttttt

* dis

Author: openshwprojects

src/driver/drv_main.c

@@ -566,22 +566,23 @@ static driver_t g_drivers[] = {
 	//drvdetail:"title":"TODO",
 	//drvdetail:"descr":"A bridge relay driver, added for [TONGOU TO-Q-SY1-JWT Din Rail Switch](https://www.elektroda.com/rtvforum/topic3934580.html). See linked topic for info.",
 	//drvdetail:"requires":""}
-	{ "Bridge",     Bridge_driver_Init, NULL,                       NULL, Bridge_driver_QuickFrame, Bridge_driver_DeInit, Bridge_driver_OnChannelChanged, NULL, false }
+	{ "Bridge",     Bridge_driver_Init, NULL,                       NULL, Bridge_driver_QuickFrame, Bridge_driver_DeInit, Bridge_driver_OnChannelChanged, NULL, false },
 #endif
 #if ENABLE_DRIVER_UART_TCP
 	//drvdetail:{"name":"UartTCP",
 	//drvdetail:"title":"TODO",
 	//drvdetail:"descr":"UART to TCP bridge, mainly for WiFi Zigbee coordinators.",
 	//drvdetail:"requires":""}
-	{ "UartTCP",		UART_TCP_Init,		NULL,	NULL, NULL, UART_TCP_Deinit, NULL, NULL, false }
+	{ "UartTCP",		UART_TCP_Init,		NULL,	NULL, NULL, UART_TCP_Deinit, NULL, NULL, false },
 #endif
 #if PLATFORM_TXW81X
 	//drvdetail:{"name":"TXWCAM",
 	//drvdetail:"title":"TODO",
 	//drvdetail:"descr":"TXW81X Camera.",
 	//drvdetail:"requires":""}
-	{ "TXWCAM", TXW_Cam_Init, TXW_Cam_RunEverySecond, NULL, NULL, NULL, NULL, NULL, false }
+	{ "TXWCAM", TXW_Cam_Init, TXW_Cam_RunEverySecond, NULL, NULL, NULL, NULL, NULL, false },
 #endif
+	//{ "", NULL, NULL, NULL, NULL, NULL, NULL, NULL, false },
 };
 
 

src/driver/drv_uart.c

@@ -272,30 +272,28 @@ void UART_LogBufState(int auartindex) {
   );
 }
 void UART_DebugTool_Run(int auartindex) {
-    byte b;
-    char tmp[128];
-    char *p = tmp;
-    int i;
+	byte b;
+	char tmp[128];
+	char *p = tmp;
 	int bytes = 0;
 
-    for (i = 0; i < sizeof(tmp) - 4; i++) {
-		if (UART_GetDataSizeEx(auartindex)==0) {
-            break;
-        }
-        b = UART_GetByteEx(auartindex,0);
-        if (i) {
-            *p = ' ';
-            p++;
-        }
+	while (UART_GetDataSizeEx(auartindex) > 0) {
+		b = UART_GetByteEx(auartindex, 0);
+
+		int needed = (bytes ? 3 : 2);  // 2 for first byte, 3 (space + 2 hex) after
+		if ((p - tmp) + needed >= sizeof(tmp) - 1) break;
+
+		if (bytes) *p++ = ' ';
+		sprintf(p, "%02X", b);
+		p += 2;
+		UART_ConsumeBytesEx(auartindex, 1);
 		bytes++;
-        sprintf(p, "%02X", b);
-        p += 2;
-        UART_ConsumeBytesEx(auartindex,1);
-    }
-    *p = 0;
+	}
+
+	*p = 0;
 	if (bytes) {
-		addLogAdv(LOG_INFO, LOG_FEATURE_CMD, "UART %i received %i bytes: %s\n",
-			auartindex, bytes, tmp);
+		addLogAdv(LOG_INFO, LOG_FEATURE_CMD,
+			"UART %i received %i bytes: %s\n", auartindex, bytes, tmp);
 	}
 }
 
@@ -307,6 +305,7 @@ void UART_RunEverySecond() {
   }
 }
 // uartInit 115200
+// uartSendASCII Hello123
 commandResult_t CMD_UART_Init(const void *context, const char *cmd, const char *args, int cmdFlags) {
     int baud;
 

src/obk_config.h

@@ -234,6 +234,8 @@
 //#define ENABLE_LFS_SPI						1
 //#define ENABLE_DRIVER_TESTSPIFLASH			1
 
+//#define	ENABLE_DRIVER_UART_TCP					1
+
 // #define	ENABLE_DRIVER_PIR					1
 //#define ENABLE_DRIVER_BKPARTITIONS				1
 #define ENABLE_HA_DISCOVERY						1