diff mbox series

[v2,1/3] ARM: dts: stm32: Duplicate cpu0-opp-table node into stm32mp15-u-boot.dtsi

Message ID 20241005011647.436456-1-marex@denx.de
State Accepted
Commit 241518cfce7701fddba17a023769d274e719e14e
Delegated to: Patrick Delaunay
Headers show
Series [v2,1/3] ARM: dts: stm32: Duplicate cpu0-opp-table node into stm32mp15-u-boot.dtsi | expand

Commit Message

Marek Vasut Oct. 5, 2024, 1:15 a.m. UTC
The cpu0-opp-table {} node does not exist in upstream Linux stm32mp151.dtsi
file, in order to enable conversion to OF_UPSTREAM, duplicate the node from
current U-Boot stm32mp151.dtsi into stm32mp15-u-boot.dtsi. This makes STM32
DTs buildable even with OF_UPSTREAM enabled. No functional change, since the
current U-Boot stm32mp151.dtsi already contains the cpu0-opp-table {} node,
stm32mp15-u-boot.dtsi is applied at the end, and does not bring in any new
content.

Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Patrice Chotard <patrice.chotard@foss.st.com>
Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
Cc: Sean Anderson <seanga2@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@dh-electronics.com
Cc: u-boot@lists.denx.de
Cc: uboot-stm32@st-md-mailman.stormreply.com
---
V2: No change
---
 arch/arm/dts/stm32mp15-u-boot.dtsi | 30 ++++++++++++++++++++++--------
 1 file changed, 22 insertions(+), 8 deletions(-)

Comments

Patrick DELAUNAY Oct. 11, 2024, 5:57 p.m. UTC | #1
Hi,

On 10/5/24 03:15, Marek Vasut wrote:
> The cpu0-opp-table {} node does not exist in upstream Linux stm32mp151.dtsi
> file, in order to enable conversion to OF_UPSTREAM, duplicate the node from
> current U-Boot stm32mp151.dtsi into stm32mp15-u-boot.dtsi. This makes STM32
> DTs buildable even with OF_UPSTREAM enabled. No functional change, since the
> current U-Boot stm32mp151.dtsi already contains the cpu0-opp-table {} node,
> stm32mp15-u-boot.dtsi is applied at the end, and does not bring in any new
> content.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Patrice Chotard <patrice.chotard@foss.st.com>
> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
> Cc: Sean Anderson <seanga2@gmail.com>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Tom Rini <trini@konsulko.com>
> Cc: u-boot@dh-electronics.com
> Cc: u-boot@lists.denx.de
> Cc: uboot-stm32@st-md-mailman.stormreply.com
> ---
> V2: No change
> ---
>   arch/arm/dts/stm32mp15-u-boot.dtsi | 30 ++++++++++++++++++++++--------
>   1 file changed, 22 insertions(+), 8 deletions(-)
>
....


Sorry for previous message,

Re: [PATCH 1/3] ARM: dts: stm32: Duplicate cpu0-opp-table node into 
stm32mp15-u-boot.dtsi

I answered on V1 by error.



Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>

Thanks
Patrick
Patrick DELAUNAY Oct. 16, 2024, 6:16 p.m. UTC | #2
Hi,

On 10/11/24 19:57, Patrick DELAUNAY wrote:
> Hi,
>
> On 10/5/24 03:15, Marek Vasut wrote:
>> The cpu0-opp-table {} node does not exist in upstream Linux 
>> stm32mp151.dtsi
>> file, in order to enable conversion to OF_UPSTREAM, duplicate the 
>> node from
>> current U-Boot stm32mp151.dtsi into stm32mp15-u-boot.dtsi. This makes 
>> STM32
>> DTs buildable even with OF_UPSTREAM enabled. No functional change, 
>> since the
>> current U-Boot stm32mp151.dtsi already contains the cpu0-opp-table {} 
>> node,
>> stm32mp15-u-boot.dtsi is applied at the end, and does not bring in 
>> any new
>> content.
>>
>> Signed-off-by: Marek Vasut <marex@denx.de>
>> ---
>> Cc: Patrice Chotard <patrice.chotard@foss.st.com>
>> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
>> Cc: Sean Anderson <seanga2@gmail.com>
>> Cc: Simon Glass <sjg@chromium.org>
>> Cc: Tom Rini <trini@konsulko.com>
>> Cc: u-boot@dh-electronics.com
>> Cc: u-boot@lists.denx.de
>> Cc: uboot-stm32@st-md-mailman.stormreply.com
>> ---
>> V2: No change
>> ---
>>   arch/arm/dts/stm32mp15-u-boot.dtsi | 30 ++++++++++++++++++++++--------
>>   1 file changed, 22 insertions(+), 8 deletions(-)
>>
> ....
>
>
> Sorry for previous message,
>
> Re: [PATCH 1/3] ARM: dts: stm32: Duplicate cpu0-opp-table node into 
> stm32mp15-u-boot.dtsi
>
> I answered on V1 by error.
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick


Applied to u-boot-stm/master, thanks!

Regards
Patrick
diff mbox series

Patch

diff --git a/arch/arm/dts/stm32mp15-u-boot.dtsi b/arch/arm/dts/stm32mp15-u-boot.dtsi
index fe56f05616a..66d4c40c6a8 100644
--- a/arch/arm/dts/stm32mp15-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp15-u-boot.dtsi
@@ -56,6 +56,24 @@ 
 			status = "okay";
 		};
 	};
+
+	cpu0_opp_table: cpu0-opp-table {
+		compatible = "operating-points-v2";
+		opp-shared;
+		bootph-pre-ram;
+		opp-650000000 {
+			bootph-pre-ram;
+			opp-hz = /bits/ 64 <650000000>;
+			opp-microvolt = <1200000>;
+			opp-supported-hw = <0x1>;
+		};
+		opp-800000000 {
+			bootph-pre-ram;
+			opp-hz = /bits/ 64 <800000000>;
+			opp-microvolt = <1350000>;
+			opp-supported-hw = <0x2>;
+		};
+	};
 };
 
 &bsec {
@@ -82,14 +100,10 @@ 
 	bootph-all;
 };
 
-&cpu0_opp_table {
-	bootph-pre-ram;
-	opp-650000000 {
-		bootph-pre-ram;
-	};
-	opp-800000000 {
-		bootph-pre-ram;
-	};
+&cpu0 {
+	nvmem-cells = <&part_number_otp>;
+	nvmem-cell-names = "part_number";
+	operating-points-v2 = <&cpu0_opp_table>;
 };
 
 &gpioa {