Commit 78d090b0 authored by Rafael J. Wysocki's avatar Rafael J. Wysocki Committed by Jesse Barnes

PCI / PM: Remove unnecessary error variable from acpi_dev_run_wake()

The result returned by acpi_dev_run_wake() is always either -EINVAL
or -ENODEV, while obviously it should return 0 on success.  The
problem is that the leftover error variable, that's not really used
in the function, is initialized with -ENODEV and then returned
without modification.

To fix this issue remove the error variable from acpi_dev_run_wake()
and make the function return 0 on success as appropriate.
Signed-off-by: default avatarRafael J. Wysocki <rjw@sisk.pl>
Signed-off-by: default avatarJesse Barnes <jbarnes@virtuousgeek.org>
parent 6af8bef1
...@@ -285,7 +285,6 @@ static int acpi_dev_run_wake(struct device *phys_dev, bool enable) ...@@ -285,7 +285,6 @@ static int acpi_dev_run_wake(struct device *phys_dev, bool enable)
{ {
struct acpi_device *dev; struct acpi_device *dev;
acpi_handle handle; acpi_handle handle;
int error = -ENODEV;
if (!device_run_wake(phys_dev)) if (!device_run_wake(phys_dev))
return -EINVAL; return -EINVAL;
...@@ -305,7 +304,7 @@ static int acpi_dev_run_wake(struct device *phys_dev, bool enable) ...@@ -305,7 +304,7 @@ static int acpi_dev_run_wake(struct device *phys_dev, bool enable)
acpi_disable_wakeup_device_power(dev); acpi_disable_wakeup_device_power(dev);
} }
return error; return 0;
} }
static void acpi_pci_propagate_run_wake(struct pci_bus *bus, bool enable) static void acpi_pci_propagate_run_wake(struct pci_bus *bus, bool enable)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment