* [PATCH] arm: dts: dra7xx-clocks: Fix the l3 and l4 clock rates
@ 2014-05-27 8:55 Rajendra Nayak
2014-07-03 8:34 ` Rajendra Nayak
0 siblings, 1 reply; 3+ messages in thread
From: Rajendra Nayak @ 2014-05-27 8:55 UTC (permalink / raw)
To: linux-arm-kernel
Without the patch:
/debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck # cat clk_rate
532000000
/debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div # cat clk_rate
532000000
/debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div/l4_root_clk_div # cat clk_rate
532000000
With the patch:
/debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck # cat clk_rate
532000000
/debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div # cat clk_rate
266000000
/debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div/l4_root_clk_div # cat clk_rate
133000000
The l3 clock derived from core DPLL is actually a divider clock,
with the default divider set to 2. l4 then derived from l3 is a fixed factor
clock, but the fixed divider is 2 and not 1. Which means the l3 clock is
half of core DPLLs h12x2 and l4 is half of l3 (as seen with this patch)
Signed-off-by: Rajendra Nayak <rnayak@ti.com>
---
arch/arm/boot/dts/dra7xx-clocks.dtsi | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/dra7xx-clocks.dtsi b/arch/arm/boot/dts/dra7xx-clocks.dtsi
index cfb8fc7..a14c99b 100644
--- a/arch/arm/boot/dts/dra7xx-clocks.dtsi
+++ b/arch/arm/boot/dts/dra7xx-clocks.dtsi
@@ -673,10 +673,12 @@
l3_iclk_div: l3_iclk_div {
#clock-cells = <0>;
- compatible = "fixed-factor-clock";
+ compatible = "ti,divider-clock";
+ ti,max-div = <2>;
+ ti,bit-shift = <4>;
+ reg = <0x0100>;
clocks = <&dpll_core_h12x2_ck>;
- clock-mult = <1>;
- clock-div = <1>;
+ ti,index-power-of-two;
};
l4_root_clk_div: l4_root_clk_div {
@@ -684,7 +686,7 @@
compatible = "fixed-factor-clock";
clocks = <&l3_iclk_div>;
clock-mult = <1>;
- clock-div = <1>;
+ clock-div = <2>;
};
video1_clk2_div: video1_clk2_div {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] arm: dts: dra7xx-clocks: Fix the l3 and l4 clock rates
2014-05-27 8:55 [PATCH] arm: dts: dra7xx-clocks: Fix the l3 and l4 clock rates Rajendra Nayak
@ 2014-07-03 8:34 ` Rajendra Nayak
2014-07-03 8:59 ` Tero Kristo
0 siblings, 1 reply; 3+ messages in thread
From: Rajendra Nayak @ 2014-07-03 8:34 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 27 May 2014 02:25 PM, Rajendra Nayak wrote:
> Without the patch:
> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck # cat clk_rate
> 532000000
> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div # cat clk_rate
> 532000000
> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div/l4_root_clk_div # cat clk_rate
> 532000000
>
> With the patch:
> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck # cat clk_rate
> 532000000
> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div # cat clk_rate
> 266000000
> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div/l4_root_clk_div # cat clk_rate
> 133000000
>
> The l3 clock derived from core DPLL is actually a divider clock,
> with the default divider set to 2. l4 then derived from l3 is a fixed factor
> clock, but the fixed divider is 2 and not 1. Which means the l3 clock is
> half of core DPLLs h12x2 and l4 is half of l3 (as seen with this patch)
Tero, this seems like is yet to be picked up. You see any issues with this that needs to be
addressed?
regards,
Rajendra
>
> Signed-off-by: Rajendra Nayak <rnayak@ti.com>
> ---
> arch/arm/boot/dts/dra7xx-clocks.dtsi | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/dra7xx-clocks.dtsi b/arch/arm/boot/dts/dra7xx-clocks.dtsi
> index cfb8fc7..a14c99b 100644
> --- a/arch/arm/boot/dts/dra7xx-clocks.dtsi
> +++ b/arch/arm/boot/dts/dra7xx-clocks.dtsi
> @@ -673,10 +673,12 @@
>
> l3_iclk_div: l3_iclk_div {
> #clock-cells = <0>;
> - compatible = "fixed-factor-clock";
> + compatible = "ti,divider-clock";
> + ti,max-div = <2>;
> + ti,bit-shift = <4>;
> + reg = <0x0100>;
> clocks = <&dpll_core_h12x2_ck>;
> - clock-mult = <1>;
> - clock-div = <1>;
> + ti,index-power-of-two;
> };
>
> l4_root_clk_div: l4_root_clk_div {
> @@ -684,7 +686,7 @@
> compatible = "fixed-factor-clock";
> clocks = <&l3_iclk_div>;
> clock-mult = <1>;
> - clock-div = <1>;
> + clock-div = <2>;
> };
>
> video1_clk2_div: video1_clk2_div {
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] arm: dts: dra7xx-clocks: Fix the l3 and l4 clock rates
2014-07-03 8:34 ` Rajendra Nayak
@ 2014-07-03 8:59 ` Tero Kristo
0 siblings, 0 replies; 3+ messages in thread
From: Tero Kristo @ 2014-07-03 8:59 UTC (permalink / raw)
To: linux-arm-kernel
On 07/03/2014 11:34 AM, Rajendra Nayak wrote:
> On Tuesday 27 May 2014 02:25 PM, Rajendra Nayak wrote:
>> Without the patch:
>> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck # cat clk_rate
>> 532000000
>> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div # cat clk_rate
>> 532000000
>> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div/l4_root_clk_div # cat clk_rate
>> 532000000
>>
>> With the patch:
>> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck # cat clk_rate
>> 532000000
>> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div # cat clk_rate
>> 266000000
>> /debug/.../dpll_core_x2_ck/dpll_core_h12x2_ck/l3_iclk_div/l4_root_clk_div # cat clk_rate
>> 133000000
>>
>> The l3 clock derived from core DPLL is actually a divider clock,
>> with the default divider set to 2. l4 then derived from l3 is a fixed factor
>> clock, but the fixed divider is 2 and not 1. Which means the l3 clock is
>> half of core DPLLs h12x2 and l4 is half of l3 (as seen with this patch)
>
> Tero, this seems like is yet to be picked up. You see any issues with this that needs to be
> addressed?
Yea this actually looks good to me. Queued for 3.16-rc dt-clk fixes.
-Tero
>
> regards,
> Rajendra
>
>>
>> Signed-off-by: Rajendra Nayak <rnayak@ti.com>
>> ---
>> arch/arm/boot/dts/dra7xx-clocks.dtsi | 10 ++++++----
>> 1 file changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/dra7xx-clocks.dtsi b/arch/arm/boot/dts/dra7xx-clocks.dtsi
>> index cfb8fc7..a14c99b 100644
>> --- a/arch/arm/boot/dts/dra7xx-clocks.dtsi
>> +++ b/arch/arm/boot/dts/dra7xx-clocks.dtsi
>> @@ -673,10 +673,12 @@
>>
>> l3_iclk_div: l3_iclk_div {
>> #clock-cells = <0>;
>> - compatible = "fixed-factor-clock";
>> + compatible = "ti,divider-clock";
>> + ti,max-div = <2>;
>> + ti,bit-shift = <4>;
>> + reg = <0x0100>;
>> clocks = <&dpll_core_h12x2_ck>;
>> - clock-mult = <1>;
>> - clock-div = <1>;
>> + ti,index-power-of-two;
>> };
>>
>> l4_root_clk_div: l4_root_clk_div {
>> @@ -684,7 +686,7 @@
>> compatible = "fixed-factor-clock";
>> clocks = <&l3_iclk_div>;
>> clock-mult = <1>;
>> - clock-div = <1>;
>> + clock-div = <2>;
>> };
>>
>> video1_clk2_div: video1_clk2_div {
>>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-07-03 8:59 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-27 8:55 [PATCH] arm: dts: dra7xx-clocks: Fix the l3 and l4 clock rates Rajendra Nayak
2014-07-03 8:34 ` Rajendra Nayak
2014-07-03 8:59 ` Tero Kristo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).