ARM: OMAP2+: hwmod data: Fix wrong lostcontext_mask for OMAP4 l4_abe
Subsystems (4)
PATCH 1/1 ARM: OMAP2+: hwmod data: Fix wrong lostcontext_mask for OMAP4 l4_abe
The following commit added the support for the lostcontext_mask along with the usage of the flag for l4_abe.
commit ce80979aedfce937926a8dd40a1f92fd4bc2fd53
Author: Tero Kristo <[email protected]>
Date: Sun Sep 23 17:28:19 2012 -0600
ARM: OMAP4: hwmod data: add support for lostcontext_mask
Unfortunately, the l4_abe does not contain this memory and thus cannot contain that flag.
Remove the flag from the l4_abe hwmod.
Change as well the mask for omap44xx_aess_hwmod to avoid missing the status in case LOSTMEM_AESSMEM is set. The AESS logic does not support retention and thus only the memory state is useful. It is even mandatory to avoid a crash in the case of the AESS resume.
TBD: Ideally each hwmod should contain an extra entry for memory context attached to it. The AESS does contain only one, but some IP like the IVAHD does have three memories with dedicated status. The granularity is never used, but at least one global information for all the memory banks is useful.
--- arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-)
arch/arm/mach-omap2/omap_hwmod_44xx_data.c +2 -4
@@ -203,9 +203,7 @@ static struct omap_hwmod omap44xx_l4_abe_hwmod = { .prcm = { .omap4 = { .clkctrl_offs = OMAP4_CM1_ABE_L4ABE_CLKCTRL_OFFSET, - .context_offs = OMAP4_RM_ABE_AESS_CONTEXT_OFFSET, - .lostcontext_mask = OMAP4430_LOSTMEM_AESSMEM_MASK, - .flags = HWMOD_OMAP4_NO_CONTEXT_LOSS_BIT, + .flags = HWMOD_OMAP4_NO_CONTEXT_LOSS_BIT, }, }, };
@@ -351,7 +349,7 @@ static struct omap_hwmod omap44xx_aess_hwmod = { .omap4 = { .clkctrl_offs = OMAP4_CM1_ABE_AESS_CLKCTRL_OFFSET, .context_offs = OMAP4_RM_ABE_AESS_CONTEXT_OFFSET, - .lostcontext_mask = OMAP4430_LOSTCONTEXT_DFF_MASK, + .lostcontext_mask = OMAP4430_LOSTMEM_AESSMEM_MASK, .modulemode = MODULEMODE_SWCTRL, }, },
Signature · 1 line
1.7.0.4
Tony Lindgren
maintainerPATCH 1/1 ARM: OMAP2+: hwmod data: Fix wrong lostcontext_mask for OMAP4 l4_abe
Quoted 20 lines · Benoit Cousson
The following commit added the support for the
lostcontext_maskalong with the usage of the flag forl4_abe.commit
ce80979aedfce937926a8dd40a1f92fd4bc2fd53
Author: Tero Kristo <[email protected]>
Date: Sun Sep 23 17:28:19 2012 -0600ARM: OMAP4: hwmod data: add support for
lostcontext_maskUnfortunately, the
l4_abedoes not contain this memory and thus cannot contain that flag.Remove the flag from the
l4_abehwmod.Change as well the mask for
omap44xx_aess_hwmodto avoid missing the status in caseLOSTMEM_AESSMEMis set. The AESS logic does not support retention and thus only the memory state is useful. It is even mandatory to avoid a crash in the case of the AESS resume.
Quoted 8 lines
TBD: Ideally each hwmod should contain an extra entry for memory context attached to it. The AESS does contain only one, but some IP like the IVAHD does have three memories with dedicated status. The granularity is never used, but at least one global information for all the memory banks is useful.
Signed-off-by Benoit Cousson <[email protected]>Cc Tero Kristo <[email protected]>
Thanks applying into omap-for-v3.7-rc1/fixes-part2.
Regards,
Tony