The patch titled serial: allow port type to be specified when calling serial8250_register_port has been added to the -mm tree. Its filename is serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find out what to do about this The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ ------------------------------------------------------ Subject: serial: allow port type to be specified when calling serial8250_register_port From: David Daney <ddaney@xxxxxxxxxxxxxxxxxx> Add flag value UPF_FIXED_TYPE which specifies that the UART type is known and should not be probed. For this case the UARTs properties are just copied out of the uart_config entry. This allows us to keep SOC specific 8250 probe code out of 8250.c. In this case we know the serial hardware will not be changing as it is on the same silicon as the CPU, and we can specify it with certainty in the board/cpu setup code. The alternative is to load up 8250.c with a bunch of OCTEON specific special cases in the probing code. Signed-off-by: David Daney <ddaney@xxxxxxxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- drivers/serial/8250.c | 9 +++++++++ drivers/serial/serial_core.c | 7 +++++-- include/linux/serial_8250.h | 1 + include/linux/serial_core.h | 2 ++ 4 files changed, 17 insertions(+), 2 deletions(-) diff -puN drivers/serial/8250.c~serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port drivers/serial/8250.c --- a/drivers/serial/8250.c~serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port +++ a/drivers/serial/8250.c @@ -2903,6 +2903,7 @@ static int __devinit serial8250_probe(st port.mapbase = p->mapbase; port.hub6 = p->hub6; port.private_data = p->private_data; + port.type = p->type; port.serial_in = p->serial_in; port.serial_out = p->serial_out; port.dev = &dev->dev; @@ -3058,6 +3059,14 @@ int serial8250_register_port(struct uart uart->port.private_data = port->private_data; if (port->dev) uart->port.dev = port->dev; + + if (port->flags & UPF_FIXED_TYPE) { + uart->port.type = port->type; + uart->port.fifosize = uart_config[port->type].fifo_size; + uart->capabilities = uart_config[port->type].flags; + uart->tx_loadsz = uart_config[port->type].tx_loadsz; + } + set_io_from_upio(&uart->port); /* Possibly override default I/O functions. */ if (port->serial_in) diff -puN drivers/serial/serial_core.c~serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port drivers/serial/serial_core.c --- a/drivers/serial/serial_core.c~serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port +++ a/drivers/serial/serial_core.c @@ -2198,11 +2198,14 @@ uart_configure_port(struct uart_driver * * Now do the auto configuration stuff. Note that config_port * is expected to claim the resources and map the port for us. */ - flags = UART_CONFIG_TYPE; + flags = 0; if (port->flags & UPF_AUTO_IRQ) flags |= UART_CONFIG_IRQ; if (port->flags & UPF_BOOT_AUTOCONF) { - port->type = PORT_UNKNOWN; + if (!(port->flags & UPF_FIXED_TYPE)) { + port->type = PORT_UNKNOWN; + flags |= UART_CONFIG_TYPE; + } port->ops->config_port(port, flags); } diff -puN include/linux/serial_8250.h~serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port include/linux/serial_8250.h --- a/include/linux/serial_8250.h~serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port +++ a/include/linux/serial_8250.h @@ -28,6 +28,7 @@ struct plat_serial8250_port { unsigned char iotype; /* UPIO_* */ unsigned char hub6; upf_t flags; /* UPF_* flags */ + unsigned int type; /* If UPF_FIXED_TYPE */ unsigned int (*serial_in)(struct uart_port *, int); void (*serial_out)(struct uart_port *, int, int); }; diff -puN include/linux/serial_core.h~serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port include/linux/serial_core.h --- a/include/linux/serial_core.h~serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port +++ a/include/linux/serial_core.h @@ -296,6 +296,8 @@ struct uart_port { #define UPF_MAGIC_MULTIPLIER ((__force upf_t) (1 << 16)) #define UPF_CONS_FLOW ((__force upf_t) (1 << 23)) #define UPF_SHARE_IRQ ((__force upf_t) (1 << 24)) +/* The exact UART type is known and should not be probed. */ +#define UPF_FIXED_TYPE ((__force upf_t) (1 << 27)) #define UPF_BOOT_AUTOCONF ((__force upf_t) (1 << 28)) #define UPF_FIXED_PORT ((__force upf_t) (1 << 29)) #define UPF_DEAD ((__force upf_t) (1 << 30)) _ Patches currently in -mm which might be from ddaney@xxxxxxxxxxxxxxxxxx are mips-make-bug-__noreturn.patch 8250-dont-clobber-spinlocks.patch 8250-serial-driver-changes-to-support-future-cavium-octeon-serial-patches.patch serial-allow-port-type-to-be-specified-when-calling-serial8250_register_port.patch 8250-allow-port-type-to-specify-bugs-that-are-not-probed-for.patch serial-uart-driver-changes-for-cavium-octeon.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html