diff mbox series

ARM: dts: marvell: kirkwood: Fix at24 EEPROM node name

Message ID 20240910215914.823546-1-robh@kernel.org (mailing list archive)
State New, archived
Headers show
Series ARM: dts: marvell: kirkwood: Fix at24 EEPROM node name | expand

Commit Message

Rob Herring Sept. 10, 2024, 9:59 p.m. UTC
at24.yaml defines the node name for at24 EEPROMs as 'eeprom'.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrew Lunn Sept. 10, 2024, 10:20 p.m. UTC | #1
On Tue, Sep 10, 2024 at 04:59:13PM -0500, Rob Herring (Arm) wrote:
> at24.yaml defines the node name for at24 EEPROMs as 'eeprom'.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew
Rob Herring Nov. 4, 2024, 1:24 p.m. UTC | #2
On Tue, Sep 10, 2024 at 4:59 PM Rob Herring (Arm) <robh@kernel.org> wrote:
>
> at24.yaml defines the node name for at24 EEPROMs as 'eeprom'.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Ping!

Are Marvell platforms still maintained?

>
> diff --git a/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts b/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts
> index 9c438f10f737..2bc4b68bd723 100644
> --- a/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts
> +++ b/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts
> @@ -44,7 +44,7 @@ sata@80000 {
>                 i2c@11100 {
>                         status = "okay";
>
> -                       s24c02: s24c02@50 {
> +                       s24c02: eeprom@50 {
>                                 compatible = "atmel,24c02";
>                                 reg = <0x50>;
>                         };
> --
> 2.45.2
>
Andrew Lunn Nov. 4, 2024, 2:07 p.m. UTC | #3
On Mon, Nov 04, 2024 at 07:24:50AM -0600, Rob Herring wrote:
> On Tue, Sep 10, 2024 at 4:59 PM Rob Herring (Arm) <robh@kernel.org> wrote:
> >
> > at24.yaml defines the node name for at24 EEPROMs as 'eeprom'.
> >
> > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> > ---
> >  arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Ping!
> 
> Are Marvell platforms still maintained?

Yes, but on a sporadic basis. Patches tend to get collected somewhere
around -rc5/-rc6. So i would expect a notification from Gregory
sometime this week.

	Andrew
Gregory CLEMENT Nov. 4, 2024, 4:05 p.m. UTC | #4
Hello Rob,

> On Tue, Sep 10, 2024 at 4:59 PM Rob Herring (Arm) <robh@kernel.org> wrote:
>>
>> at24.yaml defines the node name for at24 EEPROMs as 'eeprom'.
>>
>> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
>> ---
>>  arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> Ping!
>
> Are Marvell platforms still maintained?

Actually I just applied it this morning, and was about to notify about
it.

Applied on mvebu/dt

Thanks,

Gregory

>
>>
>> diff --git a/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts b/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts
>> index 9c438f10f737..2bc4b68bd723 100644
>> --- a/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts
>> +++ b/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts
>> @@ -44,7 +44,7 @@ sata@80000 {
>>                 i2c@11100 {
>>                         status = "okay";
>>
>> -                       s24c02: s24c02@50 {
>> +                       s24c02: eeprom@50 {
>>                                 compatible = "atmel,24c02";
>>                                 reg = <0x50>;
>>                         };
>> --
>> 2.45.2
>>
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts b/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts
index 9c438f10f737..2bc4b68bd723 100644
--- a/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts
+++ b/arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts
@@ -44,7 +44,7 @@  sata@80000 {
 		i2c@11100 {
 			status = "okay";
 
-			s24c02: s24c02@50 {
+			s24c02: eeprom@50 {
 				compatible = "atmel,24c02";
 				reg = <0x50>;
 			};