arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq

Message ID 20230606153238.1448463-1-tharvey@gateworks.com
State New
Headers
Series arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq |

Commit Message

Tim Harvey June 6, 2023, 3:32 p.m. UTC
  Add regulator config for cpu-supply in order to support cpufreq.

Signed-off-by: Tim Harvey <tharvey@gateworks.com>
---
 .../dts/freescale/imx8mm-venice-gw7901.dts     | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)
  

Comments

Tim Harvey July 12, 2023, 4:13 p.m. UTC | #1
On Tue, Jun 6, 2023 at 8:32 AM Tim Harvey <tharvey@gateworks.com> wrote:
>
> Add regulator config for cpu-supply in order to support cpufreq.
>
> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> ---
>  .../dts/freescale/imx8mm-venice-gw7901.dts     | 18 +++++++++++++++++-
>  1 file changed, 17 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> index df3b2c93d2d5..d022b5807a24 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> @@ -242,6 +242,22 @@ reg_wifi: regulator-wifi {
>         };
>  };
>
> +&A53_0 {
> +       cpu-supply = <&buck2>;
> +};
> +
> +&A53_1 {
> +       cpu-supply = <&buck2>;
> +};
> +
> +&A53_2 {
> +       cpu-supply = <&buck2>;
> +};
> +
> +&A53_3 {
> +       cpu-supply = <&buck2>;
> +};
> +
>  &ddrc {
>         operating-points-v2 = <&ddrc_opp_table>;
>
> @@ -511,7 +527,7 @@ BUCK1 {
>                         };
>
>                         /* vdd_arm: 0.805-1.0V (typ=0.9V) */
> -                       BUCK2 {
> +                       buck2: BUCK2 {
>                                 regulator-name = "buck2";
>                                 regulator-min-microvolt = <700000>;
>                                 regulator-max-microvolt = <1300000>;
> --
> 2.25.1
>

Hi Shawn,

I noticed this and several other of my pending patches delegated to
you got marked as 'archived' and no longer appear as new in my
patchwork queue [1]. I wanted to make sure they are still in your
queue or see if I need to resubmit them. I'm not sure what the process
is that you follow and if they got archived by you or something else.
Please let me know if I need to do anything:

arm64: dts: imx8mm: add imx8mm-venice-gw73xx-0x-rpidsi overlay for display
arm64: dts: freescale: fix imx8mm-venice-gw72xx-0x-imx219 overlay
arm64: dts: freescale: Introduce imx8mp-venice-gw73xx-2x
dt-bindings: arm: Add Gateworks i.MX8M GW73xx-2x board
arm64: dts: freescale: Introduce imx8mp-venice-gw72xx-2x
dt-bindings: arm: Add Gateworks i.MX8M GW72xx-2x board
arm64: dts: freescale: Introduce imx8mp-venice-gw71xx-2x
dt-bindings: arm: Add Gateworks i.MX8M GW71xx-2x board
arm64: dts: freescale: Introduce imx8mm-venice-gw7905-0x
dt-bindings: arm: Add Gateworks i.MX8M GW7905-0x board
arm64: dts: imx8mm-venice-gw7904: disable disp_blk_ctrl
arm64: dts: imx8mm-venice-gw7903: disable disp_blk_ctrl
arm64: dts: imx8mm-venice-gw7904: enable UART1 hardware flow control
arm64: dts: imx8m{m,n}-venice-gw7902: add SDR50/SDR104 SDIO support for wifi
arm64: dts: imx8mm-venice-gw7901: add SDR50/SDR104 SDIO support for wifi
arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq
arm64: dts: imx8mp-venice-gw74xx: update to revB PCB

best regards,

Tim
[1] https://patchwork.kernel.org/project/linux-arm-kernel/list/?submitter=75211
  
Peng Fan (OSS) July 13, 2023, 7:25 a.m. UTC | #2
On 6/6/2023 11:32 PM, Tim Harvey wrote:
> Add regulator config for cpu-supply in order to support cpufreq.
> 
> Signed-off-by: Tim Harvey<tharvey@gateworks.com>

LGTM:
Reviewed-by: Peng Fan <peng.fan@nxp.com>
  
Shawn Guo July 17, 2023, 12:06 a.m. UTC | #3
On Tue, Jun 06, 2023 at 08:32:38AM -0700, Tim Harvey wrote:
> Add regulator config for cpu-supply in order to support cpufreq.
> 
> Signed-off-by: Tim Harvey <tharvey@gateworks.com>

Applied, thanks!
  
Shawn Guo July 17, 2023, 12:09 a.m. UTC | #4
On Wed, Jul 12, 2023 at 09:13:49AM -0700, Tim Harvey wrote:
> On Tue, Jun 6, 2023 at 8:32 AM Tim Harvey <tharvey@gateworks.com> wrote:
> >
> > Add regulator config for cpu-supply in order to support cpufreq.
> >
> > Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> > ---
> >  .../dts/freescale/imx8mm-venice-gw7901.dts     | 18 +++++++++++++++++-
> >  1 file changed, 17 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> > index df3b2c93d2d5..d022b5807a24 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> > @@ -242,6 +242,22 @@ reg_wifi: regulator-wifi {
> >         };
> >  };
> >
> > +&A53_0 {
> > +       cpu-supply = <&buck2>;
> > +};
> > +
> > +&A53_1 {
> > +       cpu-supply = <&buck2>;
> > +};
> > +
> > +&A53_2 {
> > +       cpu-supply = <&buck2>;
> > +};
> > +
> > +&A53_3 {
> > +       cpu-supply = <&buck2>;
> > +};
> > +
> >  &ddrc {
> >         operating-points-v2 = <&ddrc_opp_table>;
> >
> > @@ -511,7 +527,7 @@ BUCK1 {
> >                         };
> >
> >                         /* vdd_arm: 0.805-1.0V (typ=0.9V) */
> > -                       BUCK2 {
> > +                       buck2: BUCK2 {
> >                                 regulator-name = "buck2";
> >                                 regulator-min-microvolt = <700000>;
> >                                 regulator-max-microvolt = <1300000>;
> > --
> > 2.25.1
> >
> 
> Hi Shawn,
> 
> I noticed this and several other of my pending patches delegated to
> you got marked as 'archived' and no longer appear as new in my
> patchwork queue [1]. I wanted to make sure they are still in your
> queue or see if I need to resubmit them. I'm not sure what the process
> is that you follow and if they got archived by you or something else.
> Please let me know if I need to do anything:

I do not use patchwork, so it doesn't reflect any status on my side.

Shawn
  

Patch

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
index df3b2c93d2d5..d022b5807a24 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
@@ -242,6 +242,22 @@  reg_wifi: regulator-wifi {
 	};
 };
 
+&A53_0 {
+	cpu-supply = <&buck2>;
+};
+
+&A53_1 {
+	cpu-supply = <&buck2>;
+};
+
+&A53_2 {
+	cpu-supply = <&buck2>;
+};
+
+&A53_3 {
+	cpu-supply = <&buck2>;
+};
+
 &ddrc {
 	operating-points-v2 = <&ddrc_opp_table>;
 
@@ -511,7 +527,7 @@  BUCK1 {
 			};
 
 			/* vdd_arm: 0.805-1.0V (typ=0.9V) */
-			BUCK2 {
+			buck2: BUCK2 {
 				regulator-name = "buck2";
 				regulator-min-microvolt = <700000>;
 				regulator-max-microvolt = <1300000>;