Message ID | 20191129164859.15632-2-m.felsch@pengutronix.de (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [1/3] ARM: dts: imx6: phycore-som: fix arm and soc minimum voltage | expand |
Hi Marco, On 29.11.19 17:48, Marco Felsch wrote: > Currently the vmmc is supplied by the 1.8V pmic rail but this is wrong. > The 1.8V pmic rail is connected to the emmc vccq (vqmmc). I just checked the schematics again and actually both VCC and VCCQ are connected to the 3.3V power rail. VCCQ can be connected to the 1.8V PMIC power rail by solder jumper but default is 3.3V. So I think either both should be connected to a fixed 3.3V regulator or removed, since the default system does not support switching these voltages. Regards, Stefan > > Fixes: ddec5d1c0047 ("ARM: dts: imx6: Add initial support for phyCORE-i.MX 6 SOM") > Signed-off-by: Marco Felsch <m.felsch@pengutronix.de> > --- > arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi > index 46d4953c5588..44e333848b4d 100644 > --- a/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi > @@ -183,7 +183,7 @@ > pinctrl-0 = <&pinctrl_usdhc4>; > bus-width = <8>; > non-removable; > - vmmc-supply = <&vdd_emmc_1p8>; > + vqmmc-supply = <&vdd_emmc_1p8>; > status = "disabled"; > }; > >
Hi Stefan, On 19-12-05 13:00, Stefan Riedmüller wrote: > Hi Marco, > > On 29.11.19 17:48, Marco Felsch wrote: > > Currently the vmmc is supplied by the 1.8V pmic rail but this is wrong. > > The 1.8V pmic rail is connected to the emmc vccq (vqmmc). > > I just checked the schematics again and actually both VCC and VCCQ are > connected to the 3.3V power rail. VCCQ can be connected to the 1.8V PMIC > power rail by solder jumper but default is 3.3V. That's right. > So I think either both should be connected to a fixed 3.3V regulator or > removed, since the default system does not support switching these voltages. The question is why the regulator output isn't the default case and setting regulator-max-microvolt to 3.3V and regulator-min-microvolt to 1.8V? The PMIC LDO4 supports output voltages up to 3.6V so this should be okay. Anyway dropping both regulators and using dummy-regulators should be fine too. I only want to remove this issue. Regards, Marco > Regards, > Stefan > > > > > Fixes: ddec5d1c0047 ("ARM: dts: imx6: Add initial support for phyCORE-i.MX 6 SOM") > > Signed-off-by: Marco Felsch <m.felsch@pengutronix.de> > > --- > > arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi > > index 46d4953c5588..44e333848b4d 100644 > > --- a/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi > > +++ b/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi > > @@ -183,7 +183,7 @@ > > pinctrl-0 = <&pinctrl_usdhc4>; > > bus-width = <8>; > > non-removable; > > - vmmc-supply = <&vdd_emmc_1p8>; > > + vqmmc-supply = <&vdd_emmc_1p8>; > > status = "disabled"; > > }; > > >
Hi Marco, On 10.12.19 10:09, Marco Felsch wrote: > Hi Stefan, > > On 19-12-05 13:00, Stefan Riedmüller wrote: >> Hi Marco, >> >> On 29.11.19 17:48, Marco Felsch wrote: >>> Currently the vmmc is supplied by the 1.8V pmic rail but this is wrong. >>> The 1.8V pmic rail is connected to the emmc vccq (vqmmc). >> >> I just checked the schematics again and actually both VCC and VCCQ are >> connected to the 3.3V power rail. VCCQ can be connected to the 1.8V PMIC >> power rail by solder jumper but default is 3.3V. > > That's right. > >> So I think either both should be connected to a fixed 3.3V regulator or >> removed, since the default system does not support switching these voltages. > > The question is why the regulator output isn't the default case and > setting regulator-max-microvolt to 3.3V and regulator-min-microvolt to > 1.8V? The PMIC LDO4 supports output voltages up to 3.6V so this should > be okay. That is a reasonable question. But since we can't change this here we need to take it as it is. So it is probably best to just drop both regulators. Regards, Stefan > > Anyway dropping both regulators and using dummy-regulators should be > fine too. I only want to remove this issue. > > Regards, > Marco > >> Regards, >> Stefan >> >>> >>> Fixes: ddec5d1c0047 ("ARM: dts: imx6: Add initial support for phyCORE-i.MX 6 SOM") >>> Signed-off-by: Marco Felsch <m.felsch@pengutronix.de> >>> --- >>> arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi >>> index 46d4953c5588..44e333848b4d 100644 >>> --- a/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi >>> +++ b/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi >>> @@ -183,7 +183,7 @@ >>> pinctrl-0 = <&pinctrl_usdhc4>; >>> bus-width = <8>; >>> non-removable; >>> - vmmc-supply = <&vdd_emmc_1p8>; >>> + vqmmc-supply = <&vdd_emmc_1p8>; >>> status = "disabled"; >>> }; >>> >> >
diff --git a/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi index 46d4953c5588..44e333848b4d 100644 --- a/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi +++ b/arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi @@ -183,7 +183,7 @@ pinctrl-0 = <&pinctrl_usdhc4>; bus-width = <8>; non-removable; - vmmc-supply = <&vdd_emmc_1p8>; + vqmmc-supply = <&vdd_emmc_1p8>; status = "disabled"; };
Currently the vmmc is supplied by the 1.8V pmic rail but this is wrong. The 1.8V pmic rail is connected to the emmc vccq (vqmmc). Fixes: ddec5d1c0047 ("ARM: dts: imx6: Add initial support for phyCORE-i.MX 6 SOM") Signed-off-by: Marco Felsch <m.felsch@pengutronix.de> --- arch/arm/boot/dts/imx6qdl-phytec-phycore-som.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)