diff options
| author | Rasmus Villemoes | 2022-05-20 13:19:08 +0200 |
|---|---|---|
| committer | Tom Rini | 2022-06-06 18:01:21 -0400 |
| commit | e5e04eaa2f1cb4dc37a12551018a00a18cab19de (patch) | |
| tree | 40cc7f350c8cf3f335098fe2f169176dfbecd5d8 /common/board_r.c | |
| parent | ba9aa40bb387385b8ef8b6594661a97ddcb8d04c (diff) | |
common/board_r.c: drop legacy and unused bi_enetaddr
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>
Diffstat (limited to 'common/board_r.c')
| -rw-r--r-- | common/board_r.c | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/common/board_r.c b/common/board_r.c index 4e3cf1f4ecc..ed29069d2de 100644 --- a/common/board_r.c +++ b/common/board_r.c @@ -469,18 +469,6 @@ static int initr_malloc_bootparams(void) } #endif -#ifdef CONFIG_CMD_NET -static int initr_ethaddr(void) -{ - struct bd_info *bd = gd->bd; - - /* kept around for legacy kernels only ... ignore the next section */ - eth_env_get_enetaddr("ethaddr", bd->bi_enetaddr); - - return 0; -} -#endif /* CONFIG_CMD_NET */ - #if defined(CONFIG_LED_STATUS) static int initr_status_led(void) { @@ -759,9 +747,6 @@ static init_fnc_t init_sequence_r[] = { initr_status_led, #endif /* PPC has a udelay(20) here dating from 2002. Why? */ -#ifdef CONFIG_CMD_NET - initr_ethaddr, -#endif #if defined(CONFIG_GPIO_HOG) gpio_hog_probe_all, #endif |