[U-Boot] [u-boot-release] [Patch v2] powerpc/eeprom: cleanup mac command
York Sun
yorksun at freescale.com
Thu Aug 11 16:35:30 CEST 2011
On Wed, 2011-08-10 at 20:27 -0500, Tabi Timur-B04825 wrote:
> York Sun wrote:
> > Change the help message to be more helpful. Print argument format.
> > Fix MAX_NUM_PORTS to comply with v1 NXID format.
> >
> > Signed-off-by: York Sun<yorksun at freescale.com>
>
> Could you also fix the commands so that they take a number in decimal
> instead of hex?
>
>
> e.mac[index][i] = simple_strtoul(p, &p, 16);
>
> ...
>
> e.mac_count = simple_strtoul(argv[2], NULL, 16);
>
> I know U-Boot uses hex for everything, but it doesn't make sense for
> this command.
Yes, I can change it. But not all of them. We use hexadecimal for mac
address but not the count of ports. Please comment on my next version.
York
More information about the U-Boot
mailing list