Hi Marco,

On 10.12.19 10:09, Marco Felsch wrote:

Hi Stefan,

On 19-12-05 13:00, Stefan Riedmüller wrote:

quoted

Hi Marco,

On 29.11.19 17:48, Marco Felsch wrote:

quoted

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.

quoted

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

quoted

Regards,
Stefan

quoted

Fixes: ddec5d1c0047 ("ARM: dts: imx6: Add initial support for phyCORE-i.MX 6 SOM")
Signed-off-by: Marco Felsch <redacted>
---
   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";
   };

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel