[LinuxBIOS] Allow human friendly suffixes for the size option
Jordan Crouse
jordan.crouse at amd.com
Thu Jul 12 01:41:21 CEST 2007
On 12/07/07 01:19 +0200, Peter Stuge wrote:
> Great idea.
>
>
> On Wed, Jul 11, 2007 at 12:26:16PM -0600, Jordan Crouse wrote:
> > + printf(" \tUse a 'k' suffix to multiply the size by 1K or\n");
> > + printf(" \ta 'm' suffix to multiple the size by 1M.\n");
>
> I would prefer 1024 rather than 1K and 1048576 or 1024*1024 rather
> than 1M.
Good idea. I think 'dd' does the same thing.
>
> > +static void parse_larsize(char *str)
>
> static u32 ..
>
>
> > + larsize = size;
>
> .. and return size ?
>
> May it's useful somewhere else too? Ie. move to lib.c?
I don't know - this is the only place we do this sort of thing -
in fact, its probably overkill to have a function - we could do it
right in the switch statement if we wanted to.
Jordan
--
Jordan Crouse
Systems Software Development Engineer
Advanced Micro Devices, Inc.
More information about the coreboot
mailing list