On Tue, Aug 18, 2020 at 9:33 PM Fabio Estevam <feste...@gmail.com> wrote:
>
> Hi Adam,
>
> On Tue, Aug 18, 2020 at 6:20 PM Adam Ford <aford...@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 <aford...@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
