| Message ID | 20240523042750.26238-1-zajec5@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | ARM: dts: omap: convert NVMEM content to layout syntax | expand |
On 23.05.2024 06:27, Rafał Miłecki wrote: > From: Rafał Miłecki <rafal@milecki.pl> > > Use cleaner (and non-deprecated) bindings syntax. See commit > bd912c991d2e ("dt-bindings: nvmem: layouts: add fixed-layout") for > details. Ping? Could I get this PATCH picked up, please? > Signed-off-by: Rafał Miłecki <rafal@milecki.pl> > --- > .../boot/dts/ti/omap/am335x-bone-common.dtsi | 64 +++++++++++++------ > arch/arm/boot/dts/ti/omap/am335x-boneblue.dts | 12 ++-- > 2 files changed, 52 insertions(+), 24 deletions(-) > > diff --git a/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi b/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi > index 2d0216840ff5..898f862acf3e 100644 > --- a/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi > +++ b/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi > @@ -221,10 +221,14 @@ baseboard_eeprom: baseboard_eeprom@50 { > reg = <0x50>; > vcc-supply = <&ldo4_reg>; > > - #address-cells = <1>; > - #size-cells = <1>; > - baseboard_data: baseboard_data@0 { > - reg = <0 0x100>; > + nvmem-layout { > + compatible = "fixed-layout"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + baseboard_data: baseboard_data@0 { > + reg = <0 0x100>; > + }; > }; > }; > }; > @@ -239,40 +243,60 @@ &i2c2 { > cape_eeprom0: cape_eeprom0@54 { > compatible = "atmel,24c256"; > reg = <0x54>; > - #address-cells = <1>; > - #size-cells = <1>; > - cape0_data: cape_data@0 { > - reg = <0 0x100>; > + > + nvmem-layout { > + compatible = "fixed-layout"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + cape0_data: cape_data@0 { > + reg = <0 0x100>; > + }; > }; > }; > > cape_eeprom1: cape_eeprom1@55 { > compatible = "atmel,24c256"; > reg = <0x55>; > - #address-cells = <1>; > - #size-cells = <1>; > - cape1_data: cape_data@0 { > - reg = <0 0x100>; > + > + nvmem-layout { > + compatible = "fixed-layout"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + cape1_data: cape_data@0 { > + reg = <0 0x100>; > + }; > }; > }; > > cape_eeprom2: cape_eeprom2@56 { > compatible = "atmel,24c256"; > reg = <0x56>; > - #address-cells = <1>; > - #size-cells = <1>; > - cape2_data: cape_data@0 { > - reg = <0 0x100>; > + > + nvmem-layout { > + compatible = "fixed-layout"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + cape2_data: cape_data@0 { > + reg = <0 0x100>; > + }; > }; > }; > > cape_eeprom3: cape_eeprom3@57 { > compatible = "atmel,24c256"; > reg = <0x57>; > - #address-cells = <1>; > - #size-cells = <1>; > - cape3_data: cape_data@0 { > - reg = <0 0x100>; > + > + nvmem-layout { > + compatible = "fixed-layout"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + cape3_data: cape_data@0 { > + reg = <0 0x100>; > + }; > }; > }; > }; > diff --git a/arch/arm/boot/dts/ti/omap/am335x-boneblue.dts b/arch/arm/boot/dts/ti/omap/am335x-boneblue.dts > index 801399702547..8878da773d67 100644 > --- a/arch/arm/boot/dts/ti/omap/am335x-boneblue.dts > +++ b/arch/arm/boot/dts/ti/omap/am335x-boneblue.dts > @@ -317,10 +317,14 @@ baseboard_eeprom: baseboard_eeprom@50 { > compatible = "atmel,24c256"; > reg = <0x50>; > > - #address-cells = <1>; > - #size-cells = <1>; > - baseboard_data: baseboard_data@0 { > - reg = <0 0x100>; > + nvmem-layout { > + compatible = "fixed-layout"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + baseboard_data: baseboard_data@0 { > + reg = <0 0x100>; > + }; > }; > }; > };
Rafał Miłecki <zajec5@gmail.com> writes: > On 23.05.2024 06:27, Rafał Miłecki wrote: >> From: Rafał Miłecki <rafal@milecki.pl> >> >> Use cleaner (and non-deprecated) bindings syntax. See commit >> bd912c991d2e ("dt-bindings: nvmem: layouts: add fixed-layout") for >> details. > > Ping? Could I get this PATCH picked up, please? > Sorry for the lag here. Tony is stepping down from maintaining these platforms and I'm stepping into help (but with some lag.) I'll queue this up for v6.12. Note: I'll do a minor subject fix to add am335x-bone to be more specific. Thanks, Kevin
On Thu, 23 May 2024 06:27:50 +0200, Rafał Miłecki wrote: > Use cleaner (and non-deprecated) bindings syntax. See commit > bd912c991d2e ("dt-bindings: nvmem: layouts: add fixed-layout") for > details. > > Applied, thanks! [1/1] ARM: dts: omap: convert NVMEM content to layout syntax commit: 0f7d5b53a0fe94a890f1854c978d2a5b43efede3 Best regards,
diff --git a/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi b/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi index 2d0216840ff5..898f862acf3e 100644 --- a/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi +++ b/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi @@ -221,10 +221,14 @@ baseboard_eeprom: baseboard_eeprom@50 { reg = <0x50>; vcc-supply = <&ldo4_reg>; - #address-cells = <1>; - #size-cells = <1>; - baseboard_data: baseboard_data@0 { - reg = <0 0x100>; + nvmem-layout { + compatible = "fixed-layout"; + #address-cells = <1>; + #size-cells = <1>; + + baseboard_data: baseboard_data@0 { + reg = <0 0x100>; + }; }; }; }; @@ -239,40 +243,60 @@ &i2c2 { cape_eeprom0: cape_eeprom0@54 { compatible = "atmel,24c256"; reg = <0x54>; - #address-cells = <1>; - #size-cells = <1>; - cape0_data: cape_data@0 { - reg = <0 0x100>; + + nvmem-layout { + compatible = "fixed-layout"; + #address-cells = <1>; + #size-cells = <1>; + + cape0_data: cape_data@0 { + reg = <0 0x100>; + }; }; }; cape_eeprom1: cape_eeprom1@55 { compatible = "atmel,24c256"; reg = <0x55>; - #address-cells = <1>; - #size-cells = <1>; - cape1_data: cape_data@0 { - reg = <0 0x100>; + + nvmem-layout { + compatible = "fixed-layout"; + #address-cells = <1>; + #size-cells = <1>; + + cape1_data: cape_data@0 { + reg = <0 0x100>; + }; }; }; cape_eeprom2: cape_eeprom2@56 { compatible = "atmel,24c256"; reg = <0x56>; - #address-cells = <1>; - #size-cells = <1>; - cape2_data: cape_data@0 { - reg = <0 0x100>; + + nvmem-layout { + compatible = "fixed-layout"; + #address-cells = <1>; + #size-cells = <1>; + + cape2_data: cape_data@0 { + reg = <0 0x100>; + }; }; }; cape_eeprom3: cape_eeprom3@57 { compatible = "atmel,24c256"; reg = <0x57>; - #address-cells = <1>; - #size-cells = <1>; - cape3_data: cape_data@0 { - reg = <0 0x100>; + + nvmem-layout { + compatible = "fixed-layout"; + #address-cells = <1>; + #size-cells = <1>; + + cape3_data: cape_data@0 { + reg = <0 0x100>; + }; }; }; }; diff --git a/arch/arm/boot/dts/ti/omap/am335x-boneblue.dts b/arch/arm/boot/dts/ti/omap/am335x-boneblue.dts index 801399702547..8878da773d67 100644 --- a/arch/arm/boot/dts/ti/omap/am335x-boneblue.dts +++ b/arch/arm/boot/dts/ti/omap/am335x-boneblue.dts @@ -317,10 +317,14 @@ baseboard_eeprom: baseboard_eeprom@50 { compatible = "atmel,24c256"; reg = <0x50>; - #address-cells = <1>; - #size-cells = <1>; - baseboard_data: baseboard_data@0 { - reg = <0 0x100>; + nvmem-layout { + compatible = "fixed-layout"; + #address-cells = <1>; + #size-cells = <1>; + + baseboard_data: baseboard_data@0 { + reg = <0 0x100>; + }; }; }; };