33 lines
1.2 KiB
Diff
33 lines
1.2 KiB
Diff
|
From 11532c10a29e4faef29b5f3b354391d1e2f90213 Mon Sep 17 00:00:00 2001
|
||
|
|
From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
|
||
|
|
Date: Thu, 14 Nov 2013 18:25:32 -0300
|
||
|
|
Subject: [PATCH 146/203] mtd: nand: pxa3xx: Remove READ0 switch/case
|
||
|
|
falltrough
|
||
|
|
|
||
|
|
READ0 and READOOB command preparation has a falltrough to SEQIN
|
||
|
|
case, where the command address is specified.
|
||
|
|
This is certainly confusing and makes the code less readable with
|
||
|
|
no added value. Let's remove it.
|
||
|
|
|
||
|
|
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
|
||
|
|
Tested-by: Daniel Mack <zonque@gmail.com>
|
||
|
|
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
|
||
|
|
---
|
||
|
|
drivers/mtd/nand/pxa3xx_nand.c | 5 +++++
|
||
|
|
1 file changed, 5 insertions(+)
|
||
|
|
|
||
|
|
--- a/drivers/mtd/nand/pxa3xx_nand.c
|
||
|
|
+++ b/drivers/mtd/nand/pxa3xx_nand.c
|
||
|
|
@@ -669,6 +669,11 @@ static int prepare_command_pool(struct p
|
||
|
|
if (mtd->writesize >= PAGE_CHUNK_SIZE)
|
||
|
|
info->ndcb0 |= NDCB0_DBC | (NAND_CMD_READSTART << 8);
|
||
|
|
|
||
|
|
+ set_command_address(info, mtd->writesize, column, page_addr);
|
||
|
|
+ info->buf_count = mtd->writesize + mtd->oobsize;
|
||
|
|
+ memset(info->data_buff, 0xFF, info->buf_count);
|
||
|
|
+ break;
|
||
|
|
+
|
||
|
|
case NAND_CMD_SEQIN:
|
||
|
|
|
||
|
|
set_command_address(info, mtd->writesize, column, page_addr);
|