* [PATCH] ARM: OMAP4: hwmod data: Add McASP data port address space @ 2012-09-27 16:33 Ricardo Neri 2012-10-09 14:15 ` Ricardo Neri 0 siblings, 1 reply; 4+ messages in thread From: Ricardo Neri @ 2012-09-27 16:33 UTC (permalink / raw) To: b-cousson; +Cc: tony, jon-hunter, linux-omap, Ricardo Neri McASP has a configuration port and a data port. This patch adds the address space entry for the data port as described in the OMAP4 TRM. Also, add names to the address spaces. Signed-off-by: Ricardo Neri <ricardo.neri@ti.com> --- arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 6 ++++++ 1 files changed, 6 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 f31f3bc..cb5b463 100644 --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c @@ -4951,10 +4951,16 @@ static struct omap_hwmod_ocp_if omap44xx_l4_abe__mcasp = { static struct omap_hwmod_addr_space omap44xx_mcasp_dma_addrs[] = { { + .name = "cfg", .pa_start = 0x49028000, .pa_end = 0x490283ff, .flags = ADDR_TYPE_RT }, + { + .name = "dat", + .pa_start = 0x4902A000, + .pa_end = 0x4902Afff, + }, { } }; -- 1.7.5.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: OMAP4: hwmod data: Add McASP data port address space 2012-09-27 16:33 [PATCH] ARM: OMAP4: hwmod data: Add McASP data port address space Ricardo Neri @ 2012-10-09 14:15 ` Ricardo Neri 2012-10-11 10:29 ` Benoit Cousson 0 siblings, 1 reply; 4+ messages in thread From: Ricardo Neri @ 2012-10-09 14:15 UTC (permalink / raw) To: b-cousson; +Cc: Ricardo Neri, tony, jon-hunter, linux-omap Hi Benoit, Have you had a chance to look at this patch? Maybe you want me to submit it differently or to a different list? Thanks! Ricardo On 09/27/2012 11:33 AM, Ricardo Neri wrote: > McASP has a configuration port and a data port. This patch adds the address > space entry for the data port as described in the OMAP4 TRM. > > Also, add names to the address spaces. > > Signed-off-by: Ricardo Neri <ricardo.neri@ti.com> > --- > arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 6 ++++++ > 1 files changed, 6 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 f31f3bc..cb5b463 100644 > --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c > +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c > @@ -4951,10 +4951,16 @@ static struct omap_hwmod_ocp_if omap44xx_l4_abe__mcasp = { > > static struct omap_hwmod_addr_space omap44xx_mcasp_dma_addrs[] = { > { > + .name = "cfg", > .pa_start = 0x49028000, > .pa_end = 0x490283ff, > .flags = ADDR_TYPE_RT > }, > + { > + .name = "dat", > + .pa_start = 0x4902A000, > + .pa_end = 0x4902Afff, > + }, > { } > }; > > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: OMAP4: hwmod data: Add McASP data port address space 2012-10-09 14:15 ` Ricardo Neri @ 2012-10-11 10:29 ` Benoit Cousson 2012-10-11 17:30 ` Ricardo Neri 0 siblings, 1 reply; 4+ messages in thread From: Benoit Cousson @ 2012-10-11 10:29 UTC (permalink / raw) To: Ricardo Neri; +Cc: tony, jon-hunter, linux-omap Hi Ricardo, On 10/09/2012 04:15 PM, Ricardo Neri wrote: > Hi Benoit, > > Have you had a chance to look at this patch? Maybe you want me to submit > it differently or to a different list? Sorry, I missed it. It looks fine and aligned with what I have done for OMAP5. I just have few minors comments. > > Thanks! > > Ricardo > > On 09/27/2012 11:33 AM, Ricardo Neri wrote: >> McASP has a configuration port and a data port. This patch adds the >> address >> space entry for the data port as described in the OMAP4 TRM. >> >> Also, add names to the address spaces. >> >> Signed-off-by: Ricardo Neri <ricardo.neri@ti.com> >> --- >> arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 6 ++++++ >> 1 files changed, 6 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 f31f3bc..cb5b463 100644 >> --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >> +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >> @@ -4951,10 +4951,16 @@ static struct omap_hwmod_ocp_if >> omap44xx_l4_abe__mcasp = { >> >> static struct omap_hwmod_addr_space omap44xx_mcasp_dma_addrs[] = { >> { >> + .name = "cfg", >> .pa_start = 0x49028000, >> .pa_end = 0x490283ff, >> .flags = ADDR_TYPE_RT >> }, >> + { >> + .name = "dat", >> + .pa_start = 0x4902A000, You should use lower case for hex number. >> + .pa_end = 0x4902Afff, In the data I have, the end is 0x4012a3ff. Do we need to extend it to fff? Regards, Benoit ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: OMAP4: hwmod data: Add McASP data port address space 2012-10-11 10:29 ` Benoit Cousson @ 2012-10-11 17:30 ` Ricardo Neri 0 siblings, 0 replies; 4+ messages in thread From: Ricardo Neri @ 2012-10-11 17:30 UTC (permalink / raw) To: Benoit Cousson; +Cc: tony, jon-hunter, linux-omap Hi Benoit, Thanks for reviewing! On 10/11/2012 05:29 AM, Benoit Cousson wrote: > Hi Ricardo, > > On 10/09/2012 04:15 PM, Ricardo Neri wrote: >> Hi Benoit, >> >> Have you had a chance to look at this patch? Maybe you want me to submit >> it differently or to a different list? > > Sorry, I missed it. It looks fine and aligned with what I have done for > OMAP5. I just have few minors comments. > >> >> Thanks! >> >> Ricardo >> >> On 09/27/2012 11:33 AM, Ricardo Neri wrote: >>> McASP has a configuration port and a data port. This patch adds the >>> address >>> space entry for the data port as described in the OMAP4 TRM. >>> >>> Also, add names to the address spaces. >>> >>> Signed-off-by: Ricardo Neri <ricardo.neri@ti.com> >>> --- >>> arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 6 ++++++ >>> 1 files changed, 6 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 f31f3bc..cb5b463 100644 >>> --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>> +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>> @@ -4951,10 +4951,16 @@ static struct omap_hwmod_ocp_if >>> omap44xx_l4_abe__mcasp = { >>> >>> static struct omap_hwmod_addr_space omap44xx_mcasp_dma_addrs[] = { >>> { >>> + .name = "cfg", >>> .pa_start = 0x49028000, >>> .pa_end = 0x490283ff, >>> .flags = ADDR_TYPE_RT >>> }, >>> + { >>> + .name = "dat", >>> + .pa_start = 0x4902A000, > > You should use lower case for hex number. Ok, I'll do it. > >>> + .pa_end = 0x4902Afff, > > In the data I have, the end is 0x4012a3ff. > Do we need to extend it to fff? Not sure. I took this number from the Table 2-7 of OMAP4 TRM. In general, I see that the .pa_end values for several modules in omap_hwmod_44xx_data.c do not match the Chapter 2 of OMAP4 TRM. Examples of this are omap44xx_aess_addrs, omap44xx_mcbsp1/2/3_addrs, omap44xx_dmic_addrs, omap44xx_dss_addrs, omap44xx_dss_dsi1_addrs, omap44xx_dss_dsi2_addrs. Is there any reason for this? Maybe your data is more accurate. BR, Ricardo > > Regards, > Benoit > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-10-11 17:32 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-09-27 16:33 [PATCH] ARM: OMAP4: hwmod data: Add McASP data port address space Ricardo Neri 2012-10-09 14:15 ` Ricardo Neri 2012-10-11 10:29 ` Benoit Cousson 2012-10-11 17:30 ` Ricardo Neri
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.