mbox series

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

Message ID 20230529140229.3225090-1-rui.silva@linaro.org
Headers show
Series corstone1000: fwu metadata and GPT | expand

Message

Rui Miguel Silva May 29, 2023, 2:02 p.m. UTC
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

[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%)

Comments

Rui Miguel Silva June 5, 2023, 6:05 p.m. UTC | #1
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
Tom Rini June 10, 2023, 12:37 a.m. UTC | #2
On Mon, May 29, 2023 at 03:02:23PM +0100, Rui Miguel Silva wrote:

> 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
> 
> [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

This patch leads to build failures now, please rebase on top of current
-next, thanks.