Skip to contentSkip to conversation

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

PATCH In review Rob Herring (Arm) 3 participants 5 messages Sep 10, 2024 – Nov 4, 2024

Subsystems (2)
lore
Share & tools
lore

Rob Herring (Arm)

author maintainer
#0 lore raw

PATCH 1/1 ARM: dts: marvell: kirkwood: Fix at24 EEPROM node name

at24.yaml defines the node name for at24 EEPROMs as 'eeprom'.

---
 arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts +1 -1
index 9c438f10f737..2bc4b68bd723 100644
@@ -44,7 +44,7 @@ sata@80000 {
 		i2c@11100 {
 			status = "okay";
 
-			s24c02: s24c02@50 {
+			s24c02: eeprom@50 {
 				compatible = "atmel,24c02";
 				reg = <0x50>;
 			};
Signature · 1 line
2.45.2
author maintainer
#2 lore raw
In reply to Rob Herring (Arm)
Quoted 7 lines · Rob Herring (Arm)

at24.yaml defines the node name for at24 EEPROMs as 'eeprom'.

---
 arch/arm/boot/dts/marvell/kirkwood-openblocks_a7.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Ping!

Are Marvell platforms still maintained?

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

2.45.2

maintainer
#3 lore raw
Quoted 4 lines · Rob Herring
Quoted 3 lines · Rob Herring (Arm)

at24.yaml defines the node name for at24 EEPROMs as 'eeprom'.

Quoted 8 lines
Quoted 4 lines
---
 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

maintainer
#4 lore raw
In reply to Rob Herring

Hello Rob,

Quoted 12 lines · Rob Herring (Arm)
Quoted 7 lines

at24.yaml defines the node name for at24 EEPROMs as 'eeprom'.

---
 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

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

2.45.2

Signature · 3 lines
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com