Re: [PATCH] ARM: dts: stm32: Add phandle to nvmem efuse into STM32MP13xx ethernet DT node

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 6/27/24 2:13 PM, Alexandre TORGUE wrote:
Hi Marek

Hi,

On 6/23/24 21:15, Marek Vasut wrote:
The efuses on STM32MP13xx can be populated with ethernet MAC address.
Add the nvmem-cells/nvmem-cell-name DT properties to ethernet MAC DT
nodes to describe the placement of both MAC address fields within the
efuses, so the ethernet MAC driver can access the efuses and populate
the correct MAC address into the hardware.

Signed-off-by: Marek Vasut <marex@xxxxxxx>
---
Cc: Alexandre Torgue <alexandre.torgue@xxxxxxxxxxx>
Cc: Christophe Roullier <christophe.roullier@xxxxxxxxxxx>
Cc: Conor Dooley <conor+dt@xxxxxxxxxx>
Cc: Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>
Cc: Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx>
Cc: Rob Herring <robh@xxxxxxxxxx>
Cc: devicetree@xxxxxxxxxxxxxxx
Cc: kernel@xxxxxxxxxxxxxxxxxx
Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
Cc: linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx
---
  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 f41508195eb51..55f4150524f3c 100644
--- a/arch/arm/boot/dts/st/stm32mp131.dtsi
+++ b/arch/arm/boot/dts/st/stm32mp131.dtsi
@@ -1528,6 +1528,8 @@ ethernet1: ethernet@5800a000 {
                       <&rcc ETH1RX>,
                       <&rcc ETH1STP>,
                       <&rcc ETH1CK_K>;
+                nvmem-cell-names = "mac-address";
+                nvmem-cells = <&ethernet_mac1_address>;

It should not be defined in soc dtsi file but rather in board file. OTP are provisioned by the end customer not by ST. If you order a SoC to ST it will not be provisioned OTP MAC addr will be set to 0.

That's right, but at least the MAC address will be in the same locations in OTP, and if the MAC from OTP is all zeroes, than it wouldn't be considered a valid MAC address.

In any case, I'll do board specific V2 shortly.




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux