ARM: dts: stm32: Update eth1addr from EEPROM if eth1 present

The STM32MP1 DHCOM has two ethernet interfaces, the on-SoM DWMAC and KS8851.
Set eth1addr for the KS8851 to a MAC address of the DWMAC incremented by 1.
The MAC of the DWMAC is set from on-SoM EEPROM already, but the MAC address
of KS8851 was left uninitialized, so fix this.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Patrice Chotard <patrice.chotard@st.com>
Cc: Patrick Delaunay <patrick.delaunay@st.com>
Reviewed-by: Patrice Chotard <patrice.chotard@st.com>

diff --git a/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi b/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi
index f96de9e7a33ac0a9ea24612eff1977e61c73a5b1..92345b7ba3642cd2394085879c3c483ec220c68e 100644 (file)
--- a/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi
@@ -18,6 +18,7 @@
                mmc1 = &sdmmc2;
                spi0 = &qspi;
                usb0 = &usbotg_hs;
+               ethernet1 = &ksz8851;
        };
 
        config {

diff --git a/board/dhelectronics/dh_stm32mp1/board.c b/board/dhelectronics/dh_stm32mp1/board.c
index eec51d233e6c2c7d0416bb3472891d46fc3ea8aa..c9abe3cc6df193816f7f2e2cb4e69caf7b15cc80 100644 (file)
--- a/board/dhelectronics/dh_stm32mp1/board.c
+++ b/board/dhelectronics/dh_stm32mp1/board.c
@@ -84,11 +84,26 @@ DECLARE_GLOBAL_DATA_PTR;
 int setup_mac_address(void)
 {
        unsigned char enetaddr[6];
+       bool skip_eth0 = false;
+       bool skip_eth1 = false;
        struct udevice *dev;
        int off, ret;
 
        ret = eth_env_get_enetaddr("ethaddr", enetaddr);
        if (ret)        /* ethaddr is already set */
+               skip_eth0 = true;
+
+       off = fdt_path_offset(gd->fdt_blob, "ethernet1");
+       if (off < 0) {
+               /* ethernet1 is not present in the system */
+               skip_eth1 = true;
+       } else {
+               ret = eth_env_get_enetaddr("eth1addr", enetaddr);
+               if (ret)        /* eth1addr is already set */
+                       skip_eth1 = true;
+       }
+
+       if (skip_eth0 && skip_eth1)
                return 0;
 
        off = fdt_path_offset(gd->fdt_blob, "eeprom0");
@@ -109,8 +124,14 @@ int setup_mac_address(void)
                return ret;
        }
 
-       if (is_valid_ethaddr(enetaddr))
-               eth_env_set_enetaddr("ethaddr", enetaddr);
+       if (is_valid_ethaddr(enetaddr)) {
+               if (!skip_eth0)
+                       eth_env_set_enetaddr("ethaddr", enetaddr);
+
+               enetaddr[5]++;
+               if (!skip_eth1)
+                       eth_env_set_enetaddr("eth1addr", enetaddr);
+       }
 
        return 0;
 }