Re: [PATCH] ARM: OMAP4: hwmod data: gpmc main clk

2012-10-16 Thread Tony Lindgren
* Benoit Cousson b-cous...@ti.com [121015 03:02]:
 Hi Afzal,
 
 On 10/15/2012 07:47 AM, Afzal Mohammed wrote:
  gpmc has been converted to a driver. It requests clock with
  con-id fck, if not available, probe fails. Clock for gpmc
  device with con-id fck is created (aliased) along with
  hwmod device, and for that to happen, main_clk needs to be
  specified in hwmod entry, add it for OMAP4.
 
 Well, in this case, this is not the preferred way. The hwmod data are 
 supposed to reflect the real HW capabilities.
 We'd better add the alias in the already existing dummy clock node.
 
 Here is the patch to do that.
 
 Thanks,
 Benoit
 
 ---
 From: Benoit Cousson b-cous...@ti.com
 Date: Mon, 15 Oct 2012 11:41:42 +0200
 Subject: [PATCH] ARM: OMAP2+: clock data: Add dev-id for the omap-gpmc dummy 
 fck
 
 The GPMC code has been converted to a driver by the following commit:
 
  commit da496873970c57c4b31e186d967933da0ffa0d7c
  Author: Afzal Mohammed af...@ti.com
  Date:   Sun Sep 23 17:28:25 2012 -0600
 
ARM: OMAP2+: gpmc: minimal driver support
 
 It now requests a clock with con-id fck otherwise the probe will fails.
 
 [0.342010] omap-gpmc omap-gpmc: error: clk_get
 [0.346771] omap-gpmc: probe of omap-gpmc failed with error -2
 
 Add the omap-gmpc dev-id and fck con-id to the already existing
 gmpc-fck dummy clock.
 
 Reported-by: Russell King rmk+ker...@arm.linux.org.uk
 Signed-off-by: Benoit Cousson b-cous...@ti.com
 Cc: Afzal Mohammed af...@ti.com

Thanks applying into omap-for-v3.7-rc1/fixes-part2.

Regards,

Tony

 ---
  arch/arm/mach-omap2/clock44xx_data.c |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
 
 diff --git a/arch/arm/mach-omap2/clock44xx_data.c 
 b/arch/arm/mach-omap2/clock44xx_data.c
 index d661d13..6efc30c 100644
 --- a/arch/arm/mach-omap2/clock44xx_data.c
 +++ b/arch/arm/mach-omap2/clock44xx_data.c
 @@ -3294,7 +3294,7 @@ static struct omap_clk omap44xx_clks[] = {
   CLK(NULL,   auxclk5_src_ck,   auxclk5_src_ck,
 CK_443X),
   CLK(NULL,   auxclk5_ck,   auxclk5_ck,
 CK_443X),
   CLK(NULL,   auxclkreq5_ck,auxclkreq5_ck, 
 CK_443X),
 - CLK(NULL,   gpmc_ck,  dummy_ck,  
 CK_443X),
 + CLK(omap-gpmc,fck,  dummy_ck,  
 CK_443X),
   CLK(omap_i2c.1,   ick,  dummy_ck,  
 CK_443X),
   CLK(omap_i2c.2,   ick,  dummy_ck,  
 CK_443X),
   CLK(omap_i2c.3,   ick,  dummy_ck,  
 CK_443X),
 -- 
 1.7.0.4
 
--
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] ARM: OMAP4: hwmod data: gpmc main clk

2012-10-15 Thread Benoit Cousson
Hi Afzal,

On 10/15/2012 07:47 AM, Afzal Mohammed wrote:
 gpmc has been converted to a driver. It requests clock with
 con-id fck, if not available, probe fails. Clock for gpmc
 device with con-id fck is created (aliased) along with
 hwmod device, and for that to happen, main_clk needs to be
 specified in hwmod entry, add it for OMAP4.

Well, in this case, this is not the preferred way. The hwmod data are supposed 
to reflect the real HW capabilities.
We'd better add the alias in the already existing dummy clock node.

Here is the patch to do that.

Thanks,
Benoit

---
From: Benoit Cousson b-cous...@ti.com
Date: Mon, 15 Oct 2012 11:41:42 +0200
Subject: [PATCH] ARM: OMAP2+: clock data: Add dev-id for the omap-gpmc dummy fck

The GPMC code has been converted to a driver by the following commit:

 commit da496873970c57c4b31e186d967933da0ffa0d7c
 Author: Afzal Mohammed af...@ti.com
 Date:   Sun Sep 23 17:28:25 2012 -0600

   ARM: OMAP2+: gpmc: minimal driver support

It now requests a clock with con-id fck otherwise the probe will fails.

[0.342010] omap-gpmc omap-gpmc: error: clk_get
[0.346771] omap-gpmc: probe of omap-gpmc failed with error -2

Add the omap-gmpc dev-id and fck con-id to the already existing
gmpc-fck dummy clock.

Reported-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Benoit Cousson b-cous...@ti.com
Cc: Afzal Mohammed af...@ti.com
---
 arch/arm/mach-omap2/clock44xx_data.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/clock44xx_data.c 
b/arch/arm/mach-omap2/clock44xx_data.c
index d661d13..6efc30c 100644
--- a/arch/arm/mach-omap2/clock44xx_data.c
+++ b/arch/arm/mach-omap2/clock44xx_data.c
@@ -3294,7 +3294,7 @@ static struct omap_clk omap44xx_clks[] = {
CLK(NULL,   auxclk5_src_ck,   auxclk5_src_ck,
CK_443X),
CLK(NULL,   auxclk5_ck,   auxclk5_ck,
CK_443X),
CLK(NULL,   auxclkreq5_ck,auxclkreq5_ck, 
CK_443X),
-   CLK(NULL,   gpmc_ck,  dummy_ck,  
CK_443X),
+   CLK(omap-gpmc,fck,  dummy_ck,  
CK_443X),
CLK(omap_i2c.1,   ick,  dummy_ck,  
CK_443X),
CLK(omap_i2c.2,   ick,  dummy_ck,  
CK_443X),
CLK(omap_i2c.3,   ick,  dummy_ck,  
CK_443X),
-- 
1.7.0.4

--
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] ARM: OMAP4: hwmod data: gpmc main clk

2012-10-14 Thread Afzal Mohammed
gpmc has been converted to a driver. It requests clock with
con-id fck, if not available, probe fails. Clock for gpmc
device with con-id fck is created (aliased) along with
hwmod device, and for that to happen, main_clk needs to be
specified in hwmod entry, add it for OMAP4.

Note that the corresponding clock is dummy.

Reported-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Afzal Mohammed af...@ti.com
---
 arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c 
b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
index 652d028..ce04002 100644
--- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
@@ -1363,6 +1363,7 @@ static struct omap_hwmod omap44xx_gpmc_hwmod = {
.flags  = HWMOD_INIT_NO_IDLE | HWMOD_INIT_NO_RESET,
.mpu_irqs   = omap44xx_gpmc_irqs,
.sdma_reqs  = omap44xx_gpmc_sdma_reqs,
+   .main_clk   = gpmc_ck,
.prcm = {
.omap4 = {
.clkctrl_offs = OMAP4_CM_L3_2_GPMC_CLKCTRL_OFFSET,
-- 
1.7.12

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