[PATCH] ARM: dts: imx8mq-kontron-pitx-imx8m: Remove off-on-delay-us property

[Heiko Thiery]

Hi Michael,

Am Di., 25. Okt. 2022 um 08:46 Uhr schrieb Michael Nazzareno Trimarchi
<michael at amarulasolutions.com>:
>
> Hi Heiko
>
> On Tue, Oct 25, 2022 at 8:43 AM Heiko Thiery <heiko.thiery at gmail.com> wrote:
> >
> > Hi Peng,
> >
> > Am Di., 25. Okt. 2022 um 07:18 Uhr schrieb Peng Fan <peng.fan at oss.nxp.com>:
> > >
> > >
> > >
> > > On 10/24/2022 7:25 PM, Heiko Thiery wrote:
> > > > U-Boot is not able to store the environment variables in the SD card.
> > > >
> > > > Remove the off-on-delay-us property to fix the regression.
> > >
> > > off-on-delay-us here would cause store env failure?
> > > Would you please share more info?
> >
> > I saw that the environment could not be loaded/stored since the
> > synchroinzation of the device tree from linux 181d1684cae07f111
> > (imx8mq: synchronise device tree with linux) and then without doing a
> > bisect I found this commit 5d42ea553534 (ARM: dts: imx8mq-evk: Remove
> > u-boot,off-on-delay-us property). After removing the delay
> > loading/storing worked again.
> >
>
> What is not clear apart the environment store if the sdcard is working or not

Ok, Accessing the SD card at all is not working. I only noticed it
first because the environment could not be loaded.

-- 
Heiko
>
> Michael
>
> > --
> > Heiko
> >
> >
> >
> > >
> > > Thanks,
> > > Peng.
> > >
> > > >
> > > > Signed-off-by: Heiko Thiery <heiko.thiery at gmail.com>
> > > > ---
> > > > The same change is done in the linux dts file.
> > > >
> > > >   arch/arm/dts/imx8mq-kontron-pitx-imx8m.dts | 1 -
> > > >   1 file changed, 1 deletion(-)
> > > >
> > > > diff --git a/arch/arm/dts/imx8mq-kontron-pitx-imx8m.dts b/arch/arm/dts/imx8mq-kontron-pitx-imx8m.dts
> > > > index a91c136797..21442e04a6 100644
> > > > --- a/arch/arm/dts/imx8mq-kontron-pitx-imx8m.dts
> > > > +++ b/arch/arm/dts/imx8mq-kontron-pitx-imx8m.dts
> > > > @@ -51,7 +51,6 @@
> > > >               regulator-min-microvolt = <3300000>;
> > > >               regulator-max-microvolt = <3300000>;
> > > >               gpio = <&gpio2 19 GPIO_ACTIVE_HIGH>;
> > > > -             off-on-delay-us = <20000>;
> > > >               enable-active-high;
> > > >       };
> > > >   };
>
>
>
> --
> Michael Nazzareno Trimarchi
> Co-Founder & Chief Executive Officer
> M. +39 347 913 2170
> michael at amarulasolutions.com
> __________________________________
>
> Amarula Solutions BV
> Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> T. +31 (0)85 111 9172
> info at amarulasolutions.com
> www.amarulasolutions.com