[U-Boot] [PATCH 1/2] km/ivm: always set ethaddr after reading IVM

If we rebrand the IVM and ethaddr was set previously we need to change ethaddr. Otherwise we end up with a wrong MAC adress for the ethernet interface.
Cc: Heiko Schocher hs@denx.de Signed-off-by: Holger Brunck holger.brunck@keymile.com --- board/keymile/common/ivm.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/board/keymile/common/ivm.c b/board/keymile/common/ivm.c index 42db54221b..3495fafffe 100644 --- a/board/keymile/common/ivm.c +++ b/board/keymile/common/ivm.c @@ -302,14 +302,11 @@ static int ivm_populate_env(unsigned char *buf, int len)
/* if an offset is defined, add it */ process_mac(valbuf, page2, CONFIG_PIGGY_MAC_ADRESS_OFFSET); - if (getenv("ethaddr") == NULL) - setenv((char *)"ethaddr", (char *)valbuf); + setenv((char *)"ethaddr", (char *)valbuf); #ifdef CONFIG_KMVECT1 /* KMVECT1 has two ethernet interfaces */ - if (getenv("eth1addr") == NULL) { - process_mac(valbuf, page2, 1); - setenv((char *)"eth1addr", (char *)valbuf); - } + process_mac(valbuf, page2, 1); + setenv((char *)"eth1addr", (char *)valbuf); #endif
return 0;

It is possible to flag MAC addresses as locally administred. In this case they don't need to be unique. This is only allowed for interfaces which have no connection to the outside. For the TEGR1 board we use this feature.
Cc: Heiko Schocher hs@denx.de Signed-off-by: Holger Brunck holger.brunck@keymile.com --- board/keymile/common/ivm.c | 24 +++++++++++++++++++++--- 1 file changed, 21 insertions(+), 3 deletions(-)
diff --git a/board/keymile/common/ivm.c b/board/keymile/common/ivm.c index 3495fafffe..e9e518cf72 100644 --- a/board/keymile/common/ivm.c +++ b/board/keymile/common/ivm.c @@ -189,7 +189,7 @@ static int ivm_check_crc(unsigned char *buf, int block)
/* take care of the possible MAC address offset and the IVM content offset */ static int process_mac(unsigned char *valbuf, unsigned char *buf, - int offset) + int offset, bool unique) { unsigned char mac[6]; unsigned long val = (buf[4] << 16) + (buf[5] << 8) + buf[6]; @@ -199,6 +199,13 @@ static int process_mac(unsigned char *valbuf, unsigned char *buf, */ memcpy(mac, buf+1, 6);
+ /* MAC adress can be set to locally administred, this is only allowed + * for interfaces which have now connection to the outside. For these + * addresses we need to set the second bit in the first byte. + */ + if (!unique) + mac[0] |= 0x2; + if (offset) { val += offset; mac[3] = (val >> 16) & 0xff; @@ -300,12 +307,23 @@ static int ivm_populate_env(unsigned char *buf, int len) return 0; page2 = &buf[CONFIG_SYS_IVM_EEPROM_PAGE_LEN*2];
+#ifndef CONFIG_KMTEGR1 /* if an offset is defined, add it */ - process_mac(valbuf, page2, CONFIG_PIGGY_MAC_ADRESS_OFFSET); + process_mac(valbuf, page2, CONFIG_PIGGY_MAC_ADRESS_OFFSET, true); setenv((char *)"ethaddr", (char *)valbuf); #ifdef CONFIG_KMVECT1 /* KMVECT1 has two ethernet interfaces */ - process_mac(valbuf, page2, 1); + process_mac(valbuf, page2, 1, true); + setenv((char *)"eth1addr", (char *)valbuf); +#endif +#else +/* KMTEGR1 has a special setup. eth0 has no connection to the outside and + * gets an locally administred MAC address, eth1 is the debug interface and + * gets the official MAC address from the IVM + */ + process_mac(valbuf, page2, CONFIG_PIGGY_MAC_ADRESS_OFFSET, false); + setenv((char *)"ethaddr", (char *)valbuf); + process_mac(valbuf, page2, CONFIG_PIGGY_MAC_ADRESS_OFFSET, true); setenv((char *)"eth1addr", (char *)valbuf); #endif

On Thu, Jul 13, 2017 at 11:15:41AM +0200, Holger Brunck wrote:
It is possible to flag MAC addresses as locally administred. In this case they don't need to be unique. This is only allowed for interfaces which have no connection to the outside. For the TEGR1 board we use this feature.
Cc: Heiko Schocher hs@denx.de Signed-off-by: Holger Brunck holger.brunck@keymile.com
Applied to u-boot/master, thanks!

On Thu, Jul 13, 2017 at 11:15:40AM +0200, Holger Brunck wrote:
If we rebrand the IVM and ethaddr was set previously we need to change ethaddr. Otherwise we end up with a wrong MAC adress for the ethernet interface.
Cc: Heiko Schocher hs@denx.de Signed-off-by: Holger Brunck holger.brunck@keymile.com
Applied to u-boot/master, thanks!
participants (2)
-
Holger Brunck
-
Tom Rini