On Fri, Jul 14, 2023 at 10:24:57PM +0200, Amadeusz Sławiński wrote: > Use PCI device IDs from pci_ids.h header. BSW replaces CHV, as 0x22a8 > was added in PCI header as BSW ID for consistency, as they are same > (similar) platforms. The ACPI IDs are used only internally and lower > 16 bits uniquely define the device as vendor ID for Intel is 8086 for > all of them. Use PCI_DEVICE_DATA() to match PCI device to be consistent > with other Intel audio drivers. One nit-pick below, otherwise Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> ... > - unsigned int dev_id; > + unsigned short dev_id; Thinking more about this change I would probably add a comment on top of sst_alloc_drv_context() call in sst_acpi.c to tell that in this case the driver handles PCI IDs in ACPI and we are using only device ID part. (If real ACPI ID appears, the kstrtouint() returns error, so we are fine with the change) So, summarize above and add a comment. -- With Best Regards, Andy Shevchenko
