On 11/2/21 10:00, Michael Walle wrote:
On Fri, Oct 29, 2021 at 2:14 PM Michal Simek <michal.si...@xilinx.com> wrote:
When MAC address is randomly generated it should be also saved to
variables. This step is there when MAC address is passed via pdata but not
when it is randomly generated.
Signed-off-by: Michal Simek <michal.si...@xilinx.com>
---
net/eth-uclass.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/net/eth-uclass.c b/net/eth-uclass.c
index 0da0e85be031..58c308f33276 100644
--- a/net/eth-uclass.c
+++ b/net/eth-uclass.c
@@ -583,6 +583,8 @@ static int eth_post_probe(struct udevice *dev)
net_random_ethaddr(pdata->enetaddr);
printf("\nWarning: %s (eth%d) using random MAC address -
%pM\n",
dev->name, dev_seq(dev), pdata->enetaddr);
+ eth_env_set_enetaddr_by_index("eth", dev_seq(dev),
+ pdata->enetaddr);
#else
printf("\nError: %s address not set.\n",
dev->name);
--
2.33.1
Reviewed-by: Ramon Fried <rfried....@gmail.com>
Please note, that this will change behavior. Before this commit, the
random mac address was local to u-boot (at least for most network drivers).
After this commit, it will also be communicated to linux.
I'm not sure what to think of this. At the very least, this should be
documented in the commit message and in the Kconfig help text.
Thanks for bringing this up. I have no issue that this address is being
propagated to Linux but others can feel this as an issue.
I can definitely extend commit message to say it.
I found this via net list command where you can see controllers but you
can't see their mac addresses which is IMHO wrong.
Thanks,
Michal
