* [PATCH] arm64: dts: imx8mp: configure GPU and NPU clocks to overdrive rate
@ 2025-02-04 18:27 Lucas Stach
2025-02-23 7:16 ` Shawn Guo
2025-03-11 1:27 ` Shawn Guo
0 siblings, 2 replies; 4+ messages in thread
From: Lucas Stach @ 2025-02-04 18:27 UTC (permalink / raw)
To: Shawn Guo
Cc: Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
imx, linux-arm-kernel, patchwork-lst
A lot of other clocks on the i.MX8MP, including the DRAM set up by the
bootloader are already running at overdrive clock rates. While this is a
deviation from the configuration of other i.MX8M* family SoCs, overdrive
is the default for most i.MX8MP boards and only some special purpose
boards will choose to run the SoC at nominal drive rates. Up the GPU and
NPU clock rates to their overdrive level to be consistent with other
clocks set up in the dtsi.
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
index e0d3b8cba221..aeaa6a5c2f56 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
@@ -816,12 +816,12 @@ pgc_mlmix: power-domain@4 {
assigned-clocks = <&clk IMX8MP_CLK_ML_CORE>,
<&clk IMX8MP_CLK_ML_AXI>,
<&clk IMX8MP_CLK_ML_AHB>;
- assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>,
+ assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_1000M>,
<&clk IMX8MP_SYS_PLL1_800M>,
<&clk IMX8MP_SYS_PLL1_800M>;
- assigned-clock-rates = <800000000>,
+ assigned-clock-rates = <1000000000>,
<800000000>,
- <300000000>;
+ <400000000>;
};
pgc_audio: power-domain@5 {
@@ -2232,9 +2232,9 @@ gpu3d: gpu@38000000 {
clock-names = "core", "shader", "bus", "reg";
assigned-clocks = <&clk IMX8MP_CLK_GPU3D_CORE>,
<&clk IMX8MP_CLK_GPU3D_SHADER_CORE>;
- assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>,
- <&clk IMX8MP_SYS_PLL1_800M>;
- assigned-clock-rates = <800000000>, <800000000>;
+ assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_1000M>,
+ <&clk IMX8MP_SYS_PLL2_1000M>;
+ assigned-clock-rates = <1000000000>, <1000000000>;
power-domains = <&pgc_gpu3d>;
};
@@ -2247,8 +2247,8 @@ gpu2d: gpu@38008000 {
<&clk IMX8MP_CLK_GPU_AHB>;
clock-names = "core", "bus", "reg";
assigned-clocks = <&clk IMX8MP_CLK_GPU2D_CORE>;
- assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>;
- assigned-clock-rates = <800000000>;
+ assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_1000M>;
+ assigned-clock-rates = <1000000000>;
power-domains = <&pgc_gpu2d>;
};
base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b
--
2.39.5
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] arm64: dts: imx8mp: configure GPU and NPU clocks to overdrive rate
2025-02-04 18:27 [PATCH] arm64: dts: imx8mp: configure GPU and NPU clocks to overdrive rate Lucas Stach
@ 2025-02-23 7:16 ` Shawn Guo
2025-02-23 18:46 ` Lucas Stach
2025-03-11 1:27 ` Shawn Guo
1 sibling, 1 reply; 4+ messages in thread
From: Shawn Guo @ 2025-02-23 7:16 UTC (permalink / raw)
To: Lucas Stach
Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
devicetree, imx, linux-arm-kernel, patchwork-lst
On Tue, Feb 04, 2025 at 07:27:37PM +0100, Lucas Stach wrote:
> A lot of other clocks on the i.MX8MP, including the DRAM set up by the
> bootloader are already running at overdrive clock rates. While this is a
> deviation from the configuration of other i.MX8M* family SoCs, overdrive
> is the default for most i.MX8MP boards and only some special purpose
> boards will choose to run the SoC at nominal drive rates.
Are any of these special purpose boards in upstream? If so, does this
change have any impact on them?
Shawn
> Up the GPU and
> NPU clock rates to their overdrive level to be consistent with other
> clocks set up in the dtsi.
>
> Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
> ---
> arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> index e0d3b8cba221..aeaa6a5c2f56 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> @@ -816,12 +816,12 @@ pgc_mlmix: power-domain@4 {
> assigned-clocks = <&clk IMX8MP_CLK_ML_CORE>,
> <&clk IMX8MP_CLK_ML_AXI>,
> <&clk IMX8MP_CLK_ML_AHB>;
> - assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>,
> + assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_1000M>,
> <&clk IMX8MP_SYS_PLL1_800M>,
> <&clk IMX8MP_SYS_PLL1_800M>;
> - assigned-clock-rates = <800000000>,
> + assigned-clock-rates = <1000000000>,
> <800000000>,
> - <300000000>;
> + <400000000>;
> };
>
> pgc_audio: power-domain@5 {
> @@ -2232,9 +2232,9 @@ gpu3d: gpu@38000000 {
> clock-names = "core", "shader", "bus", "reg";
> assigned-clocks = <&clk IMX8MP_CLK_GPU3D_CORE>,
> <&clk IMX8MP_CLK_GPU3D_SHADER_CORE>;
> - assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>,
> - <&clk IMX8MP_SYS_PLL1_800M>;
> - assigned-clock-rates = <800000000>, <800000000>;
> + assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_1000M>,
> + <&clk IMX8MP_SYS_PLL2_1000M>;
> + assigned-clock-rates = <1000000000>, <1000000000>;
> power-domains = <&pgc_gpu3d>;
> };
>
> @@ -2247,8 +2247,8 @@ gpu2d: gpu@38008000 {
> <&clk IMX8MP_CLK_GPU_AHB>;
> clock-names = "core", "bus", "reg";
> assigned-clocks = <&clk IMX8MP_CLK_GPU2D_CORE>;
> - assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>;
> - assigned-clock-rates = <800000000>;
> + assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_1000M>;
> + assigned-clock-rates = <1000000000>;
> power-domains = <&pgc_gpu2d>;
> };
>
>
> base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b
> --
> 2.39.5
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] arm64: dts: imx8mp: configure GPU and NPU clocks to overdrive rate
2025-02-23 7:16 ` Shawn Guo
@ 2025-02-23 18:46 ` Lucas Stach
0 siblings, 0 replies; 4+ messages in thread
From: Lucas Stach @ 2025-02-23 18:46 UTC (permalink / raw)
To: Shawn Guo
Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
devicetree, imx, linux-arm-kernel, patchwork-lst
Hi Shawn,
Am Sonntag, dem 23.02.2025 um 15:16 +0800 schrieb Shawn Guo:
> On Tue, Feb 04, 2025 at 07:27:37PM +0100, Lucas Stach wrote:
> > A lot of other clocks on the i.MX8MP, including the DRAM set up by the
> > bootloader are already running at overdrive clock rates. While this is a
> > deviation from the configuration of other i.MX8M* family SoCs, overdrive
> > is the default for most i.MX8MP boards and only some special purpose
> > boards will choose to run the SoC at nominal drive rates.
>
> Are any of these special purpose boards in upstream? If so, does this
> change have any impact on them?
>
I'm pretty sure that none of the boards upstream are designed to run at
the nominal drive voltages right now. imx8mp-skov is one example of a
board that's designed to run at nominal voltages, but that is only on
the way to upstream now and Ahmad is aware of this patch and will
adjust the nominal drive DT as needed.
Regards,
Lucas
> Shawn
>
> > Up the GPU and
> > NPU clock rates to their overdrive level to be consistent with other
> > clocks set up in the dtsi.
> >
> > Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
> > ---
> > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++--------
> > 1 file changed, 8 insertions(+), 8 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> > index e0d3b8cba221..aeaa6a5c2f56 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> > @@ -816,12 +816,12 @@ pgc_mlmix: power-domain@4 {
> > assigned-clocks = <&clk IMX8MP_CLK_ML_CORE>,
> > <&clk IMX8MP_CLK_ML_AXI>,
> > <&clk IMX8MP_CLK_ML_AHB>;
> > - assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>,
> > + assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_1000M>,
> > <&clk IMX8MP_SYS_PLL1_800M>,
> > <&clk IMX8MP_SYS_PLL1_800M>;
> > - assigned-clock-rates = <800000000>,
> > + assigned-clock-rates = <1000000000>,
> > <800000000>,
> > - <300000000>;
> > + <400000000>;
> > };
> >
> > pgc_audio: power-domain@5 {
> > @@ -2232,9 +2232,9 @@ gpu3d: gpu@38000000 {
> > clock-names = "core", "shader", "bus", "reg";
> > assigned-clocks = <&clk IMX8MP_CLK_GPU3D_CORE>,
> > <&clk IMX8MP_CLK_GPU3D_SHADER_CORE>;
> > - assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>,
> > - <&clk IMX8MP_SYS_PLL1_800M>;
> > - assigned-clock-rates = <800000000>, <800000000>;
> > + assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_1000M>,
> > + <&clk IMX8MP_SYS_PLL2_1000M>;
> > + assigned-clock-rates = <1000000000>, <1000000000>;
> > power-domains = <&pgc_gpu3d>;
> > };
> >
> > @@ -2247,8 +2247,8 @@ gpu2d: gpu@38008000 {
> > <&clk IMX8MP_CLK_GPU_AHB>;
> > clock-names = "core", "bus", "reg";
> > assigned-clocks = <&clk IMX8MP_CLK_GPU2D_CORE>;
> > - assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_800M>;
> > - assigned-clock-rates = <800000000>;
> > + assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_1000M>;
> > + assigned-clock-rates = <1000000000>;
> > power-domains = <&pgc_gpu2d>;
> > };
> >
> >
> > base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b
> > --
> > 2.39.5
> >
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] arm64: dts: imx8mp: configure GPU and NPU clocks to overdrive rate
2025-02-04 18:27 [PATCH] arm64: dts: imx8mp: configure GPU and NPU clocks to overdrive rate Lucas Stach
2025-02-23 7:16 ` Shawn Guo
@ 2025-03-11 1:27 ` Shawn Guo
1 sibling, 0 replies; 4+ messages in thread
From: Shawn Guo @ 2025-03-11 1:27 UTC (permalink / raw)
To: Lucas Stach
Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
devicetree, imx, linux-arm-kernel, patchwork-lst
On Tue, Feb 04, 2025 at 07:27:37PM +0100, Lucas Stach wrote:
> A lot of other clocks on the i.MX8MP, including the DRAM set up by the
> bootloader are already running at overdrive clock rates. While this is a
> deviation from the configuration of other i.MX8M* family SoCs, overdrive
> is the default for most i.MX8MP boards and only some special purpose
> boards will choose to run the SoC at nominal drive rates. Up the GPU and
> NPU clock rates to their overdrive level to be consistent with other
> clocks set up in the dtsi.
>
> Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Applied, thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-03-11 1:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-04 18:27 [PATCH] arm64: dts: imx8mp: configure GPU and NPU clocks to overdrive rate Lucas Stach
2025-02-23 7:16 ` Shawn Guo
2025-02-23 18:46 ` Lucas Stach
2025-03-11 1:27 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).