serial: 8250_dw: Drop wrong use of ACPI_PTR()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Tue, 5 Oct 2021 13:45:16 +0000 (16:45 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 10 Oct 2021 13:20:55 +0000 (15:20 +0200)
ACPI_PTR() is more harmful than helpful. For example, in this case
if CONFIG_ACPI=n, the ID table left unused which is not what we want.

Instead of adding ifdeffery here and there, drop ACPI_PTR().

Fixes: 6a7320c4669f ("serial: 8250_dw: Add ACPI 5.0 support")
Reported-by: Daniel Palmer <daniel@0x0f.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20211005134516.23218-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/8250/8250_dw.c

index a3a0154da567da4a37fb9cc7a8fb1ba631b64bc8..49559731bbcf1cf595fef5b787d528afa513889c 100644 (file)
@@ -726,7 +726,7 @@ static struct platform_driver dw8250_platform_driver = {
                .name           = "dw-apb-uart",
                .pm             = &dw8250_pm_ops,
                .of_match_table = dw8250_of_match,
-               .acpi_match_table = ACPI_PTR(dw8250_acpi_match),
+               .acpi_match_table = dw8250_acpi_match,
        },
        .probe                  = dw8250_probe,
        .remove                 = dw8250_remove,