From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga02.intel.com ([134.134.136.20]) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1UCnHd-00051T-8Y for linux-mtd@lists.infradead.org; Tue, 05 Mar 2013 08:31:38 +0000 Message-ID: <1362472331.2943.25.camel@sauron> Subject: Re: [PATCH 01/12] mtd: nand_ids: minor clean-ups From: Artem Bityutskiy To: Brian Norris Date: Tue, 05 Mar 2013 10:32:11 +0200 In-Reply-To: References: <1362415349-7107-1-git-send-email-dedekind1@gmail.com> <1362415349-7107-2-git-send-email-dedekind1@gmail.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Cc: Huang Shijie , Mike Dunn , MTD Maling List Reply-To: dedekind1@gmail.com List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, 2013-03-04 at 10:36 -0800, Brian Norris wrote: > > + * These are the new chips with large page size, page size and > > + * eraseblock size is determined from the extended id bytes. > > This is also incorrect grammar. It uses a comma to divide sentences > and also uses "is". I would write something like this: > > "These are the new chips with large page size. Their page size and > eraseblock size are determined from the extended ID bytes." Thanks Brian, will fix. -- Best Regards, Artem Bityutskiy