[PATCH 5/5] omap2+: pm_bus: make functions used as pointers as static
Kevin Hilman
khilman at ti.com
Tue Jan 4 19:40:09 EST 2011
Nishanth Menon <nm at ti.com> writes:
> omap_pm_runtime_suspend and omap_pm_runtime_resume are used
> as function pointers and does not really need to be exposed
> to the world.
>
> Fixes sparse warnings:
> arch/arm/mach-omap2/pm_bus.c:23:5: warning: symbol 'omap_pm_runtime_suspend' was not declared. Should it be static?
> arch/arm/mach-omap2/pm_bus.c:40:5: warning: symbol 'omap_pm_runtime_resume' was not declared. Should it be static?
>
> Signed-off-by: Nishanth Menon <nm at ti.com>
Thanks, will queue this one in my pm-fixes branch for 2.6.38-rc.
Kevin
> ---
> arch/arm/mach-omap2/pm_bus.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/pm_bus.c b/arch/arm/mach-omap2/pm_bus.c
> index 784989f..5acd2ab 100644
> --- a/arch/arm/mach-omap2/pm_bus.c
> +++ b/arch/arm/mach-omap2/pm_bus.c
> @@ -20,7 +20,7 @@
> #include <plat/omap-pm.h>
>
> #ifdef CONFIG_PM_RUNTIME
> -int omap_pm_runtime_suspend(struct device *dev)
> +static int omap_pm_runtime_suspend(struct device *dev)
> {
> struct platform_device *pdev = to_platform_device(dev);
> int r, ret = 0;
> @@ -37,7 +37,7 @@ int omap_pm_runtime_suspend(struct device *dev)
> return ret;
> };
>
> -int omap_pm_runtime_resume(struct device *dev)
> +static int omap_pm_runtime_resume(struct device *dev)
> {
> struct platform_device *pdev = to_platform_device(dev);
> int r;
More information about the linux-arm-kernel
mailing list