Re: [RFC 2/5] ARM: dts: am33xx: Set parent clock for timer through DT

2014-03-14 Thread Tero Kristo 

On 03/13/2014 10:35 PM, Joel Fernandes wrote:

Signed-off-by: Joel Fernandes jo...@ti.com
---
  arch/arm/boot/dts/am33xx.dtsi |2 ++
  1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
index 4e6c959..51b7008 100644
--- a/arch/arm/boot/dts/am33xx.dtsi
+++ b/arch/arm/boot/dts/am33xx.dtsi
@@ -352,6 +352,7 @@
interrupts = 67;
ti,hwmods = timer1;
ti,timer-alwon;
+   ti,default-parent = sys_clkin_ck;
};

timer2: timer@4804 {
@@ -360,6 +361,7 @@
reg = 0x4804 0x400;
interrupts = 68;
ti,hwmods = timer2;
+   ti,default-parent = sys_clkin_ck;
};

timer3: timer@48042000 {



I don't think this works, the ti,default-parent property is only 
supported for clock nodes (at least I didn't quickly find anything from 
your clocksource driver that would handle this.) You should do something 
like this:


timer1_fck {
ti,default-parent = sys_clkin_ck;
};

timer2_fck {
ti,default-parent = sys_clkin_ck;
};

-Tero

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

[RFC 2/5] ARM: dts: am33xx: Set parent clock for timer through DT

2014-03-13 Thread Joel Fernandes 
Signed-off-by: Joel Fernandes jo...@ti.com
---
 arch/arm/boot/dts/am33xx.dtsi |2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
index 4e6c959..51b7008 100644
--- a/arch/arm/boot/dts/am33xx.dtsi
+++ b/arch/arm/boot/dts/am33xx.dtsi
@@ -352,6 +352,7 @@
interrupts = 67;
ti,hwmods = timer1;
ti,timer-alwon;
+   ti,default-parent = sys_clkin_ck;
};
 
timer2: timer@4804 {
@@ -360,6 +361,7 @@
reg = 0x4804 0x400;
interrupts = 68;
ti,hwmods = timer2;
+   ti,default-parent = sys_clkin_ck;
};
 
timer3: timer@48042000 {
-- 
1.7.9.5

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