Messages in this thread | | | Subject | Re: [PATCH -next] platform: surface: fix non-PM_SLEEP build warnings | From | Randy Dunlap <> | Date | Sun, 13 Dec 2020 16:53:47 -0800 |
| |
On 12/12/20 11:07 AM, Andy Shevchenko wrote: > On Sat, Dec 12, 2020 at 7:05 PM Randy Dunlap <rdunlap@infradead.org> wrote: >> On 12/12/20 5:24 AM, Andy Shevchenko wrote: >>> On Fri, Dec 11, 2020 at 9:20 PM Randy Dunlap <rdunlap@infradead.org> 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/ > Thanks for the link. I'll send a v2.
Could we add that to the Linux BKP (Best Known Practices) document?
-- ~Randy
| |