[PATCH] ARM: dts: stm32: Add nvmem-cells to ethernet nodes for constant mac-addresses

[PATCH] ARM: dts: stm32: Add nvmem-cells to ethernet nodes for constant mac-addresses

[Uwe Kleine-König]

The efuse device tree description already has the two labels pointing to
the efuse nodes that specify the mac-addresses to be used. Wire them up
to the ethernet nodes. This is enough to make barebox pick the right
mac-addresses and pass them to Linux.

Suggested-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
---
 arch/arm/boot/dts/st/stm32mp131.dtsi | 2 ++
 arch/arm/boot/dts/st/stm32mp133.dtsi | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/st/stm32mp131.dtsi b/arch/arm/boot/dts/st/stm32mp131.dtsi
index 8512a6e46b33..9175ed77d769 100644
--- a/arch/arm/boot/dts/st/stm32mp131.dtsi
+++ b/arch/arm/boot/dts/st/stm32mp131.dtsi
@@ -1609,6 +1609,8 @@ ethernet1: ethernet@5800a000 {
 				snps,axi-config = <&stmmac_axi_config_1>;
 				snps,tso;
 				access-controllers = <&etzpc 48>;
+				nvmem-cells = <&ethernet_mac1_address>;
+				nvmem-cell-names = "mac-address";
 				status = "disabled";
 
 				stmmac_axi_config_1: stmmac-axi-config {
diff --git a/arch/arm/boot/dts/st/stm32mp133.dtsi b/arch/arm/boot/dts/st/stm32mp133.dtsi
index 73e470019ce4..708c0af50bc4 100644
--- a/arch/arm/boot/dts/st/stm32mp133.dtsi
+++ b/arch/arm/boot/dts/st/stm32mp133.dtsi
@@ -91,6 +91,8 @@ ethernet2: ethernet@5800e000 {
 		snps,axi-config = <&stmmac_axi_config_2>;
 		snps,tso;
 		access-controllers = <&etzpc 49>;
+		nvmem-cells = <&ethernet_mac2_address>;
+		nvmem-cell-names = "mac-address";
 		status = "disabled";
 
 		stmmac_axi_config_2: stmmac-axi-config {

base-commit: e21edb1638e82460f126a6e49bcdd958d452929c
-- 
2.47.1

Re: [PATCH] ARM: dts: stm32: Add nvmem-cells to ethernet nodes for constant mac-addresses

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 538 bytes --]

Hello,

On Fri, Mar 28, 2025 at 06:14:05PM +0100, Uwe Kleine-König wrote:
> The efuse device tree description already has the two labels pointing to
> the efuse nodes that specify the mac-addresses to be used. Wire them up
> to the ethernet nodes. This is enough to make barebox pick the right
> mac-addresses and pass them to Linux.
> 
> Suggested-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>

I wonder if this patch is on someone's radar?!

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH] ARM: dts: stm32: Add nvmem-cells to ethernet nodes for constant mac-addresses

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 733 bytes --]

Hello,

On Thu, Apr 24, 2025 at 05:07:00PM +0200, Uwe Kleine-König wrote:
> On Fri, Mar 28, 2025 at 06:14:05PM +0100, Uwe Kleine-König wrote:
> > The efuse device tree description already has the two labels pointing to
> > the efuse nodes that specify the mac-addresses to be used. Wire them up
> > to the ethernet nodes. This is enough to make barebox pick the right
> > mac-addresses and pass them to Linux.
> > 
> > Suggested-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
> 
> I wonder if this patch is on someone's radar?!

I still wonder. Is this just due to -ENOTIME for the maintainers, or are
there doubts about this patch?

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH] ARM: dts: stm32: Add nvmem-cells to ethernet nodes for constant mac-addresses

[Alexandre TORGUE]

Hi Uwe

On 5/16/25 11:50, Uwe Kleine-König wrote:
> Hello,
> 
> On Thu, Apr 24, 2025 at 05:07:00PM +0200, Uwe Kleine-König wrote:
>> On Fri, Mar 28, 2025 at 06:14:05PM +0100, Uwe Kleine-König wrote:
>>> The efuse device tree description already has the two labels pointing to
>>> the efuse nodes that specify the mac-addresses to be used. Wire them up
>>> to the ethernet nodes. This is enough to make barebox pick the right
>>> mac-addresses and pass them to Linux.
>>>
>>> Suggested-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
>>> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
>>
>> I wonder if this patch is on someone's radar?!
> 
> I still wonder. Is this just due to -ENOTIME for the maintainers, or are
> there doubts about this patch?

No doubt just my fault as this patch went off my radar. Sorry.

Applied on stm32-next.

Thanks
Alex

> 
> Best regards
> Uwe