[PATCH v3 16/16] x86: Allow locating UARTs by device ID

[Bin Meng]

Hi Simon,

On Mon, Mar 27, 2023 at 12:17 PM Simon Glass <sjg at chromium.org> wrote:
>
> When coreboot does not pass a UART in its sysinfo struct, there is no
> easy way to find it out. Add a way to specify known UARTs so we can
> find them without needing help from coreboot.
>
> Since coreboot does not actually init the serial device when serial is
> disabled, it is not possible to make it add this information to the
> sysinfo table.
>
> Also, we cannot use the class information, since we don't know which
> UART is being used. For example, with Alder Lake there are two:
>
> 00.16.00   0x8086     0x51e0     Simple comm. controller 0x80
> 00.1e.00   0x8086     0x51a8     Simple comm. controller 0x80
>
> In our case the second one is the right one, but thre is no way to
> distinguish it from the first one without using the device ID.
>
> If we have Adler Lake hardware which uses a different UART, we could
> perhaps look at the ACPI tables, or the machine information passed in
> the SMBIOS tables.
>
> This was discussed previously before: [1]
>
> [1] https://patchwork.ozlabs.org/project/uboot/patch/20210407163159.3.I967ea8c85e009f870c7aa944372d32c990f1b14a@changeid/
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> (no changes since v2)
>
> Changes in v2:
> - Move this patch to last in the series, so it can be dropped if needed

Since now we can parse the DBG2 ACPI table to get the UART info, I'd
rather not apply this patch.

>
>  arch/x86/dts/coreboot.dts        |  4 ++++
>  drivers/serial/serial_coreboot.c | 41 ++++++++++++++++++++++++++++++++
>  include/pci_ids.h                |  3 +++
>  3 files changed, 48 insertions(+)
>

Regards,
Bin