| Message ID | 20241203233717.185016-1-rosenp@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | arm64: dts: armada-3720-gl-mv1000: use nvmem-layout | expand |
On Tue, Dec 03, 2024 at 03:37:17PM -0800, Rosen Penev wrote:
> nvmem-layout is a more flexible replacement for nvmem-cells.
Do you have this reference board? How did you test this patch?
Andrew
On Tue, Dec 3, 2024 at 3:44 PM Andrew Lunn <andrew@lunn.ch> wrote: > > On Tue, Dec 03, 2024 at 03:37:17PM -0800, Rosen Penev wrote: > > nvmem-layout is a more flexible replacement for nvmem-cells. > > Do you have this reference board? How did you test this patch? Only compile time tested. 2cc3b37f5b6df8189d55d0e812d9658ce256dfec marks nvmem-cells as legacy and most dts files have already been converted. This is not a crazy API change that would break anything. > > Andrew
diff --git a/arch/arm64/boot/dts/marvell/armada-3720-gl-mv1000.dts b/arch/arm64/boot/dts/marvell/armada-3720-gl-mv1000.dts index 56930f2ce481..7b801b60862d 100644 --- a/arch/arm64/boot/dts/marvell/armada-3720-gl-mv1000.dts +++ b/arch/arm64/boot/dts/marvell/armada-3720-gl-mv1000.dts @@ -98,10 +98,24 @@ partition@f0000 { reg = <0xf0000 0x8000>; }; - factory: partition@f8000 { + partition@f8000 { label = "factory"; reg = <0xf8000 0x8000>; read-only; + + nvmem-layout { + compatible = "fixed-layout"; + #address-cells = <1>; + #size-cells = <1>; + + macaddr_factory_0: macaddr@0 { + reg = <0x0 0x6>; + }; + + macaddr_factory_6: macaddr@6 { + reg = <0x6 0x6>; + }; + }; }; partition@100000 { @@ -221,17 +235,3 @@ fixed-link { full-duplex; }; }; - -&factory { - compatible = "nvmem-cells"; - #address-cells = <1>; - #size-cells = <1>; - - macaddr_factory_0: macaddr@0 { - reg = <0x0 0x6>; - }; - - macaddr_factory_6: macaddr@6 { - reg = <0x6 0x6>; - }; -};
nvmem-layout is a more flexible replacement for nvmem-cells. Signed-off-by: Rosen Penev <rosenp@gmail.com> --- .../dts/marvell/armada-3720-gl-mv1000.dts | 30 +++++++++---------- 1 file changed, 15 insertions(+), 15 deletions(-)