
7 Jun
2022
7 Jun
'22
6:47 p.m.
On Fri, May 20, 2022 at 01:19:08PM +0200, Rasmus Villemoes wrote:
The bi_enetaddr field in struct bd_info is write-only; nothing ever reads back the value.
Moreover, the value we write is more or less random, and certainly not something one can rely on: If the board has a writable environment and the mac address has been stored there, we fetch that value. But if the board doesn't, this code runs before initr_net() -> eth_initialize(), and thus before the code in eth-uclass which fetches MAC addresses from eeprom, fuses or whatnot and populates the (run-time) environment with those values.
Signed-off-by: Rasmus Villemoes rasmus.villemoes@prevas.dk Reviewed-by: Tom Rini trini@konsulko.com
Applied to u-boot/next, thanks!
--
Tom