Commit dca3c3ce authored by Boris Brezillon's avatar Boris Brezillon Committed by Miquel Raynal
Browse files

mtd: rawnand: atmel: Drop redundant nand_read_page_op() 



The legacy page read path in atmel_hsmc_nand_pmecc_read_pg() issues
a nand_read_page_op() that's already issued by
atmel_nand_pmecc_read_pg(). Let's get rid of the unneeded one.

Signed-off-by: default avatarBoris Brezillon <boris.brezillon@collabora.com>
Reviewed-by: default avatarMiquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: default avatarTudor Ambarus <tudor.ambarus@microchip.com>
Signed-off-by: default avatarTudor Ambarus <tudor.ambarus@microchip.com>
Signed-off-by: default avatarMiquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20200720131356.1579073-3-tudor.ambarus@microchip.com
parent b0155dad

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

+1 −4
Original line number Diff line number Diff line
@@ -983,12 +983,9 @@ static int atmel_hsmc_nand_pmecc_read_pg(struct nand_chip *chip, u8 *buf, 
	 * connected to a native SoC R/B pin. If that's not the case, fallback

	 * to the non-optimized one.

	 */

	if (nand->activecs->rb.type != ATMEL_NAND_NATIVE_RB) {

		nand_read_page_op(chip, page, 0, NULL, 0);



	if (nand->activecs->rb.type != ATMEL_NAND_NATIVE_RB)

		return atmel_nand_pmecc_read_pg(chip, buf, oob_required, page,

						raw);

	}



	nc->op.cmds[nc->op.ncmds++] = NAND_CMD_READ0;

CRA Git | Maintained and supported by SUSTech CRA and CCSE