Message ID | 20220114133145.32329-1-pali@kernel.org |
---|---|
Headers | show |
Series | More verifications for kwbimage in SPL | expand |
On 1/14/22 14:31, Pali Rohár wrote: > Hello Stefan, > > this is v4 of series that adds more checks for kwbimage validity and > consistency to SPL, mainly checking image data checksum. > > Patches now passed Github CI testing: > https://github.com/u-boot/u-boot/pull/106 > > Changes since v3: > - fix compile erros detected by CI (patch 2) > - print addresses in hex format instead of dec format (patch 1,3,6) > - rebase on master branch as v3 does not apply (patch 8) > > Changes since v2: > - changed the error message in patch 6 according to Pali's suggestion > - added R-B tags > > Changes since v1: > - updated error messages as requested by Stefan > - fixed checkpatch warnings for uintN_t types (converted to preferred > uN) > - added more checkpatch fixes Applied to u-boot-marvell/master Thanks, Stefan > Marek Behún (4): > arm: mvebu: spl: Print srcaddr in error message > arm: mvebu: spl: Use preferred types u8/u16/u32 instead of uintN_t > arm: mvebu: spl: Use IS_ENABLED() instead of #ifdef where possible > arm: mvebu: spl: Fix 100 column exceeds > > Pali Rohár (5): > arm: mvebu: Check that kwbimage offset and blocksize are valid > SPL: Add struct spl_boot_device parameter into > spl_parse_board_header() > arm: mvebu: Check that kwbimage blockid matches boot mode > SPL: Add support for checking board / BootROM specific image types > arm: mvebu: Check for kwbimage data checksum > > arch/arm/mach-mvebu/spl.c | 133 +++++++++++++++++++--------- > arch/arm/mach-sunxi/spl_spi_sunxi.c | 2 +- > common/spl/spl.c | 13 ++- > common/spl/spl_ext.c | 9 +- > common/spl/spl_fat.c | 11 ++- > common/spl/spl_legacy.c | 3 +- > common/spl/spl_mmc.c | 43 +++++---- > common/spl/spl_nand.c | 13 +-- > common/spl/spl_net.c | 2 +- > common/spl/spl_nor.c | 4 +- > common/spl/spl_onenand.c | 2 +- > common/spl/spl_ram.c | 2 +- > common/spl/spl_sata.c | 9 +- > common/spl/spl_sdp.c | 2 +- > common/spl/spl_spi.c | 9 +- > common/spl/spl_ubi.c | 4 +- > common/spl/spl_usb.c | 4 +- > common/spl/spl_xip.c | 4 +- > common/spl/spl_ymodem.c | 4 +- > drivers/usb/gadget/f_sdp.c | 13 +-- > include/sdp.h | 3 +- > include/spl.h | 7 ++ > 22 files changed, 196 insertions(+), 100 deletions(-) > Viele Grüße, Stefan Roese