Re: [PATCH] ARM: dts: stm32: Fix reset for usart1 in scmi configuration

2024-01-19 Thread Patrice CHOTARD



On 1/4/24 18:33, Patrick DELAUNAY wrote:
> Hi,
> 
> On 1/4/24 13:37, Patrice Chotard wrote:
>> In SCMI configuration, usart1 is secure, so all its resources are secured
>> (clock and reset) and can't be set/unset by non-secure world but by OP-TEE.
>>
>> Fixes: 68d396bf ("ARM: dts: stm32: add SCMI version of STM32 boards 
>> (DK1/DK2/ED1/EV1)")
>>
>> Signed-off-by: Patrice Chotard 
>> ---
>>
>>   arch/arm/dts/stm32mp15-scmi-u-boot.dtsi | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi 
>> b/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi
>> index 7c8fec6cbfb..79494ecad90 100644
>> --- a/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi
>> +++ b/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi
>> @@ -135,7 +135,7 @@
>>   };
>>     &usart1 {
>> -    resets = <&rcc USART1_R>;
>> +    resets = <&scmi_reset RST_SCMI_USART1>;
>>   };
>>     &usart2 {
> 
> 
> Reviewed-by: Patrick Delaunay 
> 
> Thanks
> Patrick
> 
> 


Applied to u-boot-stm32/master

Thanks
Patrice


Re: [PATCH] ARM: dts: stm32: Fix reset for usart1 in scmi configuration

2024-01-04 Thread Patrick DELAUNAY

Hi,

On 1/4/24 13:37, Patrice Chotard wrote:

In SCMI configuration, usart1 is secure, so all its resources are secured
(clock and reset) and can't be set/unset by non-secure world but by OP-TEE.

Fixes: 68d396bf ("ARM: dts: stm32: add SCMI version of STM32 boards 
(DK1/DK2/ED1/EV1)")

Signed-off-by: Patrice Chotard 
---

  arch/arm/dts/stm32mp15-scmi-u-boot.dtsi | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi 
b/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi
index 7c8fec6cbfb..79494ecad90 100644
--- a/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi
@@ -135,7 +135,7 @@
  };
  
  &usart1 {

-   resets = <&rcc USART1_R>;
+   resets = <&scmi_reset RST_SCMI_USART1>;
  };
  
  &usart2 {



Reviewed-by: Patrick Delaunay 

Thanks
Patrick




[PATCH] ARM: dts: stm32: Fix reset for usart1 in scmi configuration

2024-01-04 Thread Patrice Chotard
In SCMI configuration, usart1 is secure, so all its resources are secured
(clock and reset) and can't be set/unset by non-secure world but by OP-TEE.

Fixes: 68d396bf ("ARM: dts: stm32: add SCMI version of STM32 boards 
(DK1/DK2/ED1/EV1)")

Signed-off-by: Patrice Chotard 
---

 arch/arm/dts/stm32mp15-scmi-u-boot.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi 
b/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi
index 7c8fec6cbfb..79494ecad90 100644
--- a/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp15-scmi-u-boot.dtsi
@@ -135,7 +135,7 @@
 };
 
 &usart1 {
-   resets = <&rcc USART1_R>;
+   resets = <&scmi_reset RST_SCMI_USART1>;
 };
 
 &usart2 {
-- 
2.25.1