Commit 1dc60b00 authored by Miquel Raynal's avatar Miquel Raynal Committed by Zheng Zengkai
Browse files

mtd: rawnand: arasan: Prevent an unsupported configuration 

stable inclusion
from stable-v5.10.137
commit 705dfc4575d6fae17c60a222cb5f78d8de43be38
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/I60PLB

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=705dfc4575d6fae17c60a222cb5f78d8de43be38



--------------------------------

commit fc9e18f9 upstream.

Under the following conditions:
* after rounding up by 4 the number of bytes to transfer (this is
  related to the controller's internal constraints),
* if this (rounded) amount of data is situated beyond the end of the
  device,
* and only in NV-DDR mode,
the Arasan NAND controller timeouts.

This currently can happen in a particular helper used when picking
software ECC algorithms. Let's prevent this situation by refusing to use
the NV-DDR interface with software engines.

Fixes: 4edde603 ("mtd: rawnand: arasan: Support NV-DDR interface")
Signed-off-by: default avatarMiquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20211008163640.1753821-1-miquel.raynal@bootlin.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarZheng Zengkai <zhengzengkai@huawei.com>
Reviewed-by: default avatarWei Li <liwei391@huawei.com>
parent 162f43b0

drivers/mtd/nand/raw/arasan-nand-controller.c

+15 −0
Original line number Diff line number Diff line
@@ -891,6 +891,21 @@ static int anfc_setup_interface(struct nand_chip *chip, int target, 
		nvddr = nand_get_nvddr_timings(conf);

		if (IS_ERR(nvddr))

			return PTR_ERR(nvddr);



		/*

		 * The controller only supports data payload requests which are

		 * a multiple of 4. In practice, most data accesses are 4-byte

		 * aligned and this is not an issue. However, rounding up will

		 * simply be refused by the controller if we reached the end of

		 * the device *and* we are using the NV-DDR interface(!). In

		 * this situation, unaligned data requests ending at the device

		 * boundary will confuse the controller and cannot be performed.

		 *

		 * This is something that happens in nand_read_subpage() when

		 * selecting software ECC support and must be avoided.

		 */

		if (chip->ecc.engine_type == NAND_ECC_ENGINE_TYPE_SOFT)

			return -ENOTSUPP;

	} else {

		sdr = nand_get_sdr_timings(conf);

		if (IS_ERR(sdr))