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

Reply via email to