[PATCH] ARM: common: edma: edma_pm_resume may be unused

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Fri Nov 21 05:33:50 PST 2014


Hello Arnd,

On Fri, Nov 21, 2014 at 11:52:51AM +0100, Arnd Bergmann wrote:
> The recently introduced resume hook in the edma driver
> is not referenced when CONFIG_PM_SLEEP is not set, which
> results in a compile warning in keystone_defconfig builds.
> 
> This adds an appropriate #ifdef.
> 
> Cc: Nishanth Menon <nm at ti.com>
> Cc: Daniel Mack <zonque at gmail.com>
> Cc: Joel Fernandes <joelf at ti.com>
> Cc: Sekhar Nori <nsekhar at ti.com>
> Fixes: a2b1175131: ("ARM: common: edma: add suspend resume hook")
> Signed-off-by: Arnd Bergmann <arnd at arndb.de>
> ---
> I'm applying this as the obvious fixup on top of the next/drivers
> branch.
> 
> diff --git a/arch/arm/common/edma.c b/arch/arm/common/edma.c
> index 24a2857f2285..5662a872689b 100644
> --- a/arch/arm/common/edma.c
> +++ b/arch/arm/common/edma.c
> @@ -1810,6 +1810,7 @@ static int edma_probe(struct platform_device *pdev)
>  	return 0;
>  }
>  
> +#ifdef CONFIG_PM_SLEEP
>  static int edma_pm_resume(struct device *dev)
What about

-static int edma_pm_resume(struct device *dev)
+static int __maybe_unused edma_pm_resume(struct device *dev)

instead?

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |



More information about the linux-arm-kernel mailing list