Dear Wolfgang,
On 22.11.2018 17:00, Wolfgang Denk wrote:
Dear Simon,
In message CAAh8qswJ-hf_FCvTdOggpS4MO6pThKLefjYxcquaBuhKfdQO2w@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