On 05/04/2015 10:01 AM, Peter Hurley wrote:
> Hi Robert,
>
> On 05/03/2015 05:10 PM, Robert Schwebel wrote:
>> Hi Peter,
>>
>> with 4.1-rc1, my boxes with early console enabled show something like
>> this (the example is vexpress, but it for example also happens on an
>> AM335x board):
>>
>> earlycon: no match for ttyAMA0,38400n8
>
> This shouldn't impact any previous earlycon setup. Are you saying
> you're seeing a regression?
>
> How do you have early console enabled, via the command line or via DT?
>
>
>> The box was booted with "console=ttyAMA0,38400n8" on the commandline.
>> If I understand this right, the code in drivers/tty/serial/earlycon.c
>> calls setup_earlycon() with the string above ("ttyAMA0,38400n8") and
>> fails to find that string in the "names" part of the __earlycon_table,
>> because for the pl011 component on vexpress, the early console was
>> registered in drivers/tty/serial/amba-pl011.c with:
>>
>> OF_EARLYCON_DECLARE(pl011, "arm,pl011", pl011_early_console_setup);
>> ^^^^^ name
>>
>> So isn't that trying to match "ttyAMA0" against "arm,pl011"? I have the
>> feeling that I didn't understand the logic behind that.
>>
>> Can you elaborate about how this is supposed to work correctly?
>
> Yeah, I've been meaning to write about this but simply haven't had the
> time yet; apologies for that.
>
> The facility is hopefully best explained by the existing 8250 exemplar.
> Normally, an 8250 early console is started via command line with a
> command line parameter like:
>
> earlycon=uart,io,0x2f8,115200n8
>
> Since 2007, an 8250 early console can also be started via command line
> using console= instead, like:
>
> console=uart,io,0x2f8,115200n8
>
> In this alternate form, this early console will go on to become the
> corresponding ttyS console.
>
> However, that functionality was exclusive to 8250 console/earlycon.
> To get this same behavior for the amba-pl011 console would look
> something like:
>
> /* drivers/tty/serial/amba-pl011.c */
>
> /* returns 0 if the console matches; otherwise, non-zero to use default matching */
> static int pl011_console_match(struct console *co, char *name, int idx, char *options)
> {
> unsigned char iotype;
> unsigned long addr;
>
> if (strncmp(name, "pl" 2) != 0 || idx != 11)
^
,
Sorry, missed that.
> return -ENODEV;
>
> if (uart_parse_earlycon(options, &iotype, &addr, &options))
> return -ENODEV;
>
> /* find the port from the addr */
> for (i = 0; i < ARRAY_SIZE(amba_ports); i++) {
> if (amba_ports[i] == NULL)
> continue;
> if (port->mapbase != addr)
> continue;
>
> co->index = i;
> return pl011_console_setup(co, options);
> }
>
> return -ENODEV;
> }
>
> ...
>
> static struct console amba_console = {
> ...
> .match = pl011_console_match,
> ...
> };
>
>
> Regards,
> Peter Hurley
>
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
