[U-Boot] [PATCH 03/19] x86: zboot update

Graeme Russ graeme.russ at gmail.com
Mon Oct 4 12:00:07 CEST 2010


On 04/10/10 20:54, Sergei Shtylyov wrote:
> Hello.
> 
> On 04-10-2010 7:37, Graeme Russ wrote:
> 
>> The header of recent Linux Kernels includes the size of the image, and
>> therefore is not needed to be passed to zboot. Still process the third
>> parameter (size of image) in the event that an older kernel is being
>> loaded
> 
>> Signed-off-by: Graeme Russ<graeme.russ at gmail.com>
> [...]
> 
>> diff --git a/arch/i386/lib/zimage.c b/arch/i386/lib/zimage.c
>> index 89fe015..f279b43 100644
>> --- a/arch/i386/lib/zimage.c
>> +++ b/arch/i386/lib/zimage.c
> [...]
>> @@ -256,10 +257,17 @@ int do_zboot (cmd_tbl_t *cmdtp, int flag, int
>> argc, char * const argv[])
>>       /* Setup board for maximum PC/AT Compatibility */
>>       setup_pcat_compatibility();
>>
>> -    /* argv[1] holds the address of the bzImage */
>> -    bzImage_addr = (void *)simple_strtoul(argv[1], NULL, 16);
>> +    if (argc>= 2) {
>> +        /* argv[1] holds the address of the bzImage */
>> +        bzImage_addr = (void *)simple_strtoul(argv[1], NULL, 16);
>> +    } else {
>> +        s = getenv("fileaddr");
>> +        if (s)
>> +            bzImage_addr = (void *)simple_strtoul(s, NULL, 16);
>> +    }
> 
>    Why not:
> 
> +    if (argc >= 2)
> +        /* argv[1] holds the address of the bzImage */
> +        s = argv[1];
> +    else
> +        s = getenv("fileaddr");
> +
> +    if (s)
> +        bzImage_addr = (void *)simple_strtoul(s, NULL, 16);
> 
> WBR, Sergei
> 

Nice, thanks - will do

Regards,

Graeme


More information about the U-Boot mailing list