2022-10-28 15:10:28

by Patrick Delaunay

[permalink] [raw]
Subject: [PATCH 0/3] nvmem: stm32: add OP-TEE support for STM32MP13x


This serie update the NVMEM BSEC driver to be compatible with STM32MP13x
SoC and the trusted application STM32MP BSEC in OP-TEE

This serie solve issue in initial support of STM32MP131
(using BSEC STM32MP15 compatible) and so it break the DTS compatible.

I create this serie for more efficient review.

The 2 first patches of this series are re-sent of patches already sent

1- "dt-bindings: nvmem: add new stm32mp13 compatible for stm32-romem"
https://lore.kernel.org/all/20221014172324.1.Ifc1812116ff63f5501f3edd155d3cf5c0ecc846c@changeid/
https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=685403

2- "ARM: dts: stm32mp13: fix compatible for BSEC"
https://lore.kernel.org/all/20221017134437.1.I167a5efc1f8777cce14518c6fa38400ac684de3e@changeid/
https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=685815

This DTS break is acceptable as
- the STM32MP13x SoC is not yet available outside STMicroelectronics
(not official)
- the same patch is already integrated or modifications are in progress in
the other users (arm-trusted-firmware/TF-A, OP-TEE and U-Boot) of
stm32mp131 device tree.

It is the good time to correct this issue before the real availability of
the SoC and before full support of SoC in Linux kernel.

This last patch on NVMEM STM32 ROMEM driver in depend on the preliminary
patch for the driver:

"nvmem: stm32: move STM32MP15_BSEC_NUM_LOWER in config"
https://lore.kernel.org/all/20221017174953.v2.1.I95e71328dd654723bd4c57206bd008ff81c726bb@changeid/

present in the serie

"nvmem: stm32: several minor improvements"
https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=685886

Regards

Patrick

Changes in v1:
- update commit message to indicate DTS break reason.

Patrick Delaunay (3):
dt-bindings: nvmem: add new stm32mp13 compatible for stm32-romem
ARM: dts: stm32mp13: fix compatible for BSEC
nvmem: stm32: add OP-TEE support for STM32MP13x

.../bindings/nvmem/st,stm32-romem.yaml | 1 +
arch/arm/boot/dts/stm32mp131.dtsi | 2 +-
drivers/nvmem/stm32-romem.c | 450 +++++++++++++++++-
3 files changed, 448 insertions(+), 5 deletions(-)

--
2.25.1



2022-10-31 18:19:45

by Srinivas Kandagatla

[permalink] [raw]
Subject: Re: [PATCH 0/3] nvmem: stm32: add OP-TEE support for STM32MP13x



On 28/10/2022 15:52, Patrick Delaunay wrote:
>
> This serie update the NVMEM BSEC driver to be compatible with STM32MP13x
> SoC and the trusted application STM32MP BSEC in OP-TEE
>
> This serie solve issue in initial support of STM32MP131
> (using BSEC STM32MP15 compatible) and so it break the DTS compatible.
>
> I create this serie for more efficient review.
>
> The 2 first patches of this series are re-sent of patches already sent
>
> 1- "dt-bindings: nvmem: add new stm32mp13 compatible for stm32-romem"
> https://lore.kernel.org/all/20221014172324.1.Ifc1812116ff63f5501f3edd155d3cf5c0ecc846c@changeid/
> https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=685403
>
> 2- "ARM: dts: stm32mp13: fix compatible for BSEC"
> https://lore.kernel.org/all/20221017134437.1.I167a5efc1f8777cce14518c6fa38400ac684de3e@changeid/
> https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=685815
>
> This DTS break is acceptable as
> - the STM32MP13x SoC is not yet available outside STMicroelectronics
> (not official)
> - the same patch is already integrated or modifications are in progress in
> the other users (arm-trusted-firmware/TF-A, OP-TEE and U-Boot) of
> stm32mp131 device tree.
>
> It is the good time to correct this issue before the real availability of
> the SoC and before full support of SoC in Linux kernel.
>
> This last patch on NVMEM STM32 ROMEM driver in depend on the preliminary
> patch for the driver:
>
> "nvmem: stm32: move STM32MP15_BSEC_NUM_LOWER in config"
> https://lore.kernel.org/all/20221017174953.v2.1.I95e71328dd654723bd4c57206bd008ff81c726bb@changeid/
>
> present in the serie
>
> "nvmem: stm32: several minor improvements"
> https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=685886
>
> Regards
>
> Patrick
>
> Changes in v1:
> - update commit message to indicate DTS break reason.
>
> Patrick Delaunay (3):
> dt-bindings: nvmem: add new stm32mp13 compatible for stm32-romem
> ARM: dts: stm32mp13: fix compatible for BSEC
> nvmem: stm32: add OP-TEE support for STM32MP13x

Applied 1/3 and 3/3 thanks,

dts changes need to go via the dts maintainer..

--srini
>
> .../bindings/nvmem/st,stm32-romem.yaml | 1 +
> arch/arm/boot/dts/stm32mp131.dtsi | 2 +-
> drivers/nvmem/stm32-romem.c | 450 +++++++++++++++++-
> 3 files changed, 448 insertions(+), 5 deletions(-)
>