Re: [PATCH v2 0/6] corstone1000: fwu metadata and GPT

[Rui Miguel Silva <rui.silva@linaro.org>]

Hi all,
Rui Miguel Silva <rui.silva@linaro.org> writes:

> Now that the nvmxip block driver is merged we can add on top
> of it the platform code to use GPT and FWU metadata in the
> Corstone1000.
>
> But first, push 2 fixes that are needed to make all this work:
>  - move nvmxip header to include
>  - setup fwu metadata structures as packed (we have a 32bit
>    writer - Secure enclave Cortex-M0 and a 64bit reader host
>    Cortex-A35)
>
> Cheers,
>      Rui
>
> v1 [0]-> v2:
> Ilias:
>     - add Reviewed-by tag in patch 1/6
>
> Heinrich:
>     - fix test include nvmxip header after the move to include/
>         in patch 2/6

Any other feedback on this version?
Thanks in advance.

Cheers,
   Rui
   
>
> [0]: https://lore.kernel.org/u-boot/20230502131200.2551513-1-rui.silva@linaro.org/
>
> Rui Miguel Silva (6):
>   fwu_metadata: make sure structures are packed
>   nvmxip: move header to include
>   corstone1000: add fwu-metadata store info
>   corstone1000: add boot index
>   corstone1000: set kernel_addr based on boot_idx
>   corstone1000: add nvmxip, fwu-mdata and gpt options
>
>  arch/arm/dts/corstone1000.dtsi             |  7 ++-
>  board/armltd/corstone1000/corstone1000.c   | 73 +++++++++++++++++++++-
>  board/armltd/corstone1000/corstone1000.env | 10 +--
>  configs/corstone1000_defconfig             | 14 ++++-
>  include/fwu_mdata.h                        |  6 +-
>  {drivers/mtd/nvmxip => include}/nvmxip.h   |  0
>  test/dm/nvmxip.c                           |  2 +-
>  7 files changed, 96 insertions(+), 16 deletions(-)
>  rename {drivers/mtd/nvmxip => include}/nvmxip.h (100%)
>
> -- 
> 2.40.1