2023-06-06 15:47:15

by Tim Harvey

[permalink] [raw]
Subject: [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq

Add regulator config for cpu-supply in order to support cpufreq.

Signed-off-by: Tim Harvey <[email protected]>
---
.../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



2023-07-12 16:24:35

by Tim Harvey

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq

On Tue, Jun 6, 2023 at 8:32 AM Tim Harvey <[email protected]> wrote:
>
> Add regulator config for cpu-supply in order to support cpufreq.
>
> Signed-off-by: Tim Harvey <[email protected]>
> ---
> .../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

2023-07-13 07:43:00

by Peng Fan (OSS)

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq



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<[email protected]>

LGTM:
Reviewed-by: Peng Fan <[email protected]>

2023-07-17 00:24:22

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq

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 <[email protected]>

Applied, thanks!

2023-07-17 01:04:35

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq

On Wed, Jul 12, 2023 at 09:13:49AM -0700, Tim Harvey wrote:
> On Tue, Jun 6, 2023 at 8:32 AM Tim Harvey <[email protected]> wrote:
> >
> > Add regulator config for cpu-supply in order to support cpufreq.
> >
> > Signed-off-by: Tim Harvey <[email protected]>
> > ---
> > .../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