[PATCH v2] dts: keystone: Use new "ti,keystone-i2c" compatible

[PATCH v2] dts: keystone: Use new "ti,keystone-i2c" compatible

[Alexander Sverdlin]

Now as "i2c-davinci" driver has special handling for Keystone it's time to switch
the device tree to use new "compatible" property. Old one is left for backwards-
compatibility.

Signed-off-by: Alexander Sverdlin <alexander.sverdlin-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
---

Changes in v2:
- As suggested by Mark Rutland, kept the old "compatible" for backwards-
  compatibility

 arch/arm/boot/dts/keystone.dtsi |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/keystone.dtsi b/arch/arm/boot/dts/keystone.dtsi
index 72816d6..abd7455 100644
--- a/arch/arm/boot/dts/keystone.dtsi
+++ b/arch/arm/boot/dts/keystone.dtsi
@@ -106,7 +106,7 @@
 		};
 
 		i2c0: i2c@2530000 {
-			compatible = "ti,davinci-i2c";
+			compatible = "ti,keystone-i2c", "ti,davinci-i2c";
 			reg = <0x02530000 0x400>;
 			clock-frequency = <100000>;
 			clocks = <&clki2c>;
@@ -116,7 +116,7 @@
 		};
 
 		i2c1: i2c@2530400 {
-			compatible = "ti,davinci-i2c";
+			compatible = "ti,keystone-i2c", "ti,davinci-i2c";
 			reg = <0x02530400 0x400>;
 			clock-frequency = <100000>;
 			clocks = <&clki2c>;
@@ -126,7 +126,7 @@
 		};
 
 		i2c2: i2c@2530800 {
-			compatible = "ti,davinci-i2c";
+			compatible = "ti,keystone-i2c", "ti,davinci-i2c";
 			reg = <0x02530800 0x400>;
 			clock-frequency = <100000>;
 			clocks = <&clki2c>;

Re: [PATCH v2] dts: keystone: Use new "ti,keystone-i2c" compatible

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 392 bytes --]

On Mon, Sep 14, 2015 at 11:07:02AM +0200, Alexander Sverdlin wrote:
> Now as "i2c-davinci" driver has special handling for Keystone it's time to switch
> the device tree to use new "compatible" property. Old one is left for backwards-
> compatibility.
> 
> Signed-off-by: Alexander Sverdlin <alexander.sverdlin@nokia.com>

Shall I take this one or shall it go via the davinci tree?


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH v2] dts: keystone: Use new "ti,keystone-i2c" compatible

[Grygorii Strashko]

On 10/20/2015 06:06 PM, Wolfram Sang wrote:
> On Mon, Sep 14, 2015 at 11:07:02AM +0200, Alexander Sverdlin wrote:
>> Now as "i2c-davinci" driver has special handling for Keystone it's time to switch
>> the device tree to use new "compatible" property. Old one is left for backwards-
>> compatibility.
>>
>> Signed-off-by: Alexander Sverdlin <alexander.sverdlin@nokia.com>
>
> Shall I take this one or shall it go via the davinci tree?
>

To: Sekhar

-- 
regards,
-grygorii

Re: [PATCH v2] dts: keystone: Use new "ti,keystone-i2c" compatible

[Sekhar Nori]

On Tuesday 20 October 2015 09:40 PM, Grygorii Strashko wrote:
> On 10/20/2015 06:06 PM, Wolfram Sang wrote:
>> On Mon, Sep 14, 2015 at 11:07:02AM +0200, Alexander Sverdlin wrote:
>>> Now as "i2c-davinci" driver has special handling for Keystone it's
>>> time to switch
>>> the device tree to use new "compatible" property. Old one is left for
>>> backwards-
>>> compatibility.
>>>
>>> Signed-off-by: Alexander Sverdlin <alexander.sverdlin@nokia.com>
>>
>> Shall I take this one or shall it go via the davinci tree?

I don't have the patch in my mailbox now. But as I remember this is a
driver only patch and should go via i2c tree.

Thanks,
Sekhar

Re: [PATCH v2] dts: keystone: Use new "ti,keystone-i2c" compatible

[Grygorii Strashko]

On 09/14/2015 12:07 PM, Alexander Sverdlin wrote:
> Now as "i2c-davinci" driver has special handling for Keystone it's time to switch
> the device tree to use new "compatible" property. Old one is left for backwards-
> compatibility.
>
> Signed-off-by: Alexander Sverdlin <alexander.sverdlin@nokia.com>
> ---

To: Santosh
Cc: Murali, Sekhar

Seems this patch is Keystone 2 specific.

>
> Changes in v2:
> - As suggested by Mark Rutland, kept the old "compatible" for backwards-
>    compatibility
>
>   arch/arm/boot/dts/keystone.dtsi |    6 +++---
>   1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/keystone.dtsi b/arch/arm/boot/dts/keystone.dtsi
> index 72816d6..abd7455 100644
> --- a/arch/arm/boot/dts/keystone.dtsi
> +++ b/arch/arm/boot/dts/keystone.dtsi
> @@ -106,7 +106,7 @@
>   		};
>
>   		i2c0: i2c@2530000 {
> -			compatible = "ti,davinci-i2c";
> +			compatible = "ti,keystone-i2c", "ti,davinci-i2c";
>   			reg = <0x02530000 0x400>;
>   			clock-frequency = <100000>;
>   			clocks = <&clki2c>;
> @@ -116,7 +116,7 @@
>   		};
>
>   		i2c1: i2c@2530400 {
> -			compatible = "ti,davinci-i2c";
> +			compatible = "ti,keystone-i2c", "ti,davinci-i2c";
>   			reg = <0x02530400 0x400>;
>   			clock-frequency = <100000>;
>   			clocks = <&clki2c>;
> @@ -126,7 +126,7 @@
>   		};
>
>   		i2c2: i2c@2530800 {
> -			compatible = "ti,davinci-i2c";
> +			compatible = "ti,keystone-i2c", "ti,davinci-i2c";
>   			reg = <0x02530800 0x400>;
>   			clock-frequency = <100000>;
>   			clocks = <&clki2c>;
>


-- 
regards,
-grygorii

Re: [PATCH v2] dts: keystone: Use new "ti,keystone-i2c" compatible

[Alexander Sverdlin]

Hi!

On 21/10/15 12:13, EXT Sekhar Nori wrote:
>>>> Now as "i2c-davinci" driver has special handling for Keystone it's
>>>> >>> time to switch
>>>> >>> the device tree to use new "compatible" property. Old one is left for
>>>> >>> backwards-
>>>> >>> compatibility.
>>>> >>>
>>>> >>> Signed-off-by: Alexander Sverdlin <alexander.sverdlin-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
>>> >>
>>> >> Shall I take this one or shall it go via the davinci tree?
> I don't have the patch in my mailbox now. But as I remember this is a

You can review it here:
http://patchwork.ozlabs.org/patch/517321/

> driver only patch and should go via i2c tree.

It's a DT change, because we agreed (with Grygorii) that new "compatible" is more
future proof.

-- 
Best regards,
Alexander Sverdlin.
Sent from my pdp-11
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2] dts: keystone: Use new "ti,keystone-i2c" compatible

[santosh.shilimkar]

On 10/21/15 3:38 AM, Grygorii Strashko wrote:
> On 09/14/2015 12:07 PM, Alexander Sverdlin wrote:
>> Now as "i2c-davinci" driver has special handling for Keystone it's
>> time to switch
>> the device tree to use new "compatible" property. Old one is left for
>> backwards-
>> compatibility.
>>
>> Signed-off-by: Alexander Sverdlin <alexander.sverdlin@nokia.com>
>> ---
>
Will pick it up. Thanks !!

Regards,
Santosh