Re: [PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory sections
"Aguirre, Sergio" writes: [...] >> >> Also, work with Benoit to make sure at the scripts that autogenerate >> this data are updated to include these two regions. > > Ok. > > As a side note, I might need more addresses for the rest of the ISP > components later on. I'll enable more subsystems once the CSI2-A only > initial version is in an acceptable state. That's fine, just work with Benoit to be sure that the autogen scripts are updated for all the subsystems. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory sections
Hi Kevin, Thanks for the review. On Fri, Dec 2, 2011 at 4:49 PM, Kevin Hilman wrote: > +Benoit, > > "Aguirre, Sergio" writes: > >> Hi Vaibhav, >> >> Thanks for the comments. >> >> On Thu, Dec 1, 2011 at 12:34 AM, Hiremath, Vaibhav wrote: >>> >>>> -Original Message- >>>> From: linux-media-ow...@vger.kernel.org [mailto:linux-media- >>>> ow...@vger.kernel.org] On Behalf Of Aguirre, Sergio >>>> Sent: Thursday, December 01, 2011 5:45 AM >>>> To: linux-media@vger.kernel.org >>>> Cc: linux-o...@vger.kernel.org; laurent.pinch...@ideasonboard.com; >>>> sakari.ai...@iki.fi; Aguirre, Sergio >>>> Subject: [PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory >>>> sections >>>> >>>> Signed-off-by: Sergio Aguirre >>>> --- >>>> arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 16 +--- >>>> 1 files changed, 13 insertions(+), 3 deletions(-) >>>> >>>> diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c b/arch/arm/mach- >>>> omap2/omap_hwmod_44xx_data.c >>>> index 7695e5d..1b59e2f 100644 >>>> --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>>> +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>>> @@ -2623,8 +2623,18 @@ static struct omap_hwmod_ocp_if >>>> *omap44xx_iss_masters[] = { >>>> >>>> static struct omap_hwmod_addr_space omap44xx_iss_addrs[] = { >>>> { >>>> - .pa_start = 0x5200, >>>> - .pa_end = 0x52ff, >>>> + .pa_start = OMAP44XX_ISS_TOP_BASE, >>>> + .pa_end = OMAP44XX_ISS_TOP_END, >>>> + .flags = ADDR_TYPE_RT >>>> + }, >>>> + { >>>> + .pa_start = OMAP44XX_ISS_CSI2_A_REGS1_BASE, >>>> + .pa_end = OMAP44XX_ISS_CSI2_A_REGS1_END, >>>> + .flags = ADDR_TYPE_RT >>>> + }, >>>> + { >>>> + .pa_start = OMAP44XX_ISS_CAMERARX_CORE1_BASE, >>>> + .pa_end = OMAP44XX_ISS_CAMERARX_CORE1_END, >>>> .flags = ADDR_TYPE_RT >>>> }, >>> This patch will result in build failure, because, the above base addresses >>> are getting defined in the next patch >>> >>> [PATCH v2 05/11] OMAP4: Add base addresses for ISS >> >> Agreed. Will revisit "git-bisectability" of the patch series. Will fix. > > To fix this, just drop the #defines from the header, and use raw > addresses directly. Ok, i'll drop patch #5 in this series. > > Also, work with Benoit to make sure at the scripts that autogenerate > this data are updated to include these two regions. Ok. As a side note, I might need more addresses for the rest of the ISP components later on. I'll enable more subsystems once the CSI2-A only initial version is in an acceptable state. Regards, Sergio > > Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory sections
+Benoit, "Aguirre, Sergio" writes: > Hi Vaibhav, > > Thanks for the comments. > > On Thu, Dec 1, 2011 at 12:34 AM, Hiremath, Vaibhav wrote: >> >>> -Original Message- >>> From: linux-media-ow...@vger.kernel.org [mailto:linux-media- >>> ow...@vger.kernel.org] On Behalf Of Aguirre, Sergio >>> Sent: Thursday, December 01, 2011 5:45 AM >>> To: linux-media@vger.kernel.org >>> Cc: linux-o...@vger.kernel.org; laurent.pinch...@ideasonboard.com; >>> sakari.ai...@iki.fi; Aguirre, Sergio >>> Subject: [PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory >>> sections >>> >>> Signed-off-by: Sergio Aguirre >>> --- >>> arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 16 +--- >>> 1 files changed, 13 insertions(+), 3 deletions(-) >>> >>> diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c b/arch/arm/mach- >>> omap2/omap_hwmod_44xx_data.c >>> index 7695e5d..1b59e2f 100644 >>> --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>> +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >>> @@ -2623,8 +2623,18 @@ static struct omap_hwmod_ocp_if >>> *omap44xx_iss_masters[] = { >>> >>> static struct omap_hwmod_addr_space omap44xx_iss_addrs[] = { >>> { >>> - .pa_start = 0x5200, >>> - .pa_end = 0x52ff, >>> + .pa_start = OMAP44XX_ISS_TOP_BASE, >>> + .pa_end = OMAP44XX_ISS_TOP_END, >>> + .flags = ADDR_TYPE_RT >>> + }, >>> + { >>> + .pa_start = OMAP44XX_ISS_CSI2_A_REGS1_BASE, >>> + .pa_end = OMAP44XX_ISS_CSI2_A_REGS1_END, >>> + .flags = ADDR_TYPE_RT >>> + }, >>> + { >>> + .pa_start = OMAP44XX_ISS_CAMERARX_CORE1_BASE, >>> + .pa_end = OMAP44XX_ISS_CAMERARX_CORE1_END, >>> .flags = ADDR_TYPE_RT >>> }, >> This patch will result in build failure, because, the above base addresses >> are getting defined in the next patch >> >> [PATCH v2 05/11] OMAP4: Add base addresses for ISS > > Agreed. Will revisit "git-bisectability" of the patch series. Will fix. To fix this, just drop the #defines from the header, and use raw addresses directly. Also, work with Benoit to make sure at the scripts that autogenerate this data are updated to include these two regions. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory sections
Hi Vaibhav, Thanks for the comments. On Thu, Dec 1, 2011 at 12:34 AM, Hiremath, Vaibhav wrote: > >> -Original Message- >> From: linux-media-ow...@vger.kernel.org [mailto:linux-media- >> ow...@vger.kernel.org] On Behalf Of Aguirre, Sergio >> Sent: Thursday, December 01, 2011 5:45 AM >> To: linux-media@vger.kernel.org >> Cc: linux-o...@vger.kernel.org; laurent.pinch...@ideasonboard.com; >> sakari.ai...@iki.fi; Aguirre, Sergio >> Subject: [PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory >> sections >> >> Signed-off-by: Sergio Aguirre >> --- >> arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 16 +--- >> 1 files changed, 13 insertions(+), 3 deletions(-) >> >> diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c b/arch/arm/mach- >> omap2/omap_hwmod_44xx_data.c >> index 7695e5d..1b59e2f 100644 >> --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >> +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >> @@ -2623,8 +2623,18 @@ static struct omap_hwmod_ocp_if >> *omap44xx_iss_masters[] = { >> >> static struct omap_hwmod_addr_space omap44xx_iss_addrs[] = { >> { >> - .pa_start = 0x5200, >> - .pa_end = 0x52ff, >> + .pa_start = OMAP44XX_ISS_TOP_BASE, >> + .pa_end = OMAP44XX_ISS_TOP_END, >> + .flags = ADDR_TYPE_RT >> + }, >> + { >> + .pa_start = OMAP44XX_ISS_CSI2_A_REGS1_BASE, >> + .pa_end = OMAP44XX_ISS_CSI2_A_REGS1_END, >> + .flags = ADDR_TYPE_RT >> + }, >> + { >> + .pa_start = OMAP44XX_ISS_CAMERARX_CORE1_BASE, >> + .pa_end = OMAP44XX_ISS_CAMERARX_CORE1_END, >> .flags = ADDR_TYPE_RT >> }, > This patch will result in build failure, because, the above base addresses > are getting defined in the next patch > > [PATCH v2 05/11] OMAP4: Add base addresses for ISS Agreed. Will revisit "git-bisectability" of the patch series. Will fix. Regards, Sergio > > Thanks, > Vaibhav > >> { } >> @@ -5350,7 +5360,7 @@ static __initdata struct omap_hwmod >> *omap44xx_hwmods[] = { >> &omap44xx_ipu_c1_hwmod, >> >> /* iss class */ >> -/* &omap44xx_iss_hwmod, */ >> + &omap44xx_iss_hwmod, >> >> /* iva class */ >> &omap44xx_iva_hwmod, >> -- >> 1.7.7.4 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-media" in >> the body of a message to majord...@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory sections
> -Original Message- > From: linux-media-ow...@vger.kernel.org [mailto:linux-media- > ow...@vger.kernel.org] On Behalf Of Aguirre, Sergio > Sent: Thursday, December 01, 2011 5:45 AM > To: linux-media@vger.kernel.org > Cc: linux-o...@vger.kernel.org; laurent.pinch...@ideasonboard.com; > sakari.ai...@iki.fi; Aguirre, Sergio > Subject: [PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory > sections > > Signed-off-by: Sergio Aguirre > --- > arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 16 +--- > 1 files changed, 13 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c b/arch/arm/mach- > omap2/omap_hwmod_44xx_data.c > index 7695e5d..1b59e2f 100644 > --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c > +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c > @@ -2623,8 +2623,18 @@ static struct omap_hwmod_ocp_if > *omap44xx_iss_masters[] = { > > static struct omap_hwmod_addr_space omap44xx_iss_addrs[] = { > { > - .pa_start = 0x5200, > - .pa_end = 0x52ff, > + .pa_start = OMAP44XX_ISS_TOP_BASE, > + .pa_end = OMAP44XX_ISS_TOP_END, > + .flags = ADDR_TYPE_RT > + }, > + { > + .pa_start = OMAP44XX_ISS_CSI2_A_REGS1_BASE, > + .pa_end = OMAP44XX_ISS_CSI2_A_REGS1_END, > + .flags = ADDR_TYPE_RT > + }, > + { > + .pa_start = OMAP44XX_ISS_CAMERARX_CORE1_BASE, > + .pa_end = OMAP44XX_ISS_CAMERARX_CORE1_END, > .flags = ADDR_TYPE_RT > }, This patch will result in build failure, because, the above base addresses are getting defined in the next patch [PATCH v2 05/11] OMAP4: Add base addresses for ISS Thanks, Vaibhav > { } > @@ -5350,7 +5360,7 @@ static __initdata struct omap_hwmod > *omap44xx_hwmods[] = { > &omap44xx_ipu_c1_hwmod, > > /* iss class */ > -/* &omap44xx_iss_hwmod, */ > + &omap44xx_iss_hwmod, > > /* iva class */ > &omap44xx_iva_hwmod, > -- > 1.7.7.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v2 04/11] OMAP4: hwmod: Include CSI2A and CSIPHY1 memory sections
Signed-off-by: Sergio Aguirre --- arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 16 +--- 1 files changed, 13 insertions(+), 3 deletions(-) diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c index 7695e5d..1b59e2f 100644 --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c @@ -2623,8 +2623,18 @@ static struct omap_hwmod_ocp_if *omap44xx_iss_masters[] = { static struct omap_hwmod_addr_space omap44xx_iss_addrs[] = { { - .pa_start = 0x5200, - .pa_end = 0x52ff, + .pa_start = OMAP44XX_ISS_TOP_BASE, + .pa_end = OMAP44XX_ISS_TOP_END, + .flags = ADDR_TYPE_RT + }, + { + .pa_start = OMAP44XX_ISS_CSI2_A_REGS1_BASE, + .pa_end = OMAP44XX_ISS_CSI2_A_REGS1_END, + .flags = ADDR_TYPE_RT + }, + { + .pa_start = OMAP44XX_ISS_CAMERARX_CORE1_BASE, + .pa_end = OMAP44XX_ISS_CAMERARX_CORE1_END, .flags = ADDR_TYPE_RT }, { } @@ -5350,7 +5360,7 @@ static __initdata struct omap_hwmod *omap44xx_hwmods[] = { &omap44xx_ipu_c1_hwmod, /* iss class */ -/* &omap44xx_iss_hwmod, */ + &omap44xx_iss_hwmod, /* iva class */ &omap44xx_iva_hwmod, -- 1.7.7.4 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html