mbox series

[0/6] corstone1000: fwu metadata and GPT

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

Message

Rui Miguel Silva May 2, 2023, 1:11 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

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
 6 files changed, 95 insertions(+), 15 deletions(-)
 rename {drivers/mtd/nvmxip => include}/nvmxip.h (100%)

Comments

Rui Miguel Silva May 15, 2023, 1:27 p.m. UTC | #1
Hi,
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)

Gentle ping.

Cheers,
   Rui
>
> Cheers,
>      Rui
>
> 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
>  6 files changed, 95 insertions(+), 15 deletions(-)
>  rename {drivers/mtd/nvmxip => include}/nvmxip.h (100%)
>
> -- 
> 2.40.0
Rui Miguel Silva May 29, 2023, 8:39 a.m. UTC | #2
Hi,
Rui Miguel Silva <rui.silva@linaro.org> writes:
> Hi,
> 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)
>
> Gentle ping.
>
> Cheers,
>    Rui

It would be great to have some feedback on this ones.
Many thanks.

Cheers,
    Rui
    
>>
>> Cheers,
>>      Rui
>>
>> 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
>>  6 files changed, 95 insertions(+), 15 deletions(-)
>>  rename {drivers/mtd/nvmxip => include}/nvmxip.h (100%)
>>
>> -- 
>> 2.40.0