On Tue, Aug 18, 2020 at 9:33 PM Fabio Estevam festevam@gmail.com wrote:
Hi Adam,
On Tue, Aug 18, 2020 at 6:20 PM Adam Ford aford173@gmail.com wrote:
The PMIC which powers the whole board is connected to the I2C1 bus. This patch enables I2C. With this enabled, the PMIC needs to told to keep the rail enabled, or it gets shutdown before the
Looks like a typo.
Oops.
system boots.
Signed-off-by: Adam Ford aford173@gmail.com
diff --git a/arch/arm/dts/imx6q-logicpd-u-boot.dtsi b/arch/arm/dts/imx6q-logicpd-u-boot.dtsi index ee44ed91fe..94b53d569d 100644 --- a/arch/arm/dts/imx6q-logicpd-u-boot.dtsi +++ b/arch/arm/dts/imx6q-logicpd-u-boot.dtsi @@ -28,3 +28,7 @@ &pinctrl_usdhc2 { u-boot,dm-spl; };
+&sw2_reg {
regulator-always-on;+};
It seems that this could be fixed in the main dts instead of the U-Boot one since this dts description is not U-Boot specific.
The issue I am seeing is U-Boot specific, because Linux doesn't power this rail down. I'd rather not keep this regulator-always-on in Linux, and I want to keep the devices trees between U-Boot and Linux the same. In the device tree, I have associated this rail to the SDHC interface , but for some reason, U-Boot still powers it down, so when it tries to go and read from the SD card, the system hangs.
If you're ok with this rationale, I can update the commit message to add some of this background info to V2 when I fix the typo above.
adam