* [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq
@ 2023-06-06 15:32 Tim Harvey
2023-07-12 16:13 ` Tim Harvey
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Tim Harvey @ 2023-06-06 15:32 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Li Yang,
devicetree, linux-kernel, Tim Harvey
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq
2023-06-06 15:32 [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq Tim Harvey
@ 2023-07-12 16:13 ` Tim Harvey
2023-07-17 0:09 ` Shawn Guo
2023-07-13 7:25 ` Peng Fan
2023-07-17 0:06 ` Shawn Guo
2 siblings, 1 reply; 5+ messages in thread
From: Tim Harvey @ 2023-07-12 16:13 UTC (permalink / raw)
To: Shawn Guo
Cc: Rob Herring, Krzysztof Kozlowski, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Li Yang,
devicetree, linux-kernel, linux-arm-kernel
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq
2023-06-06 15:32 [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq Tim Harvey
2023-07-12 16:13 ` Tim Harvey
@ 2023-07-13 7:25 ` Peng Fan
2023-07-17 0:06 ` Shawn Guo
2 siblings, 0 replies; 5+ messages in thread
From: Peng Fan @ 2023-07-13 7:25 UTC (permalink / raw)
To: Tim Harvey, linux-arm-kernel
Cc: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Li Yang,
devicetree, linux-kernel
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>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq
2023-06-06 15:32 [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq Tim Harvey
2023-07-12 16:13 ` Tim Harvey
2023-07-13 7:25 ` Peng Fan
@ 2023-07-17 0:06 ` Shawn Guo
2 siblings, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2023-07-17 0:06 UTC (permalink / raw)
To: Tim Harvey
Cc: linux-arm-kernel, Rob Herring, Krzysztof Kozlowski, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Li Yang,
devicetree, linux-kernel
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!
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq
2023-07-12 16:13 ` Tim Harvey
@ 2023-07-17 0:09 ` Shawn Guo
0 siblings, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2023-07-17 0:09 UTC (permalink / raw)
To: Tim Harvey
Cc: Rob Herring, Krzysztof Kozlowski, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Li Yang,
devicetree, linux-kernel, linux-arm-kernel
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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-07-17 0:09 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-06 15:32 [PATCH] arm64: dts: imx8mm-venice-gw7901: add cpu-supply node for cpufreq Tim Harvey
2023-07-12 16:13 ` Tim Harvey
2023-07-17 0:09 ` Shawn Guo
2023-07-13 7:25 ` Peng Fan
2023-07-17 0:06 ` 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).