On 4/18/25 18:44, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/11/25 18:03, Patrice Chotard wrote:
>> From: Simeon Marijon <simeon.mari...@foss.st.com>
>>
>> TAMP backup registers will be exposed as nvmem cells.
>>
>> Each registers ([0..127] for STM32MP2, [0..31] for STM32MP1) could be
>> exposed as nvmem cells under the nvram node in device tree
>>
>> Signed-off-by: Simeon Marijon <simeon.mari...@foss.st.com>
>> Signed-off-by: Patrice Chotard <patrice.chot...@foss.st.com>
>> ---
>>
>> arch/arm/mach-stm32mp/Kconfig | 9 +
>> arch/arm/mach-stm32mp/Makefile | 2 +
>> arch/arm/mach-stm32mp/nvram.c | 664 +++++++++++++++++++++++++++++++++
>> 3 files changed, 675 insertions(+)
>> create mode 100644 arch/arm/mach-stm32mp/nvram.c
>>
>> diff --git a/arch/arm/mach-stm32mp/Kconfig b/arch/arm/mach-stm32mp/Kconfig
>> index 002da2e3d3b..d7323659811 100644
>> --- a/arch/arm/mach-stm32mp/Kconfig
>> +++ b/arch/arm/mach-stm32mp/Kconfig
>> @@ -144,6 +144,15 @@ config STM32_ECDSA_VERIFY
>> ROM API provided on STM32MP.
>> The ROM API is only available during SPL for now.
>> +config STM32MP_TAMP_NVMEM
>> + bool "STM32 TAMP backup registers via NVMEM API"
>> + select NVMEM
>> + default y
>> + help
>> + Say y to enable the uclass driver for TAMP Backup registers using the
>> + NVMEM API. It allows to access to boot mode or others shared
>> information
>> + between software components/execution levels.
>> +
>> config CMD_STM32KEY
>> bool "command stm32key to fuse public key hash"
>> depends on CMDLINE
>> diff --git a/arch/arm/mach-stm32mp/Makefile b/arch/arm/mach-stm32mp/Makefile
>> index 103e3410ad9..bc0fe9e2225 100644
>> --- a/arch/arm/mach-stm32mp/Makefile
>> +++ b/arch/arm/mach-stm32mp/Makefile
>> @@ -13,6 +13,8 @@ obj-$(CONFIG_STM32MP13X) += stm32mp1/
>> obj-$(CONFIG_STM32MP25X) += stm32mp2/
>> obj-$(CONFIG_MFD_STM32_TIMERS) += timers.o
>> +obj-$(CONFIG_STM32MP_TAMP_NVMEM) += nvram.o
>
>
> I don't see reason to have reduce file name her ?
>
> filename indicate only the type nvram for nvmem used for TAMP backup register.
>
>
> I prefer fine name with indication of the device used, here TAMP ,
>
> and aligned with driver name "stm32_tamp_nvram"
>
> so rename nvram.c => tamp_nvram.c
>
> and
>
> +obj-$(CONFIG_STM32MP_TAMP_NVMEM) += tamp_nvram.o
Hi Patrick
Right, i will rename it as you suggested.
Thanks
Patrice
>
>
> thanks
>
>
> With this minor remark
>
>
> Reviewed-by: Patrick Delaunay <patrick.delau...@foss.st.com>
>
> Thanks
> Patrick
>
>
>
>
>