[U-Boot] [PATCH] imximage: header v2: Remove overwriting of flash_offset

stefano babic
2012-02-21T21:49:27Z

Am 21/02/2012 20:18, schrieb Dirk Behme:

>> I think we have then a problem when there is not a BOOT_FROM statement
>> in the configuration file, that let imxhdr->flash_offset unset.
>>
>> We need to set it with the default value
>
> Hmm, my understanding is that there can't be any default value?

This is desirable, but it does not correspond to the code.

> What do
> you think would be a default value? The one for SD boot? The one for
> NAND? Or? Who will decide what the default should be?

Whatever we choose, it is wrong. We can select SD, and one board has not
a SD interface. The same with NAND, ...

The best one is we have not a default value.

>
>> at the beginning of the
>> processing or to raise an error if we want to make this statement
>> mandatory in the configuration file.
>
> If there can't be a default value, I think to raise an error if no
> BOOT_FROM statement is there is the only option.

Agree - BOOT_FROM must be mandatory.

Best regards,
Stefano