Charles Manning <manningc2@xxxxxxxxxxxxx> writes:
> arch/arm/mach-omap2/pm24xx.c doesn't compile without CONFIG_SUSPEND
>
> diff --git a/arch/arm/mach-omap2/pm24xx.c b/arch/arm/mach-omap2/pm24xx.c
> index aaeea49..ae6abda 100644
> --- a/arch/arm/mach-omap2/pm24xx.c
> +++ b/arch/arm/mach-omap2/pm24xx.c
> @@ -53,8 +53,8 @@
> #include <plat/powerdomain.h>
> #include <plat/clockdomain.h>
>
> -#ifdef CONFIG_SUSPEND
> static suspend_state_t suspend_state = PM_SUSPEND_ON;
> +#ifdef CONFIG_SUSPEND
> static inline bool is_suspending(void)
> {
> return (suspend_state != PM_SUSPEND_ON);
> @@ -314,6 +314,7 @@ static int omap2_pm_prepare(void)
>
> static int omap2_pm_suspend(void)
> {
> +#ifdef CONFIG_SUSPEND
> u32 wken_wkup, mir1;
>
> wken_wkup = prm_read_mod_reg(WKUP_MOD, PM_WKEN);
> @@ -329,7 +330,7 @@ static int omap2_pm_suspend(void)
>
> omap_writel(mir1, 0x480fe0a4);
> prm_write_mod_reg(wken_wkup, WKUP_MOD, PM_WKEN);
> -
> +#endif
> return 0;
> }
Indeed a bug, but not quite the right fix.
Like we do on OMAP3, all of the _begin, _enter, _suspend, and _end
fucntions can actually be left out when !CONFIG_SUSPEND.
Will send patch shortly.
Thanks for reporting this and proposing a fix.
Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
