[4/4] ARM: multi_v7_defconfig: Add SCMI regulator support

Message ID 20230712142432.1885162-5-p.paillet@foss.st.com
State New
Headers
Series STM32MP13x expose SCMI regulators |

Commit Message

p.paillet@foss.st.com July 12, 2023, 2:24 p.m. UTC
  From: Pascal Paillet <p.paillet@foss.st.com>

Enable ARM SCMI regulator support.

Signed-off-by: Pascal Paillet <p.paillet@foss.st.com>
---
 arch/arm/configs/multi_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Krzysztof Kozlowski July 12, 2023, 7:25 p.m. UTC | #1
On 12/07/2023 16:24, p.paillet@foss.st.com wrote:
> From: Pascal Paillet <p.paillet@foss.st.com>
> 
> Enable ARM SCMI regulator support.
> 

This we see from the diff. Please explain why, e.g. which boards use it.

Best regards,
Krzysztof
  
Patrick Delaunay July 26, 2023, 9:33 a.m. UTC | #2
Hi,

On 7/12/23 21:25, Krzysztof Kozlowski wrote:
> On 12/07/2023 16:24, p.paillet@foss.st.com wrote:
>> From: Pascal Paillet <p.paillet@foss.st.com>
>>
>> Enable ARM SCMI regulator support.
>>
> This we see from the diff. Please explain why, e.g. which boards use it.
>
> Best regards,
> Krzysztof


The STM32MP platforms rely on OP-TEE to boot: Clocks and regulators are 
handled in
the secure world by OP-TEE and exported to the non secure world (LINUX)
thanks to ARM SCMI protocol.

Today it is used in the ARMv7 STMicrolectronics stm32mp boards:
- for STM32MP13: stm32mp135f-dk
- the STM32MP15 boards with SCMI variant, introduced by commit 5b7e58313a77
   ("ARM: dts: stm32: Add SCMI version of STM32 boards (DK1/DK2/ED1/EV1)")
   * stm32mp157c-ev1-scmi
   * stm32mp157c-ed1-scmi
   * stm32mp157c-dk2-scmi
   * stm32mp157a-dk1-scmi

This SCMI server provide access to "secured" resources protected in SoC
(for example with RCC_SECFGR); the resources need to be available early
for these board and the  SCMI server is associated with OP-TEE, already
activated by the commit 24c159793238  ("ARM: multi_v7_defconfig: Add
OPTEE support").

Regards

Patrick

> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
  
Alexandre TORGUE Aug. 10, 2023, 4:13 p.m. UTC | #3
Hi Krzysztof

On 7/12/23 21:25, Krzysztof Kozlowski wrote:
> On 12/07/2023 16:24, p.paillet@foss.st.com wrote:
>> From: Pascal Paillet <p.paillet@foss.st.com>
>>
>> Enable ARM SCMI regulator support.
>>
> 
> This we see from the diff. Please explain why, e.g. which boards use it.
> 
> Best regards,
> Krzysztof
> 

If you agree I will add some words explaining SCMI regulator on STM32 
board in the commit message (As Patrick mentioned in his answer). I 
would like to have this patch in my next PR for v6.6. If no objections 
I'll do it directly when I merge the patch in stm32-next.

Cheers
Alex
  

Patch

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index f0800f806b5f..524ca56f52d9 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -612,6 +612,7 @@  CONFIG_MFD_ACER_A500_EC=m
 CONFIG_REGULATOR_ACT8865=y
 CONFIG_REGULATOR_ACT8945A=y
 CONFIG_REGULATOR_ANATOP=y
+CONFIG_REGULATOR_ARM_SCMI=y
 CONFIG_REGULATOR_AB8500=y
 CONFIG_REGULATOR_AS3711=y
 CONFIG_REGULATOR_AS3722=y