[PATCH] ARM: stm32: Fix TAMP_SMCR BKP..PROT fields on STM32MP15xx

[Marek Vasut]

On 6/19/24 8:35 AM, Patrice CHOTARD wrote:
> 
> 
> On 6/19/24 00:57, Marek Vasut wrote:
>> Update the TAMP_SMCR BKP..PROT fields to put first 10 registers
>> into protection zone 1 and next 5 into zone 2. This fixes use of
>> boot counter which is often in zone 3 and has to be updated from
>> Linux, which runs in NS.
>>
>> Fixes: 73f7fc944cf6 ("ARM: stm32: Initialize TAMP_SMCR BKP..PROT fields on STM32MP15xx")
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> ---
>> Cc: Patrice Chotard <patrice.chotard at foss.st.com>
>> Cc: Patrick Delaunay <patrick.delaunay at foss.st.com>
>> Cc: Tom Rini <trini at konsulko.com>
>> Cc: u-boot at lists.denx.de
>> Cc: uboot-stm32 at st-md-mailman.stormreply.com
>> ---
>>   arch/arm/mach-stm32mp/stm32mp1/stm32mp15x.c | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/mach-stm32mp/stm32mp1/stm32mp15x.c b/arch/arm/mach-stm32mp/stm32mp1/stm32mp15x.c
>> index f096fe538d8..ca202bec8ee 100644
>> --- a/arch/arm/mach-stm32mp/stm32mp1/stm32mp15x.c
>> +++ b/arch/arm/mach-stm32mp/stm32mp1/stm32mp15x.c
>> @@ -148,8 +148,8 @@ static void security_init(void)
>>   	 */
>>   	clrsetbits_le32(TAMP_SMCR,
>>   			TAMP_SMCR_BKPRWDPROT | TAMP_SMCR_BKPWDPROT,
>> -			FIELD_PREP(TAMP_SMCR_BKPRWDPROT, 0x20) |
>> -			FIELD_PREP(TAMP_SMCR_BKPWDPROT, 0x20));
>> +			FIELD_PREP(TAMP_SMCR_BKPRWDPROT, 0x0A) |
>> +			FIELD_PREP(TAMP_SMCR_BKPWDPROT, 0x0F));
>>   
>>   	/* GPIOZ: deactivate the security */
>>   	writel(BIT(0), RCC_MP_AHB5ENSETR);
> 
> Reviewed-by: Patrice Chotard <patrice.chotard at foss.st.com>

I hope this bugfix will make it into 2024.10 , can you prepare a bugfix 
PR for 2024.10 ?