Re: earlycon: no match?

[Peter Hurley]

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)
		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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/