[U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library dependency

[Simon Goldschmidt]

Dear Wolfgang,

On 22.11.2018 17:00, Wolfgang Denk wrote:
> Dear Simon,
>
> In message <CAAh8qswJ-hf_FCvTdOggpS4MO6pThKLefjYxcquaBuhKfdQO2w at mail.gmail.com> you wrote:
>>> I don't know your board.
>>>
>>> It may be intentional, for example if there are size restrictions
>>> for the SPL wich don't allow to include the CRC32 code...
>>>
>> I don't think this is intentional. I discussed this in another thread with
>> Marek by now. There is no reference to the CRC code in the files in
>> common/spl or the funtions called by it.
> Depending on hardware capabilities, this may be indeed intentional,
> for example to keep the size of the SPL below a certain limit or to
> allow for other, more critical features instead.
>
>> Searching the whole code, I cannot see anything that checks the uImage CRC
>> from standard SPL. And I don't see board specific controls to do this
>> check, did I miss something?
> This may well be possible.  But then, this is usually stage two
> already - the ROM bootloader did not check any SPL checksum
> either...

Well, the thing is: on my platform the boot rom *does* check SPL crc :-)

I wanted to post a patch for this, but from reading your response, this 
should probably be configurable. Especially as it doesn't only increase 
size but also boot time.

Thanks for your input on this!

Regards,
Simon