Re: [PATCH] ramips: set Netgear R6220 MAC NVMEM cell directly in the part node

Felix Baumann felix.baumann at
Fri Jul 14 02:57:01 PDT 2023

Am 14. Juli 2023 10:26:33 MESZ schrieb "Rafał Miłecki" <zajec5 at>:
>On 9.05.2023 18:21, Felix Baumann wrote:
>> Am 9. Mai 2023 15:29:48 MESZ schrieb "Rafał Miłecki" <zajec5 at>:
>>> From: Rafał Miłecki <rafal at>
>>> There is no need to use reference if original node it specified in
>>> exactly the same file. This is a minor cleanup simplifying DTS code.
>>> Signed-off-by: Rafał Miłecki <rafal at>
>>> ---
>>> .../linux/ramips/dts/mt7621_netgear_r6220.dts  | 18 ++++++++----------
>>> 1 file changed, 8 insertions(+), 10 deletions(-)
>>> diff --git a/target/linux/ramips/dts/mt7621_netgear_r6220.dts b/target/linux/ramips/dts/mt7621_netgear_r6220.dts
>>> index 0f476ef060..7bb49c15b4 100644
>>> --- a/target/linux/ramips/dts/mt7621_netgear_r6220.dts
>>> +++ b/target/linux/ramips/dts/mt7621_netgear_r6220.dts
>>> @@ -42,9 +42,17 @@
>>> 		};
>>> 		factory: partition at 2e00000 {
>> The "factory" reference could be probably be dropped as well. Proposed changes look good to me.
>Actually it can't be dropped. While it's a bit counter-intuitive, that
>label is used in a *parent* DTS file mt7621_netgear_sercomm_ayx.dtsi .

Hello Rafał,

I was under the strong impression that dts(i) files could only get reference from the same file or parents but not childs. I remember reading a few exchanges by Ansuel, I think, that mentioned this.
I might be mixing two topics here, but was this (intended to be) fixed by dynamic partitions or something like that?

Are you certain the eeprom is loaded successfully?

Aren't dts(i) files read from top to bottom?

Felix Baumann

More information about the openwrt-devel mailing list