On 22:17 Fri 07 Aug , Prafulla Wadaskar wrote:
> eth_setenv_enetaddr is avaible by upper layer
> using this saves 204 bytes on total image size
>
> Signed-off-by: Prafulla Wadaskar <prafu...@marvell.com>
> ---
> drivers/net/kirkwood_egiga.c | 13 ++++++++-----
> 1 files changed, 8 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/net/kirkwood_egiga.c b/drivers/net/kirkwood_egiga.c
> index 3d908f8..1a208f2 100644
> --- a/drivers/net/kirkwood_egiga.c
> +++ b/drivers/net/kirkwood_egiga.c
> @@ -606,7 +606,7 @@ int kirkwood_egiga_initialize(bd_t * bis)
> struct kwgbe_device *dkwgbe;
> struct eth_device *dev;
> int devnum;
> - char *s, buf[NAMESIZE * 2];
> + char *s;
> u8 used_ports[MAX_KWGBE_DEVS] = CONFIG_KIRKWOOD_EGIGA_PORTS;
>
> for (devnum = 0; devnum < MAX_KWGBE_DEVS; devnum++) {
> @@ -665,10 +665,13 @@ int kirkwood_egiga_initialize(bd_t * bis)
>
> while (!eth_getenv_enetaddr(s, dev->enetaddr)) {
> /* Generate Ramdom MAC addresses if not set */
> - sprintf(buf, "00:50:43:%02x:%02x:%02x",
> - get_random_hex(), get_random_hex(),
> - get_random_hex());
> - setenv(s, buf);
> + dev->enetaddr[0] = 0x00;
> + dev->enetaddr[1] = 0x50;
> + dev->enetaddr[2] = 0x43;
> + dev->enetaddr[3] = get_random_hex();
> + dev->enetaddr[4] = get_random_hex();
> + dev->enetaddr[5] = get_random_hex();
> + eth_setenv_enetaddr(s, dev->enetaddr);
it will be also good to use a private mac address
dev->enetaddr[0] = 0x02;
Best Regards,
J.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
