Re: [PATCH -next] platform: surface: fix non-PM_SLEEP build warnings

From: Andy Shevchenko
Date: Sat Dec 12 2020 - 14:09:19 EST


On Sat, Dec 12, 2020 at 7:05 PM Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote:
> On 12/12/20 5:24 AM, Andy Shevchenko wrote:
> > On Fri, Dec 11, 2020 at 9:20 PM Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote:

...

> >> +#ifdef CONFIG_PM_SLEEP
> >> static int surface_gpe_suspend(struct device *dev)
> >
> > Perhaps __maybe_unused ?
>
> Yes, I am aware of that option.
> I don't know why it would be preferred though.

Here [1] is the rationale behind annotation vs. ifdeffery.

[1]: https://lore.kernel.org/patchwork/patch/732981/

--
With Best Regards,
Andy Shevchenko