[PATCH 1/3] i386/pc386: Add support for the gdb stub to use available console drivers.

Chris Johns chrisj at rtems.org
Wed Apr 20 23:42:53 UTC 2016


On 21/04/2016 9:01 AM, Joel Sherrill wrote:
>      Console/Printk Device Selection
>      ===============================
>     @@ -19,9 +19,9 @@ in the following order of priority:
>      + VGA and keyboard
>      + COM1 through COM4aaa
> 
> What is the aaa?
>

No idea, must have been cat on the keyboard.

>     +
>     +    if (comma) {
>     +      option = comma + 1;
>     +      baudrate = strtoul(option, 0, 10);
>     +      switch (baudrate) {
>     +        case 115200:
>     +        case 57600:
>     +        case 38400:
>     +        case 19200:
>     +        case 9600:
>     +        case 4800:
>     +          port->pDeviceParams = (void*) baudrate;
>     +          BSPBaseBaud = baudrate; /* REMOVE ME */
>     +          break;
>     +        default:
>     +          printk("invalid option (--gdb): bad baudrate\n");
>     +          return;
>     +      }
>     +    }
> 
> 
> Is there anyway the code parsing this option can be shared with that
> for the com ports?

Yes that could be done but I do not think so for this patch set. This is
in code in shared/comm and the BSP code needs to be reorganised.

Given the conversation with Sebastian about consoles this BSP needs some
rework. Maybe this can happen then.

Chris


More information about the devel mailing list