On Thu, May 08, 2014 at 03:52:19PM +0200, Arnd Bergmann wrote: > If we build a kernel with PM_SUSPEND set and no PM_SLEEP, > we get a build warning in the xhci-plat driver about unused > functions. > > To fix this, use "#ifdef CONFIG_PM_SLEEP", like we do in most > other drivers nowadays. > > Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> > Cc: Mathias Nyman <mathias.nyman@xxxxxxxxx> > --- > drivers/usb/host/xhci-plat.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/usb/host/xhci-plat.c b/drivers/usb/host/xhci-plat.c > index 151901c..3473296 100644 > --- a/drivers/usb/host/xhci-plat.c > +++ b/drivers/usb/host/xhci-plat.c > @@ -202,7 +202,7 @@ static int xhci_plat_remove(struct platform_device *dev) > return 0; > } > > -#ifdef CONFIG_PM > +#ifdef CONFIG_PM_SLEEP can we just drop the ifdeferry altogether, including from SET_*_OPS macros ? This is just ridiculous that every merge window we have tens of patches fixing the ifdef in several drivers. -- balbi
Attachment:
signature.asc
Description: Digital signature
