make new slip proto work

cmd/cmd.h

@@ -20,7 +20,7 @@ typedef struct __attribute__((__packed__)) {
 typedef struct __attribute__((__packed__)) {
   uint16_t  cmd;      // command to perform, from CmdName enum
   uint32_t  callback; // callback pointer to embed in response
-  uint32_t  _return;  // return value to embed in response (?)
+  uint32_t  _return;  // token to embed in response
   uint16_t  argc;     // number of arguments to command
   CmdArg    args[0];  // really args[argc]
 } CmdPacket;
@@ -33,7 +33,7 @@ typedef struct {
 
 typedef enum {
   CMD_NULL = 0,
-  CMD_RESET,          // reset esp (not honored in this implementation)
+  CMD_SYNC,           // synchronize and clear
   CMD_IS_READY,       // health-check
   CMD_WIFI_CONNECT,   // (3) connect to AP (not honored in this implementation)
   CMD_MQTT_SETUP,

cmd/handlers.c

@@ -19,8 +19,7 @@
 #endif
 
 static uint32_t CMD_Null(CmdPacket *cmd);
-static uint32_t CMD_IsReady(CmdPacket *cmd);
+//static uint32_t CMD_Reset(CmdPacket *cmd);
-static uint32_t CMD_Reset(CmdPacket *cmd);
 static uint32_t CMD_WifiConnect(CmdPacket *cmd);
 static uint32_t CMD_AddCallback(CmdPacket *cmd);
 
@@ -30,9 +29,9 @@ static bool wifiCbAdded = false;
 
 // Command dispatch table for serial -> ESP commands
 const CmdList commands[] = {
-  {CMD_NULL,            CMD_Null},
+  {CMD_NULL,            CMD_Null},        // no-op
-  {CMD_RESET,           CMD_Reset},
+  //{CMD_SYNC,            CMD_Sync},        // synchronize
-  {CMD_IS_READY,        CMD_IsReady},
+  //{CMD_IS_READY,        CMD_IsReady},
   {CMD_WIFI_CONNECT,    CMD_WifiConnect},
 #ifdef MQTT
   {CMD_MQTT_SETUP,      MQTTCMD_Setup},
@@ -55,18 +54,13 @@ const CmdList commands[] = {
 #define MAX_CALLBACKS 12
 cmdCallback callbacks[MAX_CALLBACKS]; // cleared in CMD_Reset
 
-// Command handler for IsReady (healthcheck) command
-static uint32_t ICACHE_FLASH_ATTR
-CMD_IsReady(CmdPacket *cmd) {
-  return 1;
-}
-
 // Command handler for Null command
 static uint32_t ICACHE_FLASH_ATTR
 CMD_Null(CmdPacket *cmd) {
   return 1;
 }
 
+#if 0
 // Command handler for Reset command, this was originally to reset the ESP but we don't want to
 // do that in esp-link. It is still good to clear any information the ESP has about the attached
 // uC.
@@ -76,6 +70,7 @@ CMD_Reset(CmdPacket *cmd) {
   os_memset(callbacks, 0, sizeof(callbacks));
   return 1;
 }
+#endif
 
 static uint32_t ICACHE_FLASH_ATTR
 CMD_AddCb(char* name, uint32_t cb) {

include/user_config.h

@@ -27,7 +27,7 @@
 #define RESTCMD_DBG
 #define SERBR_DBG
 #define SERLED_DBG
-#undef SLIP_DBG
+#define SLIP_DBG
 #define UART_DBG
 #define MDNS_DBG
 #define OPTIBOOT_DBG

serial/slip.c

@@ -45,13 +45,12 @@ slip_process() {
     if (crc == rcv) {
       CMD_parse_packet((uint8_t*)slip_buf, slip_len-2);
     } else {
-      os_printf("SLIP: bad CRC, crc=%x rcv=%x\n", crc, rcv);
+      os_printf("SLIP: bad CRC, crc=%04x rcv=%04x len=%d\n", crc, rcv, slip_len);
 
       for (short i=0; i<slip_len; i++) {
         if (slip_buf[i] >= ' ' && slip_buf[i] <= '~') {
           DBG("%c", slip_buf[i]);
-        }
+        } else {
-        else {
           DBG("\\%02X", slip_buf[i]);
         }
       }
@@ -79,12 +78,12 @@ static void ICACHE_FLASH_ATTR
 slip_parse_char(char c) {
   if (c == SLIP_END) {
     // either start or end of packet, process whatever we may have accumulated
+    DBG("SLIP: start or end len=%d inpkt=%d\n", slip_len, slip_inpkt);
     if (slip_len > 0) {
-      if (slip_inpkt) slip_process(); else console_process(slip_buf, slip_len);
+      if (slip_len > 2 && slip_inpkt) slip_process();
+      else console_process(slip_buf, slip_len);
     }
     slip_reset();
-    //slip_inpkt = true;
-    DBG("SLIP: start or end\n");
   } else if (slip_escaped) {
     // prev char was SLIP_ESC
     if (c == SLIP_ESC_END) c = SLIP_END;
@@ -94,7 +93,7 @@ slip_parse_char(char c) {
   } else if (slip_inpkt && c == SLIP_ESC) {
     slip_escaped = true;
   } else {
-    if (slip_len == 0 && slip_printable(c)) {
+    if (slip_len == 1 && slip_printable(slip_buf[0]) && slip_printable(c)) {
       // start of packet and it's a printable character, we're gonna assume that this is console text
       slip_inpkt = false;
     }
@@ -111,8 +110,8 @@ slip_parse_buf(char *buf, short length) {
 
   // if we're in-between packets (debug console) then print it now
   if (!slip_inpkt && length > 0) {
-    slip_process();
+    console_process(slip_buf, slip_len);
-    slip_reset();
+    slip_len = 0;
   }
 }