Hi Sricharan,
On 2/21/2011 2:46 PM, Shilimkar, Santosh wrote:
From: sricharan<r.sricharan@xxxxxx>
Add the error log registers, irqs of the l3 interconnect to the
hwmod data. The hwmod change is aligned with Benoit Cousson.
The change is almost aligned with Benoit Cousson :-)
The subject should be something like: omap4: hwmod data: Add address
space and irq in L3 hwmod
There is no errolog data in that patch. The changelog should be updated too.
Signed-off-by: Santosh Shilimkar<santosh.shilimkar@xxxxxx>
Signed-off-by: sricharan<r.sricharan@xxxxxx>
Tested-by: sricharan<r.sricharan@xxxxxx>
Cc: Benoit Cousson<b-cousson@xxxxxx>
---
arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 28 ++++++++++++++++++++++++++++
1 files changed, 28 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
index c2806bd..95001d4 100644
--- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
@@ -229,11 +229,37 @@ static struct omap_hwmod_ocp_if omap44xx_l4_cfg__l3_main_1 = {
.user = OCP_USER_MPU | OCP_USER_SDMA,
};
+/* L3 target configuration and error log registers */
+static struct omap_hwmod_irq_info omap44xx_l3_targ_irqs[] = {
+ { .irq = 9 + OMAP44XX_IRQ_GIC_START },
+ { .irq = 10 + OMAP44XX_IRQ_GIC_START },
+};
+
+static struct omap_hwmod_addr_space omap44xx_l3_targ_addrs[] = {
+ {
+ .pa_start = 0x44000000,
+ .pa_end = 0x44100000,
+ .flags = ADDR_TYPE_RT,
+ },
+ {
+ .pa_start = 0x44800000,
+ .pa_end = 0x44900000,
+ .flags = ADDR_TYPE_RT,
+ },
+ {
+ .pa_start = 0x45000000,
+ .pa_end = 0x45100000,
+ .flags = ADDR_TYPE_RT,
IIRC, we do have one address space per l3_main instance, and we do have
3 instances and thus 3 hwmods.
So you should potentially duplicate the code below at hwmod structure
level and not only at address space level.
In that case, you will re-group the 3 instances in only one omap_device.
For the IRQ, in theory we should duplicate the same entry 3 times just
in case someone want to use only the second instance. But, I'm still not
convinced this is the right thing to do... so let's keep it only for
l3_main_1 for the moment.
Regards,
Benoit
--
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
