Re: [PATCHv2 14/19] ARM: OMAP4: wakeupgen: enable clocks for save_secure_all

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

+Benoit

Tero Kristo <t-kristo@xxxxxx> writes:

> save_secure_all needs l3_main_3_ick and l4_secure_clkdm enabled,
> otherwise the secure ROM code will crash.
>
> Signed-off-by: Tero Kristo <t-kristo@xxxxxx>

I think I mentioned this already (I'm already lost in what I've said for
thisseries), but I don't think the secure RAM stuff belongs in the
wakeupgen driver.

Also, this patch suggests that the OCM RAM block needs a proper hwmod
instead of manually fiddling with the l3_main_3 hwmod and the l4_secure clkdm.

Kevin

> ---
>  arch/arm/mach-omap2/omap-wakeupgen.c |   20 ++++++++++++++++++++
>  1 files changed, 20 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/omap-wakeupgen.c b/arch/arm/mach-omap2/omap-wakeupgen.c
> index b2165e4..c7c4db4 100644
> --- a/arch/arm/mach-omap2/omap-wakeupgen.c
> +++ b/arch/arm/mach-omap2/omap-wakeupgen.c
> @@ -29,10 +29,12 @@
>  
>  #include <mach/omap-wakeupgen.h>
>  #include <mach/omap-secure.h>
> +#include <plat/omap_hwmod.h>
>  
>  #include "omap4-sar-layout.h"
>  #include "common.h"
>  #include "pm.h"
> +#include "clockdomain.h"
>  
>  #define NR_REG_BANKS		4
>  #define MAX_IRQS		128
> @@ -49,6 +51,8 @@ static DEFINE_SPINLOCK(wakeupgen_lock);
>  static unsigned int irq_target_cpu[NR_IRQS];
>  
>  static struct powerdomain *mpuss_pd;
> +static struct clockdomain *l4_secure_clkdm;
> +static struct omap_hwmod *l3_main_3_oh;
>  
>  /*
>   * Static helper functions.
> @@ -300,10 +304,18 @@ static void save_secure_ram(void)
>  static void save_secure_all(void)
>  {
>  	u32 ret;
> +
> +	omap_hwmod_enable(l3_main_3_oh);
> +	clkdm_wakeup(l4_secure_clkdm);
> +
>  	ret = omap_secure_dispatcher(OMAP4_HAL_SAVEALL_INDEX,
>  				FLAG_START_CRITICAL,
>  				1, omap_secure_ram_mempool_base(),
>  				0, 0, 0);
> +
> +	clkdm_allow_idle(l4_secure_clkdm);
> +	omap_hwmod_idle(l3_main_3_oh);
> +
>  	if (ret != API_HAL_RET_VALUE_OK)
>  		pr_err("Secure all context save failed\n");
>  }
> @@ -441,6 +453,14 @@ int __init omap_wakeupgen_init(void)
>  			sar_writel(GIC_ISR_NON_SECURE, ICDISR_SPI_OFFSET, i);
>  		iounmap(sar_base);
>  		sar_base = NULL;
> +	} else {
> +		l3_main_3_oh = omap_hwmod_lookup("l3_main_3");
> +		if (!l3_main_3_oh)
> +			pr_err("%s: failed to get l3_main_3_oh\n", __func__);
> +
> +		l4_secure_clkdm = clkdm_lookup("l4_secure_clkdm");
> +		if (!l4_secure_clkdm)
> +			pr_err("%s: failed to get l4_secure_clkdm\n", __func__);
>  	}
>  
>  	irq_hotplug_init();

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel


[Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [PDAs]     [Linux]     [Linux MIPS]     [Yosemite Campsites]     [Photos]

Add to Google Follow linuxarm on Twitter