Re: [PATCH v2] riscv: dts: starfive: Set EMMC vqmmc maximum voltage to 3.3V on JH7110 boards

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Conor,

From star64's schematic, I didn't see any difference between star64 and
vf2's emmc vqmmc source design and aldo4 is not shared with other
peripheral at all. And this patch is tested on my vf2 with no problem at
all. So I think it is supported.

Best regards,
Shengyu

在 2024/6/13 0:18, Conor Dooley 写道:
On Wed, Jun 12, 2024 at 06:33:31PM +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,

I should've asked last time around, does the star64 also support 3.3
volts?

regulator-max-microvolt of ALDO4 should be set to 3.3V.

Cc: stable@xxxxxxxxxxxxxxx
Signed-off-by: Shengyu Qu <wiagn233@xxxxxxxxxxx>
Fixes: 7dafcfa79cc9 ("riscv: dts: starfive: enable DCDC1&ALDO4 node in axp15060")
---
  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.39.2

Attachment: OpenPGP_0xE3520CC91929C8E7.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux