From 4101aeac54ded9591c52ac03a1b765afc394d6c0 Mon Sep 17 00:00:00 2001 From: Aidan MacDonald Date: Mon, 11 Jul 2022 21:08:40 +0100 Subject: x1000: simplify NAND command macros There is actually no need to parameterize most commands with row or column cycles, usually the opcode and row/column address width are linked. When this is not the case we can use raw SFC commands or define new macros. Change-Id: I22459d732dc01012e6a8ae026c4fb85495d372b4 --- firmware/target/mips/ingenic_x1000/nand-x1000.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'firmware/target/mips/ingenic_x1000/nand-x1000.c') diff --git a/firmware/target/mips/ingenic_x1000/nand-x1000.c b/firmware/target/mips/ingenic_x1000/nand-x1000.c index 896ac97d28..827a79ebce 100644 --- a/firmware/target/mips/ingenic_x1000/nand-x1000.c +++ b/firmware/target/mips/ingenic_x1000/nand-x1000.c @@ -44,11 +44,11 @@ const struct nand_chip supported_nand_chips[] = { STA_TYPE_V(1BYTE), CMD_TYPE_V(8BITS), SMP_DELAY(1)), .flags = NAND_CHIPFLAG_QUAD | NAND_CHIPFLAG_HAS_QE_BIT, - .cmd_page_read = NANDCMD_PAGE_READ(3), - .cmd_program_execute = NANDCMD_PROGRAM_EXECUTE(3), - .cmd_block_erase = NANDCMD_BLOCK_ERASE(3), - .cmd_read_cache = NANDCMD_READ_CACHE_x4(2), - .cmd_program_load = NANDCMD_PROGRAM_LOAD_x4(2), + .cmd_page_read = NANDCMD_PAGE_READ, + .cmd_program_execute = NANDCMD_PROGRAM_EXECUTE, + .cmd_block_erase = NANDCMD_BLOCK_ERASE, + .cmd_read_cache = NANDCMD_READ_CACHE_x4, + .cmd_program_load = NANDCMD_PROGRAM_LOAD_x4, }, #else { 0 }, @@ -103,7 +103,7 @@ static bool identify_chip(struct nand_drv* drv) * * Currently we use the 2nd method, aka. address read ID. */ - sfc_exec(NANDCMD_READID(1, 0), 0, drv->scratch_buf, 4|SFC_READ); + sfc_exec(NANDCMD_READID_ADDR, 0, drv->scratch_buf, 4|SFC_READ); drv->mf_id = drv->scratch_buf[0]; drv->dev_id = drv->scratch_buf[1]; drv->dev_id2 = drv->scratch_buf[2]; -- cgit v1.2.3