serial: 8250_dw: remove useless ACPI ID check
authorHeikki Krogerus <heikki.krogerus@linux.intel.com>
Wed, 18 Mar 2015 10:55:12 +0000 (12:55 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 26 Mar 2015 22:04:42 +0000 (23:04 +0100)
Having ACPI handle does not mean the same as having ACPI
identifier. The check is in any case useless, but having it
prevents this driver from being used for example with
multifunctional PCI devices, such as the newer Intel LPSS
devices.

Signed-off-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/8250/8250_dw.c

index d641886b254a3bc7fd774d56006cdfd9ccc1e548..cc943fe3fd682c11e17b2071d1f05e13cc4ec8bc 100644 (file)
@@ -391,15 +391,10 @@ static int dw8250_probe_of(struct uart_port *p,
 static int dw8250_probe_acpi(struct uart_8250_port *up,
                             struct dw8250_data *data)
 {
-       const struct acpi_device_id *id;
        struct uart_port *p = &up->port;
 
        dw8250_setup_port(up);
 
-       id = acpi_match_device(p->dev->driver->acpi_match_table, p->dev);
-       if (!id)
-               return -ENODEV;
-
        if (!p->uartclk)
                if (device_property_read_u32(p->dev, "clock-frequency",
                                             &p->uartclk))