Re: [PATCH 1/3] arm: davinci: clock node support for ECAP & EHRPWM

2013-03-14 Thread Sekhar Nori

On 3/14/2013 4:07 PM, Philip Avinash wrote:
> Add clock node support for ECAP and EHRPWM modules.
> Also adds dummy clock for EHRWPM TBCLK to comply with pwm-tiehrpwm
> driver.

This is not right. So the version of IP used on AM335x uses a TBCLK and
that's absent on the version used on DA850? If yes, the driver should
never request clock on DA850.

Thanks,
Sekhar
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 1/3] arm: davinci: clock node support for ECAP & EHRPWM

2013-03-14 Thread Philip Avinash
Add clock node support for ECAP and EHRPWM modules.
Also adds dummy clock for EHRWPM TBCLK to comply with pwm-tiehrpwm
driver.

Signed-off-by: Philip Avinash 
---
:100644 100644 0c4a26d... 891d075... M  arch/arm/mach-davinci/da850.c
 arch/arm/mach-davinci/da850.c |   24 
 1 file changed, 24 insertions(+)

diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c
index 0c4a26d..891d075 100644
--- a/arch/arm/mach-davinci/da850.c
+++ b/arch/arm/mach-davinci/da850.c
@@ -383,6 +383,27 @@ static struct clk dsp_clk = {
.flags  = PSC_LRST | PSC_FORCE,
 };
 
+static struct clk ehrpwm_clk = {
+   .name   = "ehrpwm",
+   .parent = _sysclk2,
+   .lpsc   = DA8XX_LPSC1_PWM,
+   .gpsc   = 1,
+   .flags  = DA850_CLK_ASYNC3,
+};
+
+static struct clk ehrpwm_tbclk = {
+   .name   = "ehrpwm_tbclk",
+   .parent = NULL,
+};
+
+static struct clk ecap_clk = {
+   .name   = "ecap",
+   .parent = _sysclk2,
+   .lpsc   = DA8XX_LPSC1_ECAP,
+   .gpsc   = 1,
+   .flags  = DA850_CLK_ASYNC3,
+};
+
 static struct clk_lookup da850_clks[] = {
CLK(NULL,   "ref",  _clk),
CLK(NULL,   "pll0", _clk),
@@ -430,6 +451,9 @@ static struct clk_lookup da850_clks[] = {
CLK("vpif", NULL,   _clk),
CLK("ahci", NULL,   _clk),
CLK("davinci-rproc.0",  NULL,   _clk),
+   CLK("ehrpwm",   "fck",  _clk),
+   CLK("ehrpwm",   "tbclk",_tbclk),
+   CLK("ecap", "fck",  _clk),
CLK(NULL,   NULL,   NULL),
 };
 
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 1/3] arm: davinci: clock node support for ECAP EHRPWM

2013-03-14 Thread Philip Avinash
Add clock node support for ECAP and EHRPWM modules.
Also adds dummy clock for EHRWPM TBCLK to comply with pwm-tiehrpwm
driver.

Signed-off-by: Philip Avinash avinashphi...@ti.com
---
:100644 100644 0c4a26d... 891d075... M  arch/arm/mach-davinci/da850.c
 arch/arm/mach-davinci/da850.c |   24 
 1 file changed, 24 insertions(+)

diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c
index 0c4a26d..891d075 100644
--- a/arch/arm/mach-davinci/da850.c
+++ b/arch/arm/mach-davinci/da850.c
@@ -383,6 +383,27 @@ static struct clk dsp_clk = {
.flags  = PSC_LRST | PSC_FORCE,
 };
 
+static struct clk ehrpwm_clk = {
+   .name   = ehrpwm,
+   .parent = pll0_sysclk2,
+   .lpsc   = DA8XX_LPSC1_PWM,
+   .gpsc   = 1,
+   .flags  = DA850_CLK_ASYNC3,
+};
+
+static struct clk ehrpwm_tbclk = {
+   .name   = ehrpwm_tbclk,
+   .parent = NULL,
+};
+
+static struct clk ecap_clk = {
+   .name   = ecap,
+   .parent = pll0_sysclk2,
+   .lpsc   = DA8XX_LPSC1_ECAP,
+   .gpsc   = 1,
+   .flags  = DA850_CLK_ASYNC3,
+};
+
 static struct clk_lookup da850_clks[] = {
CLK(NULL,   ref,  ref_clk),
CLK(NULL,   pll0, pll0_clk),
@@ -430,6 +451,9 @@ static struct clk_lookup da850_clks[] = {
CLK(vpif, NULL,   vpif_clk),
CLK(ahci, NULL,   sata_clk),
CLK(davinci-rproc.0,  NULL,   dsp_clk),
+   CLK(ehrpwm,   fck,  ehrpwm_clk),
+   CLK(ehrpwm,   tbclk,ehrpwm_tbclk),
+   CLK(ecap, fck,  ecap_clk),
CLK(NULL,   NULL,   NULL),
 };
 
-- 
1.7.9.5

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 1/3] arm: davinci: clock node support for ECAP EHRPWM

2013-03-14 Thread Sekhar Nori

On 3/14/2013 4:07 PM, Philip Avinash wrote:
 Add clock node support for ECAP and EHRPWM modules.
 Also adds dummy clock for EHRWPM TBCLK to comply with pwm-tiehrpwm
 driver.

This is not right. So the version of IP used on AM335x uses a TBCLK and
that's absent on the version used on DA850? If yes, the driver should
never request clock on DA850.

Thanks,
Sekhar
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/