[PATCH v1 3/3] mtd: nand: introduce a READMODE command

Gerhard Sittig gsi at denx.de
Tue Apr 1 07:10:35 PDT 2014


On Tue, 2014-04-01 at 06:56 +0000, Gupta, Pekon wrote:
> 
> Hi Gerhard,
> 
> >From: Gerhard Sittig [mailto:gsi at denx.de]
> >
> >the nand_command_lp() implementation derives a "READPAGE" sequence from
> >a passed in READ0 opcode, i.e. emits a sequence of READ0 _and_ READSTART
> >commands in this case
> >
> >introduce a "READMODE" command which sends the READ0 opcode to the chip
> >exclusively and doesn't send the READSTART opcode
> >
> >such a "READMODE" command is useful in the context of on-die-ECC support
> >where a sequence of READ0, READSTART, STATUS, READ0 is required; having
> >support for READMODE in the common nand_command_lp() routine avoids the
> >need for duplication and open coded cmd_ctrl() calls
> >
> >[ ... ]
> 
> Thanks. This should help 'David Mosberger <davidm at egauge.net>' patches.
> I think, NAND_CMD_READMODE should also be added to nand_command(), right ?  

Oh, that has slipped my attention.  I'll wait a few more days for
feedback, and update the series to support READMODE in both the
nand_command() as well as nand_command_lp() variants.  Callers
should not mind which one is the registered cmdfunc() callback.

> With that please feel free to apply ..
> Acked-by: Pekon Gupta <pekon at ti.com>

thank you, both for looking and for the ACK


virtually yours
Gerhard Sittig
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr. 5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de



More information about the linux-mtd mailing list