Re: [PATCH 1/1] mtd: rawnand: davinci: Add dummy read after sending command

[Miquel Raynal]

On Fri, 2024-03-08 at 07:46:09 UTC, Bastien Curutchet wrote:
> Sometimes, writes fail because the tWB_max is not correctly observed
> after sending PAGEPROG. It leads to the R/B pin to be read as in
> the "ready" state right after sending the command, thus preventing the
> normal tPROG delay to be actually observed. This happens because the
> ndelay() that waits for tWB_max starts before the command reaches the
> NAND chip.
> 
> Add a dummy read when a delay is requested at the end of the executed
> instruction to make sure that the sent command is received by the NAND
> before starting the short ndelay() (<1us but rounded up to 1us in
> practice). This read is done on the control register area because
> doing it on the Async Data area would change the NAND's RE pin state.
> This is not perfect as the two areas are behind two different
> devm_ioremap_resource() and could possibly be located on different
> interconnects (I did not find more details). This means either the
> additional latency due to the load operation is enough impacting, or it
> has the expected behavior of ensuring the write has been received.
> 
> This has been tested on two platforms designed off of the
> DAVINCI/OMAP-L138. The first uses a Toshiba NAND Flash (TC58NYG2S3EBAI5),
> the other a Macronix one (MX30UF4G18AC).
> 
> Signed-off-by: Bastien Curutchet <bastien.curutchet@xxxxxxxxxxx>

Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.

Miquel