Re: [PATCH 1/1] avr32: added mem kernel command line option support

[Randy Dunlap <randy.dunlap@oracle.com>]

On Mon, 15 Sep 2008 12:07:19 +0200 Marco Stornelli wrote:

> From: Marco Stornelli <marco.stornelli@gmail.com>
> 
> Added support for the mem kernel command line option
> Signed-off-by: Marco Stornelli <marco.stornelli@gmail.com>
> ---
> --- setup.c.orig    2008-09-15 11:30:00.000000000 +0200
> +++ setup.c    2008-09-15 11:30:17.000000000 +0200
> @@ -283,6 +283,25 @@ static int __init early_parse_fbmem(char
>  }
>  early_param("fbmem", early_parse_fbmem);
>  
> +/*
> + * Pick out the memory size.  We look for mem=size@start,
> + * where start and size are "size[KkMm]"

                                    [KkMmGg]

> + */
> +static int __init early_mem(char **p)
> +{
> +    unsigned long size, start;
> +
> +    start = system_ram->start;
> +    size  = memparse(*p, p);

memparse() returns an unsigned long long.  Is the truncation OK for avr32?

> +    if (**p == '@')
> +        start = memparse(*p + 1, p);
> +
> +    system_ram->start = start;
> +    system_ram->end = system_ram->start + size - 1;
> +    return 0;
> +}
> +early_param("mem", early_mem);
> +
>  static int __init parse_tag_core(struct tag *tag)
>  {
>      if (tag->hdr.size > 2) {
> --



---
~Randy
Linux Plumbers Conference, 17-19 September 2008, Portland, Oregon USA
http://linuxplumbersconf.org/