On Fri, 2018-06-01 at 13:17 +0300, Andy Shevchenko wrote: > On Fri, Jun 1, 2018 at 6:04 AM, <honghui.zhang@xxxxxxxxxxxx> wrote: > > From: Honghui Zhang <honghui.zhang@xxxxxxxxxxxx> > > > +#ifdef CONFIG_PM_SLEEP > > +static int mtk_pcie_suspend_noirq(struct device *dev) > > __maybe_unused > Hi, Andy, thanks for your review. Bjorn had point this out that at: https://www.spinics.net/lists/arm-kernel/msg656774.html So __maybe_unused is not really needed. thanks