Re: [PATCH v6 13/14] mtd: onenand: omap2: Configure driver from DT

Tony Lindgren
2018-12-23T15:56:49Z

* Aaro Koskinen <aaro.koskinen@iki.fi> [181223 12:52]:
> Hi,
>
> On Sat, Dec 22, 2018 at 11:19:38AM +0100, Ladislav Michl wrote:
> > Well, DT timings for onenand node seems to be copied over without actually
> > verifying correctness. I do not know which chip comes with N950, so please
> > verify timings in you DT according chip used. Idea is to use timings for
> > slowest chip used and driver will ask for faster one if possible.
>
> Hmm, right. I thought we had some dumped values in the DT, but apparently
> wrong ones. What is the method for dumping the "right" values? If I just
> enable GPMC debug with old kernel, it's dumping the DT entries twice,
> but neither set works with the new kernel.
>
> Then I tried the "before" values from the new kernel, and they seem to
> work (patch below)...

You can enable CONFIG_OMAP_GPMC_DEBUG=y and the gpmc driver should print
out the bootloader timings for you on boot to make sure they're what you
posted. Then if the values are different between devices, then it makes
sense to add them to the device specific dts files.

Regards,

Tony