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 d641886..cc943fe 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))