[U-Boot] [PATCH v2] cmd_bootm.c: Correct BOOTM_ERR_OVERLAP handling

Andreas Bießmann andreas.devel at googlemail.com
Tue Jul 2 12:49:51 CEST 2013


Hi all,

On 06/28/2013 11:41 PM, Simon Glass wrote:
> Hi Tom,
> 
> On Fri, Jun 28, 2013 at 1:25 PM, Tom Rini <trini at ti.com> wrote:

<snip>

>> @@ -660,34 +680,25 @@ static int do_bootm_states(cmd_tbl_t *cmdtp, int
>> flag, int argc,
>>         }
>>  #endif
>>         /* Now run the OS! We hope this doesn't return */
>> -       if (!ret && (states & BOOTM_STATE_OS_GO))
>> +       if (!ret && (states & BOOTM_STATE_OS_GO)) {
>>                 ret = boot_selected_os(argc, argv, BOOTM_STATE_OS_GO,
>>                                 images, boot_fn, &iflag);
>> +               if (ret)
>> +                       goto err;
>> +       }
>> +
>> +       return ret;
>>
> 
> Thanks for getting to the bottom of this.
> 
> Just a question here - should this fall through to display the error with
> the code below? For example if the subcommand is not supported...

as pointed out by Simon the error message 'subcommand not supported'
disappears with this patch. On avr32 this means currently:

---8<---
## Booting kernel from Legacy Image at 10400000 ...
   Image Name:   Linux-3.6.2
   Image Type:   AVR32 Linux Kernel Image (gzip compressed)
   Data Size:    1846216 Bytes = 1.8 MiB
   Load Address: 10000000
   Entry Point:  90000000
   Verifying Checksum ... OK
   Uncompressing Kernel Image ... OK
U-Boot>
--->8---

Before it was at least:
---8<---
## Booting kernel from Legacy Image at 10400000 ...
   Image Name:   Linux-3.6.2
   Image Type:   AVR32 Linux Kernel Image (gzip compressed)
   Data Size:    1846216 Bytes = 1.8 MiB
   Load Address: 10000000
   Entry Point:  90000000
   Verifying Checksum ... OK
   Uncompressing Kernel Image ... OK
subcommand not supported
U-Boot>
--->8---

Who will fix that? I'll have a look for the avr32 specific part.

Regards

Andreas Bießmann



More information about the U-Boot mailing list