On Sun, Dec 31, 2023 at 8:35 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > This avoids a build warning due to the use of ACPI_PTR(). > Given the driver already has APCI specific code under CONFIG_ACPI move > the table rather than removing the ACPI_PTR() call as we already > have the complexity of CONFIG_ACPI. ... > +static const struct acpi_device_id kx_acpi_match[] = { > + {"KXCJ1013", KXCJK1013}, > + {"KXCJ1008", KXCJ91008}, > + {"KXCJ9000", KXCJ91008}, > + {"KIOX0008", KXCJ91008}, > + {"KIOX0009", KXTJ21009}, > + {"KIOX000A", KXCJ91008}, > + {"KIOX010A", KXCJ91008}, /* KXCJ91008 in the display of a yoga 2-in-1 */ > + {"KIOX020A", KXCJ91008}, /* KXCJ91008 in the base of a yoga 2-in-1 */ > + {"KXTJ1009", KXTJ21009}, > + {"KXJ2109", KXTJ21009}, > + {"SMO8500", KXCJ91008}, > + { }, Can you drop the comma, while at it? > +}; -- With Best Regards, Andy Shevchenko