!ci maybe dont change termios

Author: laurensvalk

lib/pbio/drv/bluetooth/bluetooth_simulation.c

@@ -6,11 +6,9 @@
 #if PBDRV_CONFIG_BLUETOOTH_SIMULATION
 
 #include <errno.h>
-#include <fcntl.h>
 #include <signal.h>
 #include <stdio.h>
 #include <stdlib.h>
-#include <termios.h>
 #include <unistd.h>
 
 #include "bluetooth.h"
@@ -131,17 +129,17 @@ pbio_error_t pbdrv_bluetooth_controller_initialize(pbio_os_state_t *state, pbio_
 #define STDIN_HEADER_SIZE (1)
 
 static void pbdrv_bluetooth_simulation_tick_handler() {
-    uint8_t buf[256 + STDIN_HEADER_SIZE];
-    ssize_t r = read(STDIN_FILENO, buf + STDIN_HEADER_SIZE, sizeof(buf) - STDIN_HEADER_SIZE);
-
-    if (r > 0) {
-        buf[0] = PBIO_PYBRICKS_COMMAND_WRITE_STDIN;
-        pbdrv_bluetooth_receive_handler(buf, r + STDIN_HEADER_SIZE);
-    } else if (r == 0) {
-        // EOF.
-    } else if (errno != EAGAIN && errno != EWOULDBLOCK) {
-        // No data available.
-    }
+    // uint8_t buf[256 + STDIN_HEADER_SIZE];
+    // ssize_t r = read(STDIN_FILENO, buf + STDIN_HEADER_SIZE, sizeof(buf) - STDIN_HEADER_SIZE);
+
+    // if (r > 0) {
+    //     buf[0] = PBIO_PYBRICKS_COMMAND_WRITE_STDIN;
+    //     pbdrv_bluetooth_receive_handler(buf, r + STDIN_HEADER_SIZE);
+    // } else if (r == 0) {
+    //     // EOF.
+    // } else if (errno != EAGAIN && errno != EWOULDBLOCK) {
+    //     // No data available.
+    // }
 }
 
 static pbio_os_process_t pbdrv_bluetooth_simulation_process;
@@ -170,59 +168,7 @@ static pbio_error_t pbdrv_bluetooth_simulation_process_thread(pbio_os_state_t *s
     return bluetooth_thread_err;
 }
 
-static struct termios oldt;
-
-static void restore_terminal_settings(void) {
-    tcsetattr(STDIN_FILENO, TCSANOW, &oldt);
-}
-
-static void handle_signal(int sig) {
-    restore_terminal_settings();
-    signal(sig, SIG_DFL);
-    raise(sig);
-}
-
 void pbdrv_bluetooth_init_hci(void) {
-    struct termios newt;
-
-    // Save the original terminal settings
-    if (tcgetattr(STDIN_FILENO, &oldt) != 0) {
-        printf("DEBUG: Failed to get terminal attributes\n");
-        return;
-    }
-
-    // Register the cleanup function to restore terminal settings on exit
-    atexit(restore_terminal_settings);
-    signal(SIGINT, handle_signal);
-    signal(SIGTERM, handle_signal);
-
-    newt = oldt;
-
-    // Get one char at a time instead of newline and disable CTRL+C for exit.
-    newt.c_lflag &= ~(ICANON | ECHO | ISIG);
-
-    // MicroPython REPL expects \r for newline.
-    newt.c_iflag |= INLCR;
-    newt.c_iflag &= ~ICRNL;
-
-    if (tcsetattr(STDIN_FILENO, TCSANOW, &newt) != 0) {
-        printf("Failed to set terminal attributes\n");
-        return;
-    }
-
-    // Set stdin non-blocking so we can service it in the runloop like on
-    // embedded hubs.
-    int flags = fcntl(STDIN_FILENO, F_GETFL, 0);
-    if (flags == -1) {
-        printf("Failed to get fcntl flags\n");
-        return;
-    }
-
-    if (fcntl(STDIN_FILENO, F_SETFL, flags | O_NONBLOCK) == -1) {
-        printf("Failed to set non-blocking\n");
-        return;
-    }
-
     bluetooth_thread_err = PBIO_ERROR_AGAIN;
     bluetooth_thread_state = 0;
     pbio_os_process_start(&pbdrv_bluetooth_simulation_process, pbdrv_bluetooth_simulation_process_thread, NULL);

lib/pbio/platform/sim_hub/platform.c

@@ -1,7 +1,13 @@
 // SPDX-License-Identifier: MIT
 // Copyright (c) 2025 The Pybricks Authors
 
+#include <errno.h>
+#include <fcntl.h>
+#include <signal.h>
 #include <stdio.h>
+#include <stdlib.h>
+#include <termios.h>
+#include <unistd.h>
 
 #include "../../drv/motor_driver/motor_driver_virtual_simulation.h"
 
@@ -133,9 +139,6 @@ const pbdrv_motor_driver_virtual_simulation_platform_data_t
     },
 };
 
-extern uint8_t pbsys_hmi_native_program_buf[PBDRV_CONFIG_BLOCK_DEVICE_RAM_SIZE];
-extern uint32_t pbsys_hmi_native_program_size;
-
 int main(int argc, char **argv) {
 
     if (argc > 1) {
@@ -150,6 +153,8 @@ int main(int argc, char **argv) {
         }
 
         // Read the multi-mpy file from pipe.
+        extern uint8_t pbsys_hmi_native_program_buf[PBDRV_CONFIG_BLOCK_DEVICE_RAM_SIZE];
+        extern uint32_t pbsys_hmi_native_program_size;
         pbsys_hmi_native_program_size = fread(pbsys_hmi_native_program_buf, 1, sizeof(pbsys_hmi_native_program_buf), pipe);
         pclose(pipe);
 
@@ -159,6 +164,48 @@ int main(int argc, char **argv) {
         }
     }
 
+    // struct termios term_old, term_new;
+
+    // // Save the original terminal settings
+    // if (tcgetattr(STDIN_FILENO, &term_old) != 0) {
+    //     printf("DEBUG: Failed to get terminal attributes\n");
+    //     return 0;
+    // }
+    // term_new = term_old;
+
+    // // Get one char at a time instead of newline and disable CTRL+C for exit.
+    // term_new.c_lflag &= ~(ICANON | ECHO | ISIG);
+
+    // // MicroPython REPL expects \r for newline.
+    // term_new.c_iflag |= INLCR;
+    // term_new.c_iflag &= ~ICRNL;
+
+    // if (tcsetattr(STDIN_FILENO, TCSANOW, &term_new) != 0) {
+    //     printf("Failed to set terminal attributes\n");
+    //     return 0;
+    // }
+
+    // // Set stdin non-blocking so we can service it in the runloop like on
+    // // embedded hubs.
+    // int flags = fcntl(STDIN_FILENO, F_GETFL, 0);
+    // if (flags == -1) {
+    //     printf("Failed to get fcntl flags\n");
+    //     return 0;
+    // }
+
+    // if (fcntl(STDIN_FILENO, F_SETFL, flags | O_NONBLOCK) == -1) {
+    //     printf("Failed to set non-blocking\n");
+    //     return 0;
+    // }
+
     extern void _main(void);
     _main();
+
+    // // Restore terminal settings.
+    // if (tcsetattr(STDIN_FILENO, TCSANOW, &term_old) != 0) {
+    //     printf("Failed to restore terminal attributes\n");
+    //     return 0;
+    // }
+
+    return 0;
 }