RE: [PATCH] OMAP4: HWMOD: Do omap_hwmod_late_init for OMAP4

2010-08-09 Thread Varadarajan, Charulatha
 

> -Original Message-
> From: Cousson, Benoit 
> Sent: Friday, August 06, 2010 6:06 PM
> To: Varadarajan, Charulatha
> Cc: linux-omap@vger.kernel.org; p...@pwsan.com; 
> khil...@deeprootsystems.com; Nayak, Rajendra; Basak, Partha
> Subject: Re: [PATCH] OMAP4: HWMOD: Do omap_hwmod_late_init for OMAP4
> 
> Hi Charu,
> 
> On 8/6/2010 2:31 PM, Varadarajan, Charulatha wrote:
> > This patch includes "cpu_is" check for omap44xx cpu inorder to do
> > omap_hwmod_late_init() without which hwmods initialization does not
> > happen for omap4.
> >
> > Signed-off-by: Charulatha V
> > Signed-off-by: Basak, Partha
> > ---
> >   arch/arm/mach-omap2/io.c |2 +-
> >   1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c
> > index b89e678..9b15f46 100644
> > --- a/arch/arm/mach-omap2/io.c
> > +++ b/arch/arm/mach-omap2/io.c
> > @@ -345,7 +345,7 @@ void __init omap2_init_common_hw(struct 
> omap_sdrc_params *sdrc_cs0,
> >   #ifndef CONFIG_PM_RUNTIME
> > skip_setup_idle = 1;
> >   #endif
> > -   if (cpu_is_omap24xx() || cpu_is_omap34xx())   /* FIXME: OMAP4 */
> > +   if (cpu_is_omap24xx() || cpu_is_omap34xx() || cpu_is_omap44xx())
> > omap_hwmod_late_init(skip_setup_idle);
> >
> > if (cpu_is_omap24xx() || cpu_is_omap34xx()) {
> 
> This is already done in this patch:
> [PATCH v3 1/7] OMAP4: hwmod: Add initial data for OMAP4430 ES1 & ES2
> https://patchwork.kernel.org/patch/117347/

Okay. I found omap_hwmod_late_init() call missing for OMAP4 in
origin/pm-wip/hwmods-omap4 and sent this patch without noticing your
above mentioned patch. Please ignore this patch.

> 
> Benoit
> --
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] OMAP4: HWMOD: Do omap_hwmod_late_init for OMAP4

2010-08-06 Thread Cousson, Benoit

Hi Charu,

On 8/6/2010 2:31 PM, Varadarajan, Charulatha wrote:

This patch includes "cpu_is" check for omap44xx cpu inorder to do
omap_hwmod_late_init() without which hwmods initialization does not
happen for omap4.

Signed-off-by: Charulatha V
Signed-off-by: Basak, Partha
---
  arch/arm/mach-omap2/io.c |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c
index b89e678..9b15f46 100644
--- a/arch/arm/mach-omap2/io.c
+++ b/arch/arm/mach-omap2/io.c
@@ -345,7 +345,7 @@ void __init omap2_init_common_hw(struct omap_sdrc_params 
*sdrc_cs0,
  #ifndef CONFIG_PM_RUNTIME
skip_setup_idle = 1;
  #endif
-   if (cpu_is_omap24xx() || cpu_is_omap34xx())   /* FIXME: OMAP4 */
+   if (cpu_is_omap24xx() || cpu_is_omap34xx() || cpu_is_omap44xx())
omap_hwmod_late_init(skip_setup_idle);

if (cpu_is_omap24xx() || cpu_is_omap34xx()) {


This is already done in this patch:
[PATCH v3 1/7] OMAP4: hwmod: Add initial data for OMAP4430 ES1 & ES2
https://patchwork.kernel.org/patch/117347/

Benoit
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH] OMAP4: HWMOD: Do omap_hwmod_late_init for OMAP4

2010-08-06 Thread Shilimkar, Santosh


> -Original Message-
> From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-
> ow...@vger.kernel.org] On Behalf Of Varadarajan, Charulatha
> Sent: Friday, August 06, 2010 6:01 PM
> To: linux-omap@vger.kernel.org
> Cc: p...@pwsan.com; khil...@deeprootsystems.com; Cousson, Benoit; Nayak,
> Rajendra; Varadarajan, Charulatha; Basak, Partha
> Subject: [PATCH] OMAP4: HWMOD: Do omap_hwmod_late_init for OMAP4
> 
> This patch includes "cpu_is" check for omap44xx cpu inorder to do
> omap_hwmod_late_init() without which hwmods initialization does not
> happen for omap4.
> 
> Signed-off-by: Charulatha V 
> Signed-off-by: Basak, Partha 
> ---
>  arch/arm/mach-omap2/io.c |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c
> index b89e678..9b15f46 100644
> --- a/arch/arm/mach-omap2/io.c
> +++ b/arch/arm/mach-omap2/io.c
> @@ -345,7 +345,7 @@ void __init omap2_init_common_hw(struct
> omap_sdrc_params *sdrc_cs0,
>  #ifndef CONFIG_PM_RUNTIME
>   skip_setup_idle = 1;
>  #endif
> - if (cpu_is_omap24xx() || cpu_is_omap34xx())   /* FIXME: OMAP4 */
> + if (cpu_is_omap24xx() || cpu_is_omap34xx() || cpu_is_omap44xx())
Rather use now 
if (cpu_class_is_omap2())

>   omap_hwmod_late_init(skip_setup_idle);
> 
>   if (cpu_is_omap24xx() || cpu_is_omap34xx()) {
> --
> 1.6.3.3
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" 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-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html