[gdb] Allow gdbstub to be started on an arbitrary serial port

Signed-off-by: Michael Brown <mcb30@ipxe.org>

diff --git a/src/core/gdbserial.c b/src/core/gdbserial.c
index 802463f4508b89595bc1ee29383ba3b7f96bd9a8..0983f255743154ebe25cec594d214a7bbeb1f2dd 100644 (file)
--- a/src/core/gdbserial.c
+++ b/src/core/gdbserial.c
@@ -24,6 +24,8 @@
 FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL );
 
 #include <stddef.h>
+#include <stdio.h>
+#include <stdlib.h>
 #include <assert.h>
 #include <ipxe/uart.h>
 #include <ipxe/gdbstub.h>
@@ -54,6 +56,8 @@ FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL );
 /** GDB serial UART */
 static struct uart gdbserial_uart;
 
+struct gdb_transport serial_gdb_transport __gdb_transport;
+
 static size_t gdbserial_recv ( char *buf, size_t len ) {
 
        assert ( len > 0 );
@@ -69,15 +73,27 @@ static void gdbserial_send ( const char *buf, size_t len ) {
        }
 }
 
-static int gdbserial_init ( int argc __unused, char **argv __unused ) {
-       int rc;
+static int gdbserial_init ( int argc, char **argv ) {
+       unsigned int port;
+       char *endp;
 
-       if ( ( rc = uart_select ( &gdbserial_uart, GDBSERIAL_PORT ) ) != 0 )
-               return rc;
+       if ( argc == 0 ) {
+               port = GDBSERIAL_PORT;
+       } else if ( argc == 1 ) {
+               port = strtoul ( argv[0], &endp, 10 );
+               if ( *endp ) {
+                       printf ( "serial: invalid port\n" );
+                       return 1;
+               }
+       } else {
+               printf ( "serial: syntax <port>\n" );
+               return 1;
+       }
 
-       if ( ( rc = uart_init ( &gdbserial_uart, GDBSERIAL_BAUD,
-                               GDBSERIAL_LCR ) ) != 0 )
-               return rc;
+       if ( ! gdbserial_configure ( port, GDBSERIAL_BAUD, GDBSERIAL_LCR ) ) {
+               printf ( "serial: unable to configure\n" );
+               return 1;
+       }
 
        return 0;
 }
@@ -88,3 +104,16 @@ struct gdb_transport serial_gdb_transport __gdb_transport = {
        .recv = gdbserial_recv,
        .send = gdbserial_send,
 };
+
+struct gdb_transport * gdbserial_configure ( unsigned int port,
+                                            unsigned int baud, uint8_t lcr ) {
+       int rc;
+
+       if ( ( rc = uart_select ( &gdbserial_uart, port ) ) != 0 )
+               return NULL;
+
+       if ( ( rc = uart_init ( &gdbserial_uart, baud, lcr ) ) != 0 )
+               return NULL;
+
+       return &serial_gdb_transport;
+}

diff --git a/src/include/ipxe/gdbserial.h b/src/include/ipxe/gdbserial.h
index db9cb1567aee0ec919e029fc4e88a610e176c0d8..e1040c94e62013378f1c5baa5a23c9ea12b208ca 100644 (file)
--- a/src/include/ipxe/gdbserial.h
+++ b/src/include/ipxe/gdbserial.h
@@ -9,4 +9,12 @@
 
 FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL );
 
+#include <stdint.h>
+
+struct gdb_transport;
+
+extern struct gdb_transport * gdbserial_configure ( unsigned int port,
+                                                   unsigned int baud,
+                                                   uint8_t lcr );
+
 #endif /* _IPXE_GDBSERIAL_H */