mtd: rawnand: Ensure continuous reads are well disabled
authorMiquel Raynal <miquel.raynal@bootlin.com>
Thu, 7 Mar 2024 11:53:15 +0000 (12:53 +0100)
committerMiquel Raynal <miquel.raynal@bootlin.com>
Thu, 14 Mar 2024 23:04:51 +0000 (00:04 +0100)
commit4120aa0e3961f68f1f8cfe6b4c3c809ffea31fdc
tree2fa58d86bdfd307cdb4abc69cc923a30298710b1
parent78ffbefba8d7822b232585570b293de5bc397da6
mtd: rawnand: Ensure continuous reads are well disabled

The cont_read.ongoing flag should only be enabled at the beginning of a
read operation, and also disabled at its end, so we never end up
triggering nasty side effects outside of this scope. The mtd core being
highly serialized, we should not be bothered by parallel accesses
anyway.

In case we reach the end of a read operation and the boolean was not
properly disabled, it's a bug, but it's totally manageable. So warn, and
then fix the boolean state.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20240307115315.1942678-2-miquel.raynal@bootlin.com
drivers/mtd/nand/raw/nand_base.c