Skip to contentSkip to conversation

ARM: OMAP2+: hwmod data: Fix wrong lostcontext_mask for OMAP4 l4_abe

PATCH New Benoit Cousson 2 participants 2 messages Oct 16, 2012

Subsystems (4)
lore
Share & tools
lore
#0 lore raw

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
index 652d028..c820de2 100644
@@ -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
maintainer
#1 lore raw

PATCH 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_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.

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.

Thanks applying into omap-for-v3.7-rc1/fixes-part2.

Regards,

Tony