On 10 October 2017 at 21:12, Frédéric Danis <frederic.danis.oss@xxxxxxxxx> wrote: > This patch allows SerDev module to manage serial devices declared as > attached to an UART in ACPI table. > > acpi_serdev_add_device() callback will only take into account entries > without enumerated flag set. This flags is set for all entries during > ACPI scan, except for SPI and I2C serial devices, and for UART with > 2nd patch in the series. > > Check if a serdev device as been allocated during acpi_walk_namespace() > to prevent serdev controller registration instead of the tty-class device. <snip> Hi Fred, I tested the v2 patch and can confirm it works: Before v1 patch: $ dmesg | grep tty [ 0.000000] console [tty0] enabled [ 2.409003] 00:01: ttyS0 at I/O 0x3f8 (irq = 4, base_baud = 115200) is a 16550A [ 5.380336] 8086228A:00: ttyS4 at MMIO 0x9151b000 (irq = 39, base_baud = 2764800) is a 16550A [ 5.388412] 8086228A:01: ttyS5 at MMIO 0x91519000 (irq = 40, base_baud = 2764800) is a 16550A $ After v1 patch: $ dmesg | grep tty [ 0.000000] console [tty0] enabled [ 2.416973] 00:01: ttyS0 at I/O 0x3f8 (irq = 4, base_baud = 115200) is a 16550A [ 2.417317] serial serial0: tty port ttyS0 registered [ 5.928492] 8086228A:00: ttyS4 at MMIO 0x9151b000 (irq = 39, base_baud = 2764800) is a 16550A [ 5.982195] serial serial1: tty port ttyS4 registered [ 6.004426] 8086228A:01: ttyS5 at MMIO 0x91519000 (irq = 40, base_baud = 2764800) is a 16550A [ 6.019369] serial serial2: tty port ttyS5 registered $ After v2 patch: $ dmesg | grep tty [ 0.000000] console [tty0] enabled [ 2.420295] 00:01: ttyS0 at I/O 0x3f8 (irq = 4, base_baud = 115200) is a 16550A [ 5.884777] 8086228A:00: ttyS4 at MMIO 0x9151b000 (irq = 39, base_baud = 2764800) is a 16550A [ 5.917980] serial serial0: tty port ttyS4 registered [ 5.933160] 8086228A:01: ttyS5 at MMIO 0x91519000 (irq = 40, base_baud = 2764800) is a 16550A $ Regards, Ian -- 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