On 10/27/25 18:25, Patrice CHOTARD wrote:
> 
> 
> On 10/23/25 23:48, Marek Vasut wrote:
>> Add support for reading out the MAC address from SoC fuses on DH STM32MP1 
>> DHSOM.
>> The DH STM32MP1 DHSOM may contain external ethernet MACs, which benefit from 
>> the
>> MAC address stored in SoC fuses as well, handle those consistently. This 
>> however
>> means the architecture setup_mac_address() cannot be used and instead a 
>> simpler
>> local fuse read out is implemented in the board file.
>>
>> Signed-off-by: Marek Vasut <[email protected]>
>> ---
>> Cc: Patrice Chotard <[email protected]>
>> Cc: Patrick Delaunay <[email protected]>
>> Cc: Tom Rini <[email protected]>
>> Cc: [email protected]
>> Cc: [email protected]
>> Cc: [email protected]
>> ---
>>  board/dhelectronics/dh_stm32mp1/board.c | 29 +++++++++++++++++++++++++
>>  1 file changed, 29 insertions(+)
>>
>> diff --git a/board/dhelectronics/dh_stm32mp1/board.c 
>> b/board/dhelectronics/dh_stm32mp1/board.c
>> index 065d2f338c2..c18f1911fe4 100644
>> --- a/board/dhelectronics/dh_stm32mp1/board.c
>> +++ b/board/dhelectronics/dh_stm32mp1/board.c
>> @@ -119,6 +119,29 @@ static bool dh_stm32_mac_is_in_ks8851(void)
>>      return false;
>>  }
>>  
>> +static int dh_stm32_get_mac_from_fuse(unsigned char *enetaddr, int index)
>> +{
>> +    struct udevice *dev;
>> +    u8 otp[12];
>> +    int ret;
>> +
>> +    ret = uclass_get_device_by_driver(UCLASS_MISC,
>> +                                      DM_DRIVER_GET(stm32mp_bsec),
>> +                                      &dev);
>> +    if (ret)
>> +            return ret;
>> +
>> +    ret = misc_read(dev, STM32_BSEC_SHADOW(BSEC_OTP_MAC), otp, sizeof(otp));
>> +    if (ret < 0)
>> +            return ret;
>> +
>> +    memcpy(enetaddr, otp + ARP_HLEN * index, ARP_HLEN);
>> +    if (!is_valid_ethaddr(enetaddr))
>> +            return -EINVAL;
>> +
>> +    return 0;
>> +}
>> +
>>  static int dh_stm32_setup_ethaddr(struct eeprom_id_page *eip)
>>  {
>>      unsigned char enetaddr[6];
>> @@ -129,6 +152,9 @@ static int dh_stm32_setup_ethaddr(struct eeprom_id_page 
>> *eip)
>>      if (dh_get_mac_is_enabled("ethernet0"))
>>              return 0;
>>  
>> +    if (!dh_stm32_get_mac_from_fuse(enetaddr, 0))
>> +            goto out;
>> +
>>      if (!dh_get_value_from_eeprom_buffer(DH_MAC0, enetaddr, 
>> sizeof(enetaddr), eip))
>>              goto out;
>>  
>> @@ -154,6 +180,9 @@ static int dh_stm32_setup_eth1addr(struct eeprom_id_page 
>> *eip)
>>      if (dh_stm32_mac_is_in_ks8851())
>>              return 0;
>>  
>> +    if (!dh_stm32_get_mac_from_fuse(enetaddr, 1))
>> +            goto out;
>> +
>>      if (!dh_get_value_from_eeprom_buffer(DH_MAC1, enetaddr, 
>> sizeof(enetaddr), eip))
>>              goto out;
>>  
> 
> Hi 
> 
> Reviewed-by: Patrice Chotard <[email protected]>
> 
> Thanks
> Patrice
Applied to u-boot-stm32/master

Thanks
Patrice

Reply via email to