| Message ID | 20210930155633.2745201-9-frieder@fris.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Devicetree fixes and additions for Kontron SL/BL i.MX8MM boards | expand |
Hi Frieder, Am Do., 30. Sept. 2021 um 17:57 Uhr schrieb Frieder Schrempf <frieder@fris.de>: > > From: Frieder Schrempf <frieder.schrempf@kontron.de> > > When the cpufreq driver is enabled, the buck2 regulator is kept powered on > by the dependency between the CPU nodes with 'cpu-supply' set. Without the > cpufreq driver the kernel will power off the regulator as it doesn't see > any users. This is obviously not what we want, therefore keep the regulator > powered on in any case. > > Reported-by: Heiko Thiery <heiko.thiery@gmail.com> > Fixes: 21c4f45b335f ("arm64: dts: Add the Kontron i.MX8M Mini SoMs and baseboards") > Cc: stable@vger.kernel.org > Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de> Tested-by: Heiko Thiery <heiko.thiery@gmail.com> > --- > arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi > index 213014f59b46..c3418d263eb4 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi > @@ -105,6 +105,7 @@ reg_vdd_arm: BUCK2 { > regulator-min-microvolt = <850000>; > regulator-max-microvolt = <950000>; > regulator-boot-on; > + regulator-always-on; > regulator-ramp-delay = <3125>; > nxp,dvs-run-voltage = <950000>; > nxp,dvs-standby-voltage = <850000>; > -- > 2.33.0 >
On Thu, Sep 30, 2021 at 05:56:31PM +0200, Frieder Schrempf wrote: > From: Frieder Schrempf <frieder.schrempf@kontron.de> > > When the cpufreq driver is enabled, the buck2 regulator is kept powered on > by the dependency between the CPU nodes with 'cpu-supply' set. Without the > cpufreq driver the kernel will power off the regulator as it doesn't see > any users. This is obviously not what we want, therefore keep the regulator > powered on in any case. > > Reported-by: Heiko Thiery <heiko.thiery@gmail.com> > Fixes: 21c4f45b335f ("arm64: dts: Add the Kontron i.MX8M Mini SoMs and baseboards") > Cc: stable@vger.kernel.org > Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de> Already picked up the one from Heiko. https://lore.kernel.org/all/20210915120325.20248-1-heiko.thiery@gmail.com/ Shawn > --- > arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi > index 213014f59b46..c3418d263eb4 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi > @@ -105,6 +105,7 @@ reg_vdd_arm: BUCK2 { > regulator-min-microvolt = <850000>; > regulator-max-microvolt = <950000>; > regulator-boot-on; > + regulator-always-on; > regulator-ramp-delay = <3125>; > nxp,dvs-run-voltage = <950000>; > nxp,dvs-standby-voltage = <850000>; > -- > 2.33.0 >
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi index 213014f59b46..c3418d263eb4 100644 --- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi @@ -105,6 +105,7 @@ reg_vdd_arm: BUCK2 { regulator-min-microvolt = <850000>; regulator-max-microvolt = <950000>; regulator-boot-on; + regulator-always-on; regulator-ramp-delay = <3125>; nxp,dvs-run-voltage = <950000>; nxp,dvs-standby-voltage = <850000>;