FIX Re: GPMI IMX6ull timeout on dma

[Miquel Raynal]

Hi Michael,

Michael Nazzareno Trimarchi <michael at amarulasolutions.com> wrote on
Wed, 27 Jan 2021 18:01:01 +0100:

> Hi all
> 
> On Mon, Oct 29, 2018 at 10:53 AM Miquel Raynal
> <miquel.raynal at bootlin.com> wrote:
> >
> > Hi Michael,
> >
> > Michael Nazzareno Trimarchi <michael at amarulasolutions.com> wrote on
> > Mon, 29 Oct 2018 10:43:05 +0100:
> >  
> > > Hi
> > >
> > > On Mon, Oct 29, 2018 at 10:41 AM Miquel Raynal
> > > <miquel.raynal at bootlin.com> wrote:  
> > > >
> > > > Hi Michael,
> > > >
> > > > Michael Nazzareno Trimarchi <michael at amarulasolutions.com> wrote on
> > > > Fri, 5 Oct 2018 12:32:18 +0200:
> > > >  
> > > > > Hi Boris
> > > > >
> > > > > I have reverted for now
> > > > >
> > > > > Revert "mtd: rawnand: gpmi: support ->setup_data_interface()"
> > > > > This reverts commit 76e1a0086a0c3276b384f77905345e0fcc886fdd.
> > > > >
> > > > > Revert "mtd: rawnand: gpmi: use core timings instead of an empirical
> > > > > derivation"
> > > > > This reverts commit b1206122069aadabe1a8c50789277a978aaa4df7.  
> > > >
> > > > Sorry for the delay, I've been very busy lately.
> > > >
> > > > Please also Cc: me in your e-mails to the MTD ML.
> > > >
> > > > AFAIR I'm the author of these patches so I would like to understand and
> > > > fix what's wrong there. As a start, can you discriminate which patch
> > > > actually makes the timings inconsistent? Are you ready to do more
> > > > testing to find what's wrong?  
> > >
> > > Yes it's on my plan. Now for field I need to revert them but I have
> > > them in roadmap.
> > > We are testing now an imx6dl. It can be a combination with the micron
> > > part number too.  
> >
> > Sure, let us know about your discoveries, I really want the GPMI driver
> > to use core timings and ->setup_data_interface() so if there is
> > something wrong there it should be fixed.
> >  
> 
> Need to move on lts kernel and  I have time to work and I must work on
> this topic. I don't see any update but
> another report. Is this supposed to be fixed?

Not at all, this hasn't moved forward.

Thanks,
Miquèl