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
