| Message ID | 1385514296-26702-2-git-send-email-soren.brinkmann@xilinx.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 11/27/2013 02:04 AM, Soren Brinkmann wrote: > The bindings for the TTC changed in commit 'arm: zynq: Use standard > timer binding' (e932900a3279b5dbb6d8f43c7b369003620e137c). That change > removed possible subnodes from this driver rendering the 'clock-ranges' > property invalid for this node. > > Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com> > Acked-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com> > Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> > --- > arch/arm/boot/dts/zynq-7000.dtsi | 2 -- > 1 file changed, 2 deletions(-) I think this is completely unrelated to cpufreq and should just go directly via arm-soc tree. If yes, I will add 1/9 and 2/9 to zynq/dt branch. Thanks, Michal
Hi Michal, On Thu, Dec 12, 2013 at 09:53:13AM +0100, Michal Simek wrote: > On 11/27/2013 02:04 AM, Soren Brinkmann wrote: > > The bindings for the TTC changed in commit 'arm: zynq: Use standard > > timer binding' (e932900a3279b5dbb6d8f43c7b369003620e137c). That change > > removed possible subnodes from this driver rendering the 'clock-ranges' > > property invalid for this node. > > > > Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com> > > Acked-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com> > > Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> > > --- > > arch/arm/boot/dts/zynq-7000.dtsi | 2 -- > > 1 file changed, 2 deletions(-) > > I think this is completely unrelated to cpufreq and should just go > directly via arm-soc tree. > If yes, I will add 1/9 and 2/9 to zynq/dt branch. I agree. This is just clean up that I found during this work. The second patch could go in as well I think. cpufreq will need the CPU nodes later, but I think there is no real reason to not add them now. Sören
On 12/12/2013 06:01 PM, Sören Brinkmann wrote: > Hi Michal, > > On Thu, Dec 12, 2013 at 09:53:13AM +0100, Michal Simek wrote: >> On 11/27/2013 02:04 AM, Soren Brinkmann wrote: >>> The bindings for the TTC changed in commit 'arm: zynq: Use standard >>> timer binding' (e932900a3279b5dbb6d8f43c7b369003620e137c). That change >>> removed possible subnodes from this driver rendering the 'clock-ranges' >>> property invalid for this node. >>> >>> Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com> >>> Acked-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com> >>> Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> >>> --- >>> arch/arm/boot/dts/zynq-7000.dtsi | 2 -- >>> 1 file changed, 2 deletions(-) >> >> I think this is completely unrelated to cpufreq and should just go >> directly via arm-soc tree. >> If yes, I will add 1/9 and 2/9 to zynq/dt branch. > > I agree. This is just clean up that I found during this work. The second > patch could go in as well I think. cpufreq will need the CPU nodes > later, but I think there is no real reason to not add them now. ok then. 1/9 and 2/9 applied to zynq/dt branch. Thanks, Michal
diff --git a/arch/arm/boot/dts/zynq-7000.dtsi b/arch/arm/boot/dts/zynq-7000.dtsi index e7f73b2e4550..605a2d0a39fc 100644 --- a/arch/arm/boot/dts/zynq-7000.dtsi +++ b/arch/arm/boot/dts/zynq-7000.dtsi @@ -106,7 +106,6 @@ compatible = "cdns,ttc"; clocks = <&clkc 6>; reg = <0xF8001000 0x1000>; - clock-ranges; }; ttc1: ttc1@f8002000 { @@ -115,7 +114,6 @@ compatible = "cdns,ttc"; clocks = <&clkc 6>; reg = <0xF8002000 0x1000>; - clock-ranges; }; scutimer: scutimer@f8f00600 { interrupt-parent = <&intc>;