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 <ma...@denx.de>
---
Cc: Patrice Chotard <patrice.chot...@foss.st.com>
Cc: Patrick Delaunay <patrick.delau...@foss.st.com>
Cc: Tom Rini <tr...@konsulko.com>
Cc: u-boot@lists.denx.de
Cc: uboot-st...@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.chot...@foss.st.com>

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

Reply via email to