[PATCH v2] arc: dts: use 'atmel' as manufacturer for at24 in axs10x_mb

[PATCH v2] arc: dts: use 'atmel' as manufacturer for at24 in axs10x_mb

[Bartosz Golaszewski]

Using compatible strings without the <manufacturer> part for at24 is
deprecated since commit 6da28acf745f ("dt-bindings: at24: consistently
document the compatible property"). Use a correct 'atmel,<model>'
value.

Signed-off-by: Bartosz Golaszewski <brgl at bgdev.pl>
---
- explicitly mention the commit in which we deprecated compatible
  fallbacks other than "atmel,<manufacturer>"

 arch/arc/boot/dts/axs10x_mb.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arc/boot/dts/axs10x_mb.dtsi b/arch/arc/boot/dts/axs10x_mb.dtsi
index 74d070cd3c13..47b74fbc403c 100644
--- a/arch/arc/boot/dts/axs10x_mb.dtsi
+++ b/arch/arc/boot/dts/axs10x_mb.dtsi
@@ -214,13 +214,13 @@
 			};
 
 			eeprom at 0x54{
-				compatible = "24c01";
+				compatible = "atmel,24c01";
 				reg = <0x54>;
 				pagesize = <0x8>;
 			};
 
 			eeprom at 0x57{
-				compatible = "24c04";
+				compatible = "atmel,24c04";
 				reg = <0x57>;
 				pagesize = <0x8>;
 			};
-- 
2.16.1

[PATCH v2] arc: dts: use 'atmel' as manufacturer for at24 in axs10x_mb

[Alexey Brodkin]

Hi Bartosz,

On Tue, 2018-02-06@12:08 +0100, Bartosz Golaszewski wrote:
> Using compatible strings without the <manufacturer> part for at24 is
> deprecated since commit 6da28acf745f ("dt-bindings: at24: consistently
> document the compatible property"). Use a correct 'atmel,<model>'
> value.
> 
> Signed-off-by: Bartosz Golaszewski <brgl at bgdev.pl>

So my further input might be a bit unexpected :)

I took a look at the real board and understood that we specified in .dts
wrong ICs. Those Atmel parts are not connected to the CPU at all (used by some
external device) and instead I2C EEPROM we really have connected (but not yet
obviously used in the Linux kernel) is NXP's PCF8594C-2.

That said we may accept your change because it doesn't break anything
and then I'll send an update to axs10x_mb.dts with replacement of EEPROM nodes.
Or we may drop your change at all since I'm going to replace corresponding nodes.

-Alexey

[PATCH v2] arc: dts: use 'atmel' as manufacturer for at24 in axs10x_mb

[Bartosz Golaszewski]

2018-02-06 13:25 GMT+01:00 Alexey Brodkin <Alexey.Brodkin at synopsys.com>:
> Hi Bartosz,
>
> On Tue, 2018-02-06@12:08 +0100, Bartosz Golaszewski wrote:
>> Using compatible strings without the <manufacturer> part for at24 is
>> deprecated since commit 6da28acf745f ("dt-bindings: at24: consistently
>> document the compatible property"). Use a correct 'atmel,<model>'
>> value.
>>
>> Signed-off-by: Bartosz Golaszewski <brgl at bgdev.pl>
>
> So my further input might be a bit unexpected :)
>
> I took a look at the real board and understood that we specified in .dts
> wrong ICs. Those Atmel parts are not connected to the CPU at all (used by some
> external device) and instead I2C EEPROM we really have connected (but not yet
> obviously used in the Linux kernel) is NXP's PCF8594C-2.
>
> That said we may accept your change because it doesn't break anything
> and then I'll send an update to axs10x_mb.dts with replacement of EEPROM nodes.
> Or we may drop your change at all since I'm going to replace corresponding nodes.
>
> -Alexey

Looks like PCF8594C-2 is compatible with atmel,24c04 - when you'll be
submitting this patch, please extend the at24 DT bindings with a new
naming exception (just like the one for renesas).

Feel free to drop this patch if you plan to submit the fix soon -
otherwise, apply it if it may take some time.

Thanks,
Bartosz

[PATCH v2] arc: dts: use 'atmel' as manufacturer for at24 in axs10x_mb

[Alexey Brodkin]

Hi Bartosz,

> -----Original Message-----
> From: Bartosz Golaszewski [mailto:brgl at bgdev.pl]
> Sent: Tuesday, February 6, 2018 7:02 PM
> To: Alexey Brodkin <Alexey.Brodkin at synopsys.com>
> Cc: linux-kernel at vger.kernel.org; mark.rutland at arm.com; Vineet.Gupta1 at synopsys.com; robh+dt at kernel.org;
> devicetree at vger.kernel.org; linux-snps-arc at lists.infradead.org
> Subject: Re: [PATCH v2] arc: dts: use 'atmel' as manufacturer for at24 in axs10x_mb
> 
> 2018-02-06 13:25 GMT+01:00 Alexey Brodkin <Alexey.Brodkin at synopsys.com>:
> > Hi Bartosz,
> >
> > On Tue, 2018-02-06@12:08 +0100, Bartosz Golaszewski wrote:
> >> Using compatible strings without the <manufacturer> part for at24 is
> >> deprecated since commit 6da28acf745f ("dt-bindings: at24: consistently
> >> document the compatible property"). Use a correct 'atmel,<model>'
> >> value.
> >>
> >> Signed-off-by: Bartosz Golaszewski <brgl at bgdev.pl>
> >
> > So my further input might be a bit unexpected :)
> >
> > I took a look at the real board and understood that we specified in .dts
> > wrong ICs. Those Atmel parts are not connected to the CPU at all (used by some
> > external device) and instead I2C EEPROM we really have connected (but not yet
> > obviously used in the Linux kernel) is NXP's PCF8594C-2.
> >
> > That said we may accept your change because it doesn't break anything
> > and then I'll send an update to axs10x_mb.dts with replacement of EEPROM nodes.
> > Or we may drop your change at all since I'm going to replace corresponding nodes.
> >
> > -Alexey
> 
> Looks like PCF8594C-2 is compatible with atmel,24c04 - when you'll be
> submitting this patch, please extend the at24 DT bindings with a new
> naming exception (just like the one for renesas).

Thanks for checking this.

> Feel free to drop this patch if you plan to submit the fix soon -
> otherwise, apply it if it may take some time.

No I think it worth applying especially in the light of your comment about NXP EEPROM.

Vineet, could you please take this one?

-Alexey

[PATCH v2] arc: dts: use 'atmel' as manufacturer for at24 in axs10x_mb

[Vineet Gupta]

On 02/06/2018 01:04 PM, Alexey Brodkin wrote:
>> Feel free to drop this patch if you plan to submit the fix soon -
>> otherwise, apply it if it may take some time.
> No I think it worth applying especially in the light of your comment about NXP EEPROM.
>
> Vineet, could you please take this one?

I've added your Acked-by to patch and pushed to for-curr !

-Vineet