FIX Re: GPMI IMX6ull timeout on dma

[Miquel Raynal]

Hi Michael,

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

> Hi
> 
> On Wed, Jan 27, 2021 at 8:12 PM Miquel Raynal <miquel.raynal at bootlin.com> wrote:
> >
> > 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.
> >  
> 
> Try to reproduce on a 4.19.y version stable branch.
> There was some difference on register values I have seen last time, did you
> get time to check them?

If you now have time to investigate the differences it would be great.
I can perhaps help you to try understand the meaning of the differences
if you give me precise information.