RE: [PATCH v6 3/7] omap: gpmc: enable irq mode in gpmc

2010-12-22 Thread Ghorai, Sukumar


> -Original Message-
> From: Tony Lindgren [mailto:t...@atomide.com]
> Sent: Saturday, December 18, 2010 5:40 AM
> To: Ghorai, Sukumar
> Cc: linux-omap@vger.kernel.org; linux-...@lists.infradead.org; linux-arm-
> ker...@lists.infradead.org
> Subject: Re: [PATCH v6 3/7] omap: gpmc: enable irq mode in gpmc
> 
> * Sukumar Ghorai  [101126 07:25]:
> > add support the irq mode in GPMC.
> > gpmc_init() function move after omap_init_irq() as it has dependecy on
> irq.
> >
> > Signed-off-by: Sukumar Ghorai 
> > ---
> >  arch/arm/mach-omap2/board-2430sdp.c|1 +
> >  arch/arm/mach-omap2/board-3430sdp.c|1 +
> >  arch/arm/mach-omap2/board-3630sdp.c|1 +
> >  arch/arm/mach-omap2/board-4430sdp.c|2 +
> >  arch/arm/mach-omap2/board-am3517evm.c  |2 +
> >  arch/arm/mach-omap2/board-apollon.c|1 +
> >  arch/arm/mach-omap2/board-cm-t35.c |1 +
> >  arch/arm/mach-omap2/board-devkit8000.c |1 +
> >  arch/arm/mach-omap2/board-generic.c|2 +
> >  arch/arm/mach-omap2/board-h4.c |1 +
> >  arch/arm/mach-omap2/board-igep0020.c   |1 +
> >  arch/arm/mach-omap2/board-ldp.c|1 +
> >  arch/arm/mach-omap2/board-n8x0.c   |2 +
> >  arch/arm/mach-omap2/board-omap3beagle.c|1 +
> >  arch/arm/mach-omap2/board-omap3evm.c   |2 +
> >  arch/arm/mach-omap2/board-omap3pandora.c   |2 +
> >  arch/arm/mach-omap2/board-omap3stalker.c   |1 +
> >  arch/arm/mach-omap2/board-omap3touchbook.c |1 +
> >  arch/arm/mach-omap2/board-omap4panda.c |2 +
> >  arch/arm/mach-omap2/board-overo.c  |1 +
> >  arch/arm/mach-omap2/board-rx51.c   |1 +
> >  arch/arm/mach-omap2/board-zoom2.c  |2 +
> >  arch/arm/mach-omap2/board-zoom3.c  |2 +
> >  arch/arm/mach-omap2/gpmc.c |   39
> ++-
> >  arch/arm/mach-omap2/io.c   |2 -
> >  arch/arm/plat-omap/include/plat/gpmc.h |4 +++
> >  arch/arm/plat-omap/include/plat/irqs.h |9 +-
> >  27 files changed, 81 insertions(+), 5 deletions(-)
> >
> > diff --git a/arch/arm/mach-omap2/board-2430sdp.c b/arch/arm/mach-
> omap2/board-2430sdp.c
> > index b527f8d..11c89dc 100644
> > --- a/arch/arm/mach-omap2/board-2430sdp.c
> > +++ b/arch/arm/mach-omap2/board-2430sdp.c
> > @@ -145,6 +145,7 @@ static void __init omap_2430sdp_init_irq(void)
> > omap_board_config_size = ARRAY_SIZE(sdp2430_config);
> > omap2_init_common_hw(NULL, NULL);
> > omap_init_irq();
> > +   gpmc_init();
> > omap_gpio_init();
> >  }
> >
> > diff --git a/arch/arm/mach-omap2/board-3430sdp.c b/arch/arm/mach-
> omap2/board-3430sdp.c
> > index 4e3742c..a040165 100644
> > --- a/arch/arm/mach-omap2/board-3430sdp.c
> > +++ b/arch/arm/mach-omap2/board-3430sdp.c
> > @@ -328,6 +328,7 @@ static void __init omap_3430sdp_init_irq(void)
> > omap3_pm_init_cpuidle(omap3_cpuidle_params_table);
> > omap2_init_common_hw(hyb18m512160af6_sdrc_params, NULL);
> > omap_init_irq();
> > +   gpmc_init();
> > omap_gpio_init();
> >  }
> ...
> 
> The gpmc init should be done with subsys_initcall instead.

[Ghorai] You suggested the same in previous v5; and I mentioned 
the issues [1] that - nand_init()called before subsys_initcall(gpmc_init);
Let me know if I am missing anything again.
 
[1] https://patchwork.kernel.org/patch/212452/

--
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 v6 3/7] omap: gpmc: enable irq mode in gpmc

2010-12-17 Thread Tony Lindgren
* Sukumar Ghorai  [101126 07:25]:
> add support the irq mode in GPMC.
> gpmc_init() function move after omap_init_irq() as it has dependecy on irq.
> 
> Signed-off-by: Sukumar Ghorai 
> ---
>  arch/arm/mach-omap2/board-2430sdp.c|1 +
>  arch/arm/mach-omap2/board-3430sdp.c|1 +
>  arch/arm/mach-omap2/board-3630sdp.c|1 +
>  arch/arm/mach-omap2/board-4430sdp.c|2 +
>  arch/arm/mach-omap2/board-am3517evm.c  |2 +
>  arch/arm/mach-omap2/board-apollon.c|1 +
>  arch/arm/mach-omap2/board-cm-t35.c |1 +
>  arch/arm/mach-omap2/board-devkit8000.c |1 +
>  arch/arm/mach-omap2/board-generic.c|2 +
>  arch/arm/mach-omap2/board-h4.c |1 +
>  arch/arm/mach-omap2/board-igep0020.c   |1 +
>  arch/arm/mach-omap2/board-ldp.c|1 +
>  arch/arm/mach-omap2/board-n8x0.c   |2 +
>  arch/arm/mach-omap2/board-omap3beagle.c|1 +
>  arch/arm/mach-omap2/board-omap3evm.c   |2 +
>  arch/arm/mach-omap2/board-omap3pandora.c   |2 +
>  arch/arm/mach-omap2/board-omap3stalker.c   |1 +
>  arch/arm/mach-omap2/board-omap3touchbook.c |1 +
>  arch/arm/mach-omap2/board-omap4panda.c |2 +
>  arch/arm/mach-omap2/board-overo.c  |1 +
>  arch/arm/mach-omap2/board-rx51.c   |1 +
>  arch/arm/mach-omap2/board-zoom2.c  |2 +
>  arch/arm/mach-omap2/board-zoom3.c  |2 +
>  arch/arm/mach-omap2/gpmc.c |   39 ++-
>  arch/arm/mach-omap2/io.c   |2 -
>  arch/arm/plat-omap/include/plat/gpmc.h |4 +++
>  arch/arm/plat-omap/include/plat/irqs.h |9 +-
>  27 files changed, 81 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/board-2430sdp.c 
> b/arch/arm/mach-omap2/board-2430sdp.c
> index b527f8d..11c89dc 100644
> --- a/arch/arm/mach-omap2/board-2430sdp.c
> +++ b/arch/arm/mach-omap2/board-2430sdp.c
> @@ -145,6 +145,7 @@ static void __init omap_2430sdp_init_irq(void)
>   omap_board_config_size = ARRAY_SIZE(sdp2430_config);
>   omap2_init_common_hw(NULL, NULL);
>   omap_init_irq();
> + gpmc_init();
>   omap_gpio_init();
>  }
>  
> diff --git a/arch/arm/mach-omap2/board-3430sdp.c 
> b/arch/arm/mach-omap2/board-3430sdp.c
> index 4e3742c..a040165 100644
> --- a/arch/arm/mach-omap2/board-3430sdp.c
> +++ b/arch/arm/mach-omap2/board-3430sdp.c
> @@ -328,6 +328,7 @@ static void __init omap_3430sdp_init_irq(void)
>   omap3_pm_init_cpuidle(omap3_cpuidle_params_table);
>   omap2_init_common_hw(hyb18m512160af6_sdrc_params, NULL);
>   omap_init_irq();
> + gpmc_init();
>   omap_gpio_init();
>  }
...

The gpmc init should be done with subsys_initcall instead.

Tony
--
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


[PATCH v6 3/7] omap: gpmc: enable irq mode in gpmc

2010-11-26 Thread Sukumar Ghorai
add support the irq mode in GPMC.
gpmc_init() function move after omap_init_irq() as it has dependecy on irq.

Signed-off-by: Sukumar Ghorai 
---
 arch/arm/mach-omap2/board-2430sdp.c|1 +
 arch/arm/mach-omap2/board-3430sdp.c|1 +
 arch/arm/mach-omap2/board-3630sdp.c|1 +
 arch/arm/mach-omap2/board-4430sdp.c|2 +
 arch/arm/mach-omap2/board-am3517evm.c  |2 +
 arch/arm/mach-omap2/board-apollon.c|1 +
 arch/arm/mach-omap2/board-cm-t35.c |1 +
 arch/arm/mach-omap2/board-devkit8000.c |1 +
 arch/arm/mach-omap2/board-generic.c|2 +
 arch/arm/mach-omap2/board-h4.c |1 +
 arch/arm/mach-omap2/board-igep0020.c   |1 +
 arch/arm/mach-omap2/board-ldp.c|1 +
 arch/arm/mach-omap2/board-n8x0.c   |2 +
 arch/arm/mach-omap2/board-omap3beagle.c|1 +
 arch/arm/mach-omap2/board-omap3evm.c   |2 +
 arch/arm/mach-omap2/board-omap3pandora.c   |2 +
 arch/arm/mach-omap2/board-omap3stalker.c   |1 +
 arch/arm/mach-omap2/board-omap3touchbook.c |1 +
 arch/arm/mach-omap2/board-omap4panda.c |2 +
 arch/arm/mach-omap2/board-overo.c  |1 +
 arch/arm/mach-omap2/board-rx51.c   |1 +
 arch/arm/mach-omap2/board-zoom2.c  |2 +
 arch/arm/mach-omap2/board-zoom3.c  |2 +
 arch/arm/mach-omap2/gpmc.c |   39 ++-
 arch/arm/mach-omap2/io.c   |2 -
 arch/arm/plat-omap/include/plat/gpmc.h |4 +++
 arch/arm/plat-omap/include/plat/irqs.h |9 +-
 27 files changed, 81 insertions(+), 5 deletions(-)

diff --git a/arch/arm/mach-omap2/board-2430sdp.c 
b/arch/arm/mach-omap2/board-2430sdp.c
index b527f8d..11c89dc 100644
--- a/arch/arm/mach-omap2/board-2430sdp.c
+++ b/arch/arm/mach-omap2/board-2430sdp.c
@@ -145,6 +145,7 @@ static void __init omap_2430sdp_init_irq(void)
omap_board_config_size = ARRAY_SIZE(sdp2430_config);
omap2_init_common_hw(NULL, NULL);
omap_init_irq();
+   gpmc_init();
omap_gpio_init();
 }
 
diff --git a/arch/arm/mach-omap2/board-3430sdp.c 
b/arch/arm/mach-omap2/board-3430sdp.c
index 4e3742c..a040165 100644
--- a/arch/arm/mach-omap2/board-3430sdp.c
+++ b/arch/arm/mach-omap2/board-3430sdp.c
@@ -328,6 +328,7 @@ static void __init omap_3430sdp_init_irq(void)
omap3_pm_init_cpuidle(omap3_cpuidle_params_table);
omap2_init_common_hw(hyb18m512160af6_sdrc_params, NULL);
omap_init_irq();
+   gpmc_init();
omap_gpio_init();
 }
 
diff --git a/arch/arm/mach-omap2/board-3630sdp.c 
b/arch/arm/mach-omap2/board-3630sdp.c
index bbcf580..17983ed 100644
--- a/arch/arm/mach-omap2/board-3630sdp.c
+++ b/arch/arm/mach-omap2/board-3630sdp.c
@@ -76,6 +76,7 @@ static void __init omap_sdp_init_irq(void)
omap2_init_common_hw(h8mbx00u0mer0em_sdrc_params,
h8mbx00u0mer0em_sdrc_params);
omap_init_irq();
+   gpmc_init();
omap_gpio_init();
 }
 
diff --git a/arch/arm/mach-omap2/board-4430sdp.c 
b/arch/arm/mach-omap2/board-4430sdp.c
index df5a425..8d15604 100644
--- a/arch/arm/mach-omap2/board-4430sdp.c
+++ b/arch/arm/mach-omap2/board-4430sdp.c
@@ -34,6 +34,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include "hsmmc.h"
 #include "timer-gp.h"
@@ -222,6 +223,7 @@ static void __init omap_4430sdp_init_irq(void)
omap2_gp_clockevent_set_gptimer(1);
 #endif
gic_init_irq();
+   gpmc_init();
omap_gpio_init();
 }
 
diff --git a/arch/arm/mach-omap2/board-am3517evm.c 
b/arch/arm/mach-omap2/board-am3517evm.c
index 0739950..460e3d1 100644
--- a/arch/arm/mach-omap2/board-am3517evm.c
+++ b/arch/arm/mach-omap2/board-am3517evm.c
@@ -35,6 +35,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include "mux.h"
 #include "control.h"
@@ -392,6 +393,7 @@ static void __init am3517_evm_init_irq(void)
 
omap2_init_common_hw(NULL, NULL);
omap_init_irq();
+   gpmc_init();
omap_gpio_init();
 }
 
diff --git a/arch/arm/mach-omap2/board-apollon.c 
b/arch/arm/mach-omap2/board-apollon.c
index 2c6db1a..8264e7a 100644
--- a/arch/arm/mach-omap2/board-apollon.c
+++ b/arch/arm/mach-omap2/board-apollon.c
@@ -280,6 +280,7 @@ static void __init omap_apollon_init_irq(void)
omap_board_config_size = ARRAY_SIZE(apollon_config);
omap2_init_common_hw(NULL, NULL);
omap_init_irq();
+   gpmc_init();
omap_gpio_init();
apollon_init_smc91x();
 }
diff --git a/arch/arm/mach-omap2/board-cm-t35.c 
b/arch/arm/mach-omap2/board-cm-t35.c
index 63f764e..7c9a834 100644
--- a/arch/arm/mach-omap2/board-cm-t35.c
+++ b/arch/arm/mach-omap2/board-cm-t35.c
@@ -686,6 +686,7 @@ static void __init cm_t35_init_irq(void)
omap2_init_common_hw(mt46h32m32lf6_sdrc_params,
 mt46h32m32lf6_sdrc_params);
omap_init_irq();
+   gpmc_init();
omap_gpio_init();
 }
 
diff --git a/arch/arm/mach-omap2/b