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
I believe in this case legacy means non-device tree PowerPC Linux (and perhaps VxWorks?) since for ARM we would pass in ATAGS stuff instead but PowerPC we did do gd? Been so long...
Reviewed-by: Tom Rini trini@konsulko.com