| Message ID | TY3P286MB2611936BD43C24D34B442E6D98C72@TY3P286MB2611.JPNP286.PROD.OUTLOOK.COM (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | riscv: dts: starfive: Set EMMC vqmmc maximum voltage to 3.3V on JH7110 boards | expand |
On Tue, Jun 11, 2024 at 10:56:41PM +0800, Shengyu Qu wrote: > Currently, for JH7110 boards with EMMC slot, vqmmc voltage for EMMC is > fixed to 1.8V, while the spec needs it to be 3.3V on low speed mode and > should support switching to 1.8V when using higher speed mode. Since > there are no other peripherals using the same voltage source of EMMC's > vqmmc(ALDO4) on every board currently supported by mainline kernel, > regulator-max-microvolt of ALDO4 should be set to 3.3V. > > Cc: stable@vger.kernel.org > Signed-off-by: Shengyu Qu <wiagn233@outlook.com> > Fixes: ac9a37e2d6b6 ("riscv: dts: starfive: introduce a common board dtsi for jh7110 based boards") I don't think this fixes tag is correct, it just moved in that commit. It has been there since commit 7dafcfa79cc9 ("riscv: dts: starfive: enable DCDC1&ALDO4 node in axp15060"). Thanks, Conor. > --- > arch/riscv/boot/dts/starfive/jh7110-common.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi > index 37b4c294ffcc..c7a549ec7452 100644 > --- a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi > +++ b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi > @@ -244,7 +244,7 @@ emmc_vdd: aldo4 { > regulator-boot-on; > regulator-always-on; > regulator-min-microvolt = <1800000>; > - regulator-max-microvolt = <1800000>; > + regulator-max-microvolt = <3300000>; > regulator-name = "emmc_vdd"; > }; > }; > -- > 2.34.1 >
diff --git a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi index 37b4c294ffcc..c7a549ec7452 100644 --- a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi +++ b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi @@ -244,7 +244,7 @@ emmc_vdd: aldo4 { regulator-boot-on; regulator-always-on; regulator-min-microvolt = <1800000>; - regulator-max-microvolt = <1800000>; + regulator-max-microvolt = <3300000>; regulator-name = "emmc_vdd"; }; };
Currently, for JH7110 boards with EMMC slot, vqmmc voltage for EMMC is fixed to 1.8V, while the spec needs it to be 3.3V on low speed mode and should support switching to 1.8V when using higher speed mode. Since there are no other peripherals using the same voltage source of EMMC's vqmmc(ALDO4) on every board currently supported by mainline kernel, regulator-max-microvolt of ALDO4 should be set to 3.3V. Cc: stable@vger.kernel.org Signed-off-by: Shengyu Qu <wiagn233@outlook.com> Fixes: ac9a37e2d6b6 ("riscv: dts: starfive: introduce a common board dtsi for jh7110 based boards") --- arch/riscv/boot/dts/starfive/jh7110-common.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)