Re: [U-Boot] [PATCH v2] mvgbe: remove setting of ethaddr within the driver
Dear Michael Walle, In message 1321570349-4224-1-git-send-email-mich...@walle.cc you wrote: A network driver should not touch the environment at all. This patch fixes this behaviour by removing the code for setting a default/randomized MAC address. Signed-off-by: Michael Walle mich...@walle.cc Acked-by: Mike Frysinger vap...@gentoo.org Acked-by: Prafulla Wadaskar prafu...@marvell.com Cc: Mike Frysinger vap...@gentoo.org Cc: Prafulla Wadaskar prafu...@marvell.com Cc: Valentin Longchamp valentin.longch...@keymile.com Cc: Eric Cooper e...@cmu.edu Cc: Jason Cooper u-b...@lakedaemon.net Cc: Siddarth Gore go...@marvell.com Cc: Albert ARIBAUD albert.u.b...@aribaud.net Cc: Simon Guinot simon.gui...@sequanux.org --- drivers/net/mvgbe.c | 23 --- 1 files changed, 0 insertions(+), 23 deletions(-) Applied to next branch, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Any sufficiently advanced technology is indistinguishable from a rigged demo. - Andy Finkel, computer guy ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] mvgbe: remove setting of ethaddr within the driver
Dear Michael Walle, In message 1321570349-4224-1-git-send-email-mich...@walle.cc you wrote: A network driver should not touch the environment at all. This patch fixes this behaviour by removing the code for setting a default/randomized MAC address. Signed-off-by: Michael Walle mich...@walle.cc Acked-by: Mike Frysinger vap...@gentoo.org Acked-by: Prafulla Wadaskar prafu...@marvell.com Cc: Mike Frysinger vap...@gentoo.org Cc: Prafulla Wadaskar prafu...@marvell.com Cc: Valentin Longchamp valentin.longch...@keymile.com Cc: Eric Cooper e...@cmu.edu Cc: Jason Cooper u-b...@lakedaemon.net Cc: Siddarth Gore go...@marvell.com Cc: Albert ARIBAUD albert.u.b...@aribaud.net Cc: Simon Guinot simon.gui...@sequanux.org --- drivers/net/mvgbe.c | 23 --- 1 files changed, 0 insertions(+), 23 deletions(-) Applied to next branch, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de It is a good thing for an uneducated man to read books of quotations. - Sir Winston Churchill _My Early Life_ ch. 9 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] mvgbe: remove setting of ethaddr within the driver
-Original Message- From: Michael Walle [mailto:mich...@walle.cc] Sent: Friday, November 18, 2011 4:22 AM To: u-boot@lists.denx.de Cc: Wolfgang Denk; Michael Walle; Mike Frysinger; Prafulla Wadaskar; Valentin Longchamp; Eric Cooper; Jason Cooper; Siddarth Gore; Albert ARIBAUD; Simon Guinot Subject: [PATCH v2] mvgbe: remove setting of ethaddr within the driver A network driver should not touch the environment at all. This patch fixes this behaviour by removing the code for setting a default/randomized MAC address. Signed-off-by: Michael Walle mich...@walle.cc Acked-by: Mike Frysinger vap...@gentoo.org Acked-by: Prafulla Wadaskar prafu...@marvell.com Cc: Mike Frysinger vap...@gentoo.org Cc: Prafulla Wadaskar prafu...@marvell.com Cc: Valentin Longchamp valentin.longch...@keymile.com Cc: Eric Cooper e...@cmu.edu Cc: Jason Cooper u-b...@lakedaemon.net Cc: Siddarth Gore go...@marvell.com Cc: Albert ARIBAUD albert.u.b...@aribaud.net Cc: Simon Guinot simon.gui...@sequanux.org --- drivers/net/mvgbe.c | 23 --- 1 files changed, 0 insertions(+), 23 deletions(-) diff --git a/drivers/net/mvgbe.c b/drivers/net/mvgbe.c index c7f7446..21be642 100644 --- a/drivers/net/mvgbe.c +++ b/drivers/net/mvgbe.c @@ -647,7 +647,6 @@ int mvgbe_initialize(bd_t *bis) struct mvgbe_device *dmvgbe; struct eth_device *dev; int devnum; - char *s; u8 used_ports[MAX_MVGBE_DEVS] = CONFIG_MVGBE_PORTS; for (devnum = 0; devnum MAX_MVGBE_DEVS; devnum++) { @@ -702,16 +701,13 @@ error1: /* must be less than NAMESIZE (16) */ sprintf(dev-name, egiga%d, devnum); - /* Extract the MAC address from the environment */ switch (devnum) { case 0: dmvgbe-regs = (void *)MVGBE0_BASE; - s = ethaddr; break; #if defined(MVGBE1_BASE) case 1: dmvgbe-regs = (void *)MVGBE1_BASE; - s = eth1addr; break; #endif default:/* this should never happen */ @@ -720,25 +716,6 @@ error1: return -1; } - while (!eth_getenv_enetaddr(s, dev-enetaddr)) { - /* Generate Private MAC addr if not set */ - dev-enetaddr[0] = 0x02; - dev-enetaddr[1] = 0x50; - dev-enetaddr[2] = 0x43; -#if defined (CONFIG_SKIP_LOCAL_MAC_RANDOMIZATION) - /* Generate fixed lower MAC half using devnum */ - dev-enetaddr[3] = 0; - dev-enetaddr[4] = 0; - dev-enetaddr[5] = devnum; -#else - /* Generate random lower MAC half */ - dev-enetaddr[3] = get_random_hex(); - dev-enetaddr[4] = get_random_hex(); - dev-enetaddr[5] = get_random_hex(); -#endif - eth_setenv_enetaddr(s, dev-enetaddr); - } - dev-init = (void *)mvgbe_init; dev-halt = (void *)mvgbe_halt; dev-send = (void *)mvgbe_send; -- 1.7.2.5 Ack for this patch Acked-by: Prafulla Wadaskar prafu...@marvell.com Regards.. Prafulla . . . ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] mvgbe: remove setting of ethaddr within the driver
On Thursday 17 November 2011 17:52:29 Michael Walle wrote: --- a/drivers/net/mvgbe.c +++ b/drivers/net/mvgbe.c switch (devnum) { case 0: dmvgbe-regs = (void *)MVGBE0_BASE; unrelated, but usually this is pushed into an option to the init func ... the caller passes in the base address for the regs rather than the func trying to decode things itself. -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH v2] mvgbe: remove setting of ethaddr within the driver
A network driver should not touch the environment at all. This patch fixes this behaviour by removing the code for setting a default/randomized MAC address. Signed-off-by: Michael Walle mich...@walle.cc Acked-by: Mike Frysinger vap...@gentoo.org Acked-by: Prafulla Wadaskar prafu...@marvell.com Cc: Mike Frysinger vap...@gentoo.org Cc: Prafulla Wadaskar prafu...@marvell.com Cc: Valentin Longchamp valentin.longch...@keymile.com Cc: Eric Cooper e...@cmu.edu Cc: Jason Cooper u-b...@lakedaemon.net Cc: Siddarth Gore go...@marvell.com Cc: Albert ARIBAUD albert.u.b...@aribaud.net Cc: Simon Guinot simon.gui...@sequanux.org --- drivers/net/mvgbe.c | 23 --- 1 files changed, 0 insertions(+), 23 deletions(-) diff --git a/drivers/net/mvgbe.c b/drivers/net/mvgbe.c index c7f7446..21be642 100644 --- a/drivers/net/mvgbe.c +++ b/drivers/net/mvgbe.c @@ -647,7 +647,6 @@ int mvgbe_initialize(bd_t *bis) struct mvgbe_device *dmvgbe; struct eth_device *dev; int devnum; - char *s; u8 used_ports[MAX_MVGBE_DEVS] = CONFIG_MVGBE_PORTS; for (devnum = 0; devnum MAX_MVGBE_DEVS; devnum++) { @@ -702,16 +701,13 @@ error1: /* must be less than NAMESIZE (16) */ sprintf(dev-name, egiga%d, devnum); - /* Extract the MAC address from the environment */ switch (devnum) { case 0: dmvgbe-regs = (void *)MVGBE0_BASE; - s = ethaddr; break; #if defined(MVGBE1_BASE) case 1: dmvgbe-regs = (void *)MVGBE1_BASE; - s = eth1addr; break; #endif default:/* this should never happen */ @@ -720,25 +716,6 @@ error1: return -1; } - while (!eth_getenv_enetaddr(s, dev-enetaddr)) { - /* Generate Private MAC addr if not set */ - dev-enetaddr[0] = 0x02; - dev-enetaddr[1] = 0x50; - dev-enetaddr[2] = 0x43; -#if defined (CONFIG_SKIP_LOCAL_MAC_RANDOMIZATION) - /* Generate fixed lower MAC half using devnum */ - dev-enetaddr[3] = 0; - dev-enetaddr[4] = 0; - dev-enetaddr[5] = devnum; -#else - /* Generate random lower MAC half */ - dev-enetaddr[3] = get_random_hex(); - dev-enetaddr[4] = get_random_hex(); - dev-enetaddr[5] = get_random_hex(); -#endif - eth_setenv_enetaddr(s, dev-enetaddr); - } - dev-init = (void *)mvgbe_init; dev-halt = (void *)mvgbe_halt; dev-send = (void *)mvgbe_send; -- 1.7.2.5 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot