Суббота, 22 февраля 2014, 11:59 -03:00 от Fabio Estevam <festevam@xxxxxxxxx>: > On Sat, Feb 22, 2014 at 11:57 AM, Alexander Shiyan <shc_work@xxxxxxx> wrote: > > Суббота, 22 февраля 2014, 11:51 -03:00 от Fabio Estevam <festevam@xxxxxxxxx>: > >> On Sat, Feb 22, 2014 at 11:46 AM, Alexander Shiyan <shc_work@xxxxxxx> wrote: > >> > Signed-off-by: Alexander Shiyan <shc_work@xxxxxxx> > >> > --- > >> > drivers/mmc/host/mxcmmc.c | 13 +++---------- > >> > 1 file changed, 3 insertions(+), 10 deletions(-) > >> > > >> > diff --git a/drivers/mmc/host/mxcmmc.c b/drivers/mmc/host/mxcmmc.c > >> > index 84d630e..86401c0 100644 > >> > --- a/drivers/mmc/host/mxcmmc.c > >> > +++ b/drivers/mmc/host/mxcmmc.c > >> > @@ -1220,8 +1220,7 @@ static int mxcmci_remove(struct platform_device *pdev) > >> > return 0; > >> > } > >> > > >> > -#ifdef CONFIG_PM > >> > -static int mxcmci_suspend(struct device *dev) > >> > >> Doesn't this need to be protected by CONFIG_PM_SLEEP? > > > > No, this already protected by SIMPLE_DEV_PM_OPS(), > > see include/linux/pm.h. > > but then you would get a warning saying that "mxcmci_suspend/resume" > are defined by not used" when CONFIG_PM_SLEEP=n Please see the patch first, this warnings is avoided by "__maybe_unused" attributes. --- ��.n��������+%������w��{.n�����{��i��)��jg��������ݢj����G�������j:+v���w�m������w�������h�����٥