GPMI iMX6ull timeout on DMA

[Christian Eggers]

On Friday, 8 October 2021, 15:36:31 CEST, Miquel Raynal wrote:
> 
> miquel.raynal at bootlin.com wrote on Fri, 8 Oct 2021 15:29:05 +0200:
> 
> > 
> > If this clock (as I understand) does not prevent us to access the
> > registers but only feeds the external NAND bus part, then there is no
> > need to enable it in the probe, just acquiring it will be enough.

clocks[0] is "gpmi_io" which sounds more like i/o than registers. So lets
try to remove the initial call to clk_set_rate().

> > Then, the first call for an IO operation with ->must_apply_timings
> > should:
> > 
> > 	if (imx6)
> > 		disable_clk();
> > 
> > 	clk_set_rate();
> > 
> > 	if (imx6)
> > 		enable_clk();

Do you think that the need for avoiding clock glitches is i.MX6 specific?
The errata I mentioned is specific for the bootloader software, but (I think)
the requirement for switching off the clocks gates prior changing the dividers
may apply also for other series.

> Actually we should ensure clks are enabled in the !imx6 case anyway,
> but this is needed only once so either we keep enabling the clock in
> the probe or we check here if the clk has already been enabled or not.
The clocks are already enabled (and kept on) in probe. The initial call to
clk_set_rate() is just above this (but the clocks are not disabled at this
stage as all gates have been enabled by the boot loader).

regards
Christian