Re: [PATCH v2 1/2] serdev: Add ACPI support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux