[PATCH] arm: imx6q_logic: Enable I2C PMIC

[Adam Ford]

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