[Silabs][SiWx917] Fixing uart to read only once (#25902)

* fixing uart to read only once

* Restyled by clang-format

* Uart fix to read character by character

* restyling the PR

* removing the duplicate files

* removing the siwx917 file

* Restyled by gn

* adding some comments

---------

Co-authored-by: Restyled.io <[email protected]>

Author: 2 people

examples/platform/silabs/SiWx917/uart.cpp

@@ -33,13 +33,8 @@ extern ARM_DRIVER_USART Driver_USART0;
 static ARM_DRIVER_USART * UARTdrv = &Driver_USART0;
 
 ARM_USART_CAPABILITIES drv_capabilities;
-ARM_USART_STATUS status;
-
-#define BUFFER_SIZE 1
-uint8_t rx_buffer;
 
 #define BAUD_VALUE 115200
-
 #define UART_CONSOLE_ERR -1 // Negative value in case of UART Console action failed. Triggers a failure for PW_RPC
 
 void ARM_USART_SignalEvent(uint32_t event);
@@ -141,10 +136,6 @@ void uartConsoleInit(void)
         DEBUGOUT("\r\n Receives data success  \r\n");
     }
 
-    // Creating the receive event as a temp buffer
-    // this is not used anywhere
-    status = UARTdrv->Receive(&rx_buffer, 1);
-
     NVIC_EnableIRQ(USART0_IRQn);
 
     NVIC_SetPriority(USART0_IRQn, 7);

src/lib/shell/BUILD.gn

@@ -50,15 +50,11 @@ static_library("shell") {
       "MainLoopESP32.cpp",
       "streamer_esp32.cpp",
     ]
-  } else if (chip_device_platform == "efr32") {
+  } else if (chip_device_platform == "efr32" ||
+             chip_device_platform == "SiWx917") {
     sources += [
-      "MainLoopEFR32.cpp",
-      "streamer_efr32.cpp",
-    ]
-  } else if (chip_device_platform == "SiWx917") {
-    sources += [
-      "MainLoopEFR32.cpp",
-      "streamer_efr32.cpp",
+      "MainLoopSilabs.cpp",
+      "streamer_silabs.cpp",
     ]
   } else if (chip_device_platform == "k32w0") {
     sources += [

src/lib/shell/MainLoopEFR32.cpp renamed to src/lib/shell/MainLoopSilabs.cpp

@@ -49,11 +49,22 @@ void ReadLine(char * buffer, size_t max)
             break;
         }
 
+#ifdef BRD4325A
+        // for 917 SoC board, we need to create a rx event before we wait for the shell activity
+        // NotifyShellProcessFromISR() is called once the buffer is filled
+        if (streamer_read(streamer_get(), buffer + line_sz, 1) != 1)
+        {
+            continue;
+        }
+#endif
         chip::WaitForShellActivity();
+#ifndef BRD4325A
+        // for EFR32 boards
         if (streamer_read(streamer_get(), buffer + line_sz, 1) != 1)
         {
             continue;
         }
+#endif
 
         // Process character we just read.
         switch (buffer[line_sz])