On 26 December 2017 at 01:50, Rafael J. Wysocki <r...@rjwysocki.net> wrote: > From: Rafael J. Wysocki <rafael.j.wyso...@intel.com> > > Since both device_wakeup_enable() and device_wakeup_disable() check > if dev is not NULL and whether or not power.can_wakeup is set for it, > device_set_wakeup_enable() doesn't have to do that, so drop that > check from it. > > No intentional changes in functionality. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wyso...@intel.com>
Reviewed-by: Ulf Hansson <ulf.hans...@linaro.org> Kind regards Uffe > --- > drivers/base/power/wakeup.c | 3 --- > 1 file changed, 3 deletions(-) > > Index: linux-pm/drivers/base/power/wakeup.c > =================================================================== > --- linux-pm.orig/drivers/base/power/wakeup.c > +++ linux-pm/drivers/base/power/wakeup.c > @@ -464,9 +464,6 @@ EXPORT_SYMBOL_GPL(device_init_wakeup); > */ > int device_set_wakeup_enable(struct device *dev, bool enable) > { > - if (!dev || !dev->power.can_wakeup) > - return -EINVAL; > - > return enable ? device_wakeup_enable(dev) : > device_wakeup_disable(dev); > } > EXPORT_SYMBOL_GPL(device_set_wakeup_enable); >