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

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de

Reply via email to