2014-02-26 07:15:30

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: manual merge of the clk tree with the keystone tree

Hi Mike,

Today's linux-next merge of the clk tree got a conflict in
arch/arm/boot/dts/keystone-clocks.dtsi between commit 0cfc9ccec2a8 ("ARM:
dts: keystone: preparatory patch to support K2L and K2E SOCs") from the
keystone tree and commit 565bbdcd3b91 ("ARM: keystone: dts: fix clkvcp3
control register address") from the clk tree.

I fixed it up (by adding the following merge fix patch) and can carry the
fix as necessary (no action is required).

From: Stephen Rothwell <[email protected]>
Date: Wed, 26 Feb 2014 18:12:55 +1100
Subject: [PATCH] ARM: keystone: dts: fix for code movement

Signed-off-by: Stephen Rothwell <[email protected]>
---
arch/arm/boot/dts/k2hk-clocks.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/k2hk-clocks.dtsi b/arch/arm/boot/dts/k2hk-clocks.dtsi
index 4eed84feb761..a71aa2996321 100644
--- a/arch/arm/boot/dts/k2hk-clocks.dtsi
+++ b/arch/arm/boot/dts/k2hk-clocks.dtsi
@@ -339,7 +339,7 @@ clocks {
compatible = "ti,keystone,psc-clock";
clocks = <&chipclk13>;
clock-output-names = "vcp-3";
- reg = <0x0235000a8 0xb00>, <0x02350060 0x400>;
+ reg = <0x023500a8 0xb00>, <0x02350060 0x400>;
reg-names = "control", "domain";
domain-id = <24>;
};
--
1.9.0

--
Cheers,
Stephen Rothwell [email protected]


Attachments:
(No filename) (1.29 kB)
(No filename) (836.00 B)
Download all attachments

2014-02-26 07:29:22

by Mike Turquette

[permalink] [raw]
Subject: Re: linux-next: manual merge of the clk tree with the keystone tree

On Tue, Feb 25, 2014 at 11:15 PM, Stephen Rothwell <[email protected]> wrote:
> Hi Mike,
>
> Today's linux-next merge of the clk tree got a conflict in
> arch/arm/boot/dts/keystone-clocks.dtsi between commit 0cfc9ccec2a8 ("ARM:
> dts: keystone: preparatory patch to support K2L and K2E SOCs") from the
> keystone tree and commit 565bbdcd3b91 ("ARM: keystone: dts: fix clkvcp3
> control register address") from the clk tree.
>
> I fixed it up (by adding the following merge fix patch) and can carry the
> fix as necessary (no action is required).
>
> From: Stephen Rothwell <[email protected]>
> Date: Wed, 26 Feb 2014 18:12:55 +1100
> Subject: [PATCH] ARM: keystone: dts: fix for code movement
>
> Signed-off-by: Stephen Rothwell <[email protected]>

Looks good to me.

Thanks!
Mike

> ---
> arch/arm/boot/dts/k2hk-clocks.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/k2hk-clocks.dtsi b/arch/arm/boot/dts/k2hk-clocks.dtsi
> index 4eed84feb761..a71aa2996321 100644
> --- a/arch/arm/boot/dts/k2hk-clocks.dtsi
> +++ b/arch/arm/boot/dts/k2hk-clocks.dtsi
> @@ -339,7 +339,7 @@ clocks {
> compatible = "ti,keystone,psc-clock";
> clocks = <&chipclk13>;
> clock-output-names = "vcp-3";
> - reg = <0x0235000a8 0xb00>, <0x02350060 0x400>;
> + reg = <0x023500a8 0xb00>, <0x02350060 0x400>;
> reg-names = "control", "domain";
> domain-id = <24>;
> };
> --
> 1.9.0
>
> --
> Cheers,
> Stephen Rothwell [email protected]

2014-02-26 11:41:13

by Ivan Khoronzhuk

[permalink] [raw]
Subject: Re: linux-next: manual merge of the clk tree with the keystone tree


On 02/26/2014 09:15 AM, Stephen Rothwell wrote:
> Hi Mike,
>
> Today's linux-next merge of the clk tree got a conflict in
> arch/arm/boot/dts/keystone-clocks.dtsi between commit 0cfc9ccec2a8 ("ARM:
> dts: keystone: preparatory patch to support K2L and K2E SOCs") from the
> keystone tree and commit 565bbdcd3b91 ("ARM: keystone: dts: fix clkvcp3
> control register address") from the clk tree.
>
> I fixed it up (by adding the following merge fix patch) and can carry the
> fix as necessary (no action is required).
>
> From: Stephen Rothwell <[email protected]>
> Date: Wed, 26 Feb 2014 18:12:55 +1100
> Subject: [PATCH] ARM: keystone: dts: fix for code movement
>
> Signed-off-by: Stephen Rothwell <[email protected]>
> ---
> arch/arm/boot/dts/k2hk-clocks.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/k2hk-clocks.dtsi b/arch/arm/boot/dts/k2hk-clocks.dtsi
> index 4eed84feb761..a71aa2996321 100644
> --- a/arch/arm/boot/dts/k2hk-clocks.dtsi
> +++ b/arch/arm/boot/dts/k2hk-clocks.dtsi
> @@ -339,7 +339,7 @@ clocks {
> compatible = "ti,keystone,psc-clock";
> clocks = <&chipclk13>;
> clock-output-names = "vcp-3";
> - reg = <0x0235000a8 0xb00>, <0x02350060 0x400>;
> + reg = <0x023500a8 0xb00>, <0x02350060 0x400>;
> reg-names = "control", "domain";
> domain-id = <24>;
> };

The patch "ARM: dts: keystone: preparatory patch to support K2L and K2E
SOCs"
is from "[PATCH 0/4] ARM: dts: keystone: Add support for K2H and K2E
SOCs/EVMs"
series (http://www.spinics.net/lists/arm-kernel/msg310946.html).

As I understand, this series splits keystone clock tree into tree boards.
As result clock "clkvcp3" (for wich reg is corrected) is moved to the
following two dts:

k2hk-clocks.dtsi
k2l-clocks.dtsi

So probably you should apply this change to k2l-clocks.dtsi too.

--
Regards,
Ivan Khoronzhuk

2014-02-26 14:26:35

by Santosh Shilimkar

[permalink] [raw]
Subject: Re: linux-next: manual merge of the clk tree with the keystone tree

On Wednesday 26 February 2014 06:41 AM, Khoronzhuk, Ivan wrote:
>
> On 02/26/2014 09:15 AM, Stephen Rothwell wrote:
>> Hi Mike,
>>
>> Today's linux-next merge of the clk tree got a conflict in
>> arch/arm/boot/dts/keystone-clocks.dtsi between commit 0cfc9ccec2a8 ("ARM:
>> dts: keystone: preparatory patch to support K2L and K2E SOCs") from the
>> keystone tree and commit 565bbdcd3b91 ("ARM: keystone: dts: fix clkvcp3
>> control register address") from the clk tree.
>>
>> I fixed it up (by adding the following merge fix patch) and can carry the
>> fix as necessary (no action is required).
>>
>> From: Stephen Rothwell <[email protected]>
>> Date: Wed, 26 Feb 2014 18:12:55 +1100
>> Subject: [PATCH] ARM: keystone: dts: fix for code movement
>>
>> Signed-off-by: Stephen Rothwell <[email protected]>
>> ---
>> arch/arm/boot/dts/k2hk-clocks.dtsi | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/k2hk-clocks.dtsi b/arch/arm/boot/dts/k2hk-clocks.dtsi
>> index 4eed84feb761..a71aa2996321 100644
>> --- a/arch/arm/boot/dts/k2hk-clocks.dtsi
>> +++ b/arch/arm/boot/dts/k2hk-clocks.dtsi
>> @@ -339,7 +339,7 @@ clocks {
>> compatible = "ti,keystone,psc-clock";
>> clocks = <&chipclk13>;
>> clock-output-names = "vcp-3";
>> - reg = <0x0235000a8 0xb00>, <0x02350060 0x400>;
>> + reg = <0x023500a8 0xb00>, <0x02350060 0x400>;
>> reg-names = "control", "domain";
>> domain-id = <24>;
>> };
>
> The patch "ARM: dts: keystone: preparatory patch to support K2L and K2E
> SOCs"
> is from "[PATCH 0/4] ARM: dts: keystone: Add support for K2H and K2E
> SOCs/EVMs"
> series (http://www.spinics.net/lists/arm-kernel/msg310946.html).
>
> As I understand, this series splits keystone clock tree into tree boards.
Not boards but SOCs.

> As result clock "clkvcp3" (for wich reg is corrected) is moved to the
> following two dts:
>
> k2hk-clocks.dtsi
> k2l-clocks.dtsi
>
> So probably you should apply this change to k2l-clocks.dtsi too.
>
I will fix my tree. Thanks for spotting it Ivan !!

Regards,
Santosh

2014-02-26 15:00:15

by Karicheri, Muralidharan

[permalink] [raw]
Subject: RE: linux-next: manual merge of the clk tree with the keystone tree

>-----Original Message-----
>From: Khoronzhuk, Ivan
>Sent: Wednesday, February 26, 2014 6:41 AM
>To: Stephen Rothwell; Mike Turquette; Shilimkar, Santosh
>Cc: [email protected]; [email protected]; Karicheri, Muralidharan;
>Strashko, Grygorii
>Subject: Re: linux-next: manual merge of the clk tree with the keystone tree
>
>
>On 02/26/2014 09:15 AM, Stephen Rothwell wrote:
>> Hi Mike,
>>
>> Today's linux-next merge of the clk tree got a conflict in
>> arch/arm/boot/dts/keystone-clocks.dtsi between commit 0cfc9ccec2a8 ("ARM:
>> dts: keystone: preparatory patch to support K2L and K2E SOCs") from
>> the keystone tree and commit 565bbdcd3b91 ("ARM: keystone: dts: fix
>> clkvcp3 control register address") from the clk tree.
>>
>> I fixed it up (by adding the following merge fix patch) and can carry
>> the fix as necessary (no action is required).
>>
>> From: Stephen Rothwell <[email protected]>
>> Date: Wed, 26 Feb 2014 18:12:55 +1100
>> Subject: [PATCH] ARM: keystone: dts: fix for code movement
>>
>> Signed-off-by: Stephen Rothwell <[email protected]>
>> ---
>> arch/arm/boot/dts/k2hk-clocks.dtsi | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/k2hk-clocks.dtsi
>> b/arch/arm/boot/dts/k2hk-clocks.dtsi
>> index 4eed84feb761..a71aa2996321 100644
>> --- a/arch/arm/boot/dts/k2hk-clocks.dtsi
>> +++ b/arch/arm/boot/dts/k2hk-clocks.dtsi
>> @@ -339,7 +339,7 @@ clocks {
>> compatible = "ti,keystone,psc-clock";
>> clocks = <&chipclk13>;
>> clock-output-names = "vcp-3";
>> - reg = <0x0235000a8 0xb00>, <0x02350060 0x400>;
>> + reg = <0x023500a8 0xb00>, <0x02350060 0x400>;
>> reg-names = "control", "domain";
>> domain-id = <24>;
>> };
>
>The patch "ARM: dts: keystone: preparatory patch to support K2L and K2E SOCs"
>is from "[PATCH 0/4] ARM: dts: keystone: Add support for K2H and K2E SOCs/EVMs"
>series (http://www.spinics.net/lists/arm-kernel/msg310946.html).
>
>As I understand, this series splits keystone clock tree into tree boards.
>As result clock "clkvcp3" (for wich reg is corrected) is moved to the following two dts:
>
>k2hk-clocks.dtsi
>k2l-clocks.dtsi
>
>So probably you should apply this change to k2l-clocks.dtsi too.
>

Yes. The fix is needed for k2l-clocks.dtsi also. Thanks for taking care of this.

Murali Karicheri
Linux Kernel, Software Development

>--
>Regards,
>Ivan Khoronzhuk