mbox series

[v5,0/6] FWU: Add support for mtd backed feature on DeveloperBox

Message ID 20230410230143.72556-1-jaswinder.singh@linaro.org
Headers show
Series FWU: Add support for mtd backed feature on DeveloperBox | expand

Message

Jassi Brar April 10, 2023, 11:01 p.m. UTC
From: Jassi Brar <jaswinder.singh@linaro.org>

Introduce support for mtd backed storage for FWU feature and enable it on
Synquacer platform based DeveloperBox.

This revision is rebased onto patchset that trims the FWU api
 https://lore.kernel.org/u-boot/20230306231747.1888513-1-jassisinghbrar@gmail.com/

Changes since v4:
	* Provide default/weak implementations of fwu_plat_get_alt_num and fwu_plat_get_bootidx
	* Provide man page for mkfwumdata
	* Misc typo fixes and cosmetic changes

Changes since v3:
	* Fix and Update documentation to also build optee for FWU FIP image.
	* Fixed checkpatch warnings
	* Made local functions static.
	* Split config changes to a separate patch
	* Fix authorship of three patches.

Jassi Brar (4):
  dt: fwu: developerbox: enable fwu banks and mdata regions
  configs: move to new flash layout and boot flow
  fwu: DeveloperBox: add support for FWU
  fwu: provide default fwu_plat_get_bootidx

Masami Hiramatsu (2):
  FWU: Add FWU metadata access driver for MTD storage regions
  tools: Add mkfwumdata tool for FWU metadata image

 .../synquacer-sc2a11-developerbox-u-boot.dtsi |  49 ++-
 board/socionext/developerbox/Makefile         |   1 +
 board/socionext/developerbox/developerbox.c   |   8 +
 board/socionext/developerbox/fwu_plat.c       |  37 ++
 configs/synquacer_developerbox_defconfig      |  12 +-
 doc/board/socionext/developerbox.rst          | 155 +++++++-
 doc/mkfwumdata.1                              |  89 +++++
 drivers/fwu-mdata/Kconfig                     |  15 +
 drivers/fwu-mdata/Makefile                    |   1 +
 drivers/fwu-mdata/raw_mtd.c                   | 272 ++++++++++++++
 include/configs/synquacer.h                   |  10 +
 include/fwu.h                                 |  32 ++
 lib/fwu_updates/Makefile                      |   1 +
 lib/fwu_updates/fwu.c                         |  18 +
 lib/fwu_updates/fwu_mtd.c                     | 185 ++++++++++
 tools/Kconfig                                 |   9 +
 tools/Makefile                                |   4 +
 tools/mkfwumdata.c                            | 334 ++++++++++++++++++
 18 files changed, 1221 insertions(+), 11 deletions(-)
 create mode 100644 board/socionext/developerbox/fwu_plat.c
 create mode 100644 doc/mkfwumdata.1
 create mode 100644 drivers/fwu-mdata/raw_mtd.c
 create mode 100644 lib/fwu_updates/fwu_mtd.c
 create mode 100644 tools/mkfwumdata.c

Comments

Michal Simek April 14, 2023, 2:18 p.m. UTC | #1
On 4/11/23 01:01, jaswinder.singh@linaro.org wrote:
> From: Jassi Brar <jaswinder.singh@linaro.org>
> 
> Introduce support for mtd backed storage for FWU feature and enable it on
> Synquacer platform based DeveloperBox.
> 
> This revision is rebased onto patchset that trims the FWU api
>   https://lore.kernel.org/u-boot/20230306231747.1888513-1-jassisinghbrar@gmail.com/
> 
> Changes since v4:
> 	* Provide default/weak implementations of fwu_plat_get_alt_num and fwu_plat_get_bootidx
> 	* Provide man page for mkfwumdata
> 	* Misc typo fixes and cosmetic changes
> 
> Changes since v3:
> 	* Fix and Update documentation to also build optee for FWU FIP image.
> 	* Fixed checkpatch warnings
> 	* Made local functions static.
> 	* Split config changes to a separate patch
> 	* Fix authorship of three patches.
> 
> Jassi Brar (4):
>    dt: fwu: developerbox: enable fwu banks and mdata regions
>    configs: move to new flash layout and boot flow
>    fwu: DeveloperBox: add support for FWU
>    fwu: provide default fwu_plat_get_bootidx
> 
> Masami Hiramatsu (2):
>    FWU: Add FWU metadata access driver for MTD storage regions
>    tools: Add mkfwumdata tool for FWU metadata image
> 
>   .../synquacer-sc2a11-developerbox-u-boot.dtsi |  49 ++-
>   board/socionext/developerbox/Makefile         |   1 +
>   board/socionext/developerbox/developerbox.c   |   8 +
>   board/socionext/developerbox/fwu_plat.c       |  37 ++
>   configs/synquacer_developerbox_defconfig      |  12 +-
>   doc/board/socionext/developerbox.rst          | 155 +++++++-
>   doc/mkfwumdata.1                              |  89 +++++
>   drivers/fwu-mdata/Kconfig                     |  15 +
>   drivers/fwu-mdata/Makefile                    |   1 +
>   drivers/fwu-mdata/raw_mtd.c                   | 272 ++++++++++++++
>   include/configs/synquacer.h                   |  10 +
>   include/fwu.h                                 |  32 ++
>   lib/fwu_updates/Makefile                      |   1 +
>   lib/fwu_updates/fwu.c                         |  18 +
>   lib/fwu_updates/fwu_mtd.c                     | 185 ++++++++++
>   tools/Kconfig                                 |   9 +
>   tools/Makefile                                |   4 +
>   tools/mkfwumdata.c                            | 334 ++++++++++++++++++
>   18 files changed, 1221 insertions(+), 11 deletions(-)
>   create mode 100644 board/socionext/developerbox/fwu_plat.c
>   create mode 100644 doc/mkfwumdata.1
>   create mode 100644 drivers/fwu-mdata/raw_mtd.c
>   create mode 100644 lib/fwu_updates/fwu_mtd.c
>   create mode 100644 tools/mkfwumdata.c
> 

Jassi sent email to dt guys and I would wait for finding the right location for 
dt binding and get it review before this can be merged.

I have commented some stuff in v4 which I would like to see in v6 especially to 
explain that uuid which is not used anywhere.

Thanks,
Michal