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

[Marek Vasut]

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 at denx.de>
>> ---
>> Cc: Alexandre Torgue <alexandre.torgue at foss.st.com>
>> Cc: Christophe Roullier <christophe.roullier at foss.st.com>
>> Cc: Conor Dooley <conor+dt at kernel.org>
>> Cc: Krzysztof Kozlowski <krzk+dt at kernel.org>
>> Cc: Maxime Coquelin <mcoquelin.stm32 at gmail.com>
>> Cc: Rob Herring <robh at kernel.org>
>> Cc: devicetree at vger.kernel.org
>> Cc: kernel at dh-electronics.com
>> Cc: linux-arm-kernel at lists.infradead.org
>> Cc: linux-stm32 at st-md-mailman.stormreply.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 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 at 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.