[PATCH v3 1/9] mtd: nand: define struct nand_timings
b.brezillon.dev at gmail.com
Thu May 1 10:36:39 PDT 2014
On 30/04/2014 19:51, Brian Norris wrote:
> Hi Boris,
> On Wed, Mar 12, 2014 at 07:07:36PM +0100, Boris BREZILLON wrote:
>> + * struct nand_sdr_timings - SDR NAND chip timings
>> + *
>> + * This struct defines the timing requirements of a SDR NAND chip.
>> + * These informations can be found in every NAND datasheets and the timings
>> + * meaning are described in the ONFI specifications:
>> + * www.onfi.org/~/media/ONFI/specs/onfi_3_1_spec.pdf (chapter 4.15 Timing
> Can you remove the unicode U+200E character?
>> + * Parameters)
> Please document the units for these fields here. It looks like you're
> using picoseconds.
I'll add field units (which are indeed picoseconds) to this comment.
More information about the linux-mtd