From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from kroah.org ([198.145.64.141] helo=coco.kroah.org) by canuck.infradead.org with esmtps (Exim 4.72 #1 (Red Hat Linux)) id 1QKAs2-0000tS-Cf for linux-mtd@lists.infradead.org; Wed, 11 May 2011 14:58:39 +0000 Date: Wed, 11 May 2011 07:54:31 -0700 From: Greg KH To: Kishore Kadiyala Subject: Re: [stable] [PATCH RESEND] omap : nand : fix subpage ecc issue with prefetch Message-ID: <20110511145431.GA18317@kroah.com> References: <1305123897-11371-1-git-send-email-kishore.kadiyala@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1305123897-11371-1-git-send-email-kishore.kadiyala@ti.com> Cc: Vimal Singh , linux-omap@vger.kernel.org, linux-mtd@lists.infradead.org, stable@kernel.org, dedekind1@gmail.com List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed, May 11, 2011 at 07:54:57PM +0530, Kishore Kadiyala wrote: > When reading/writing a subpage (When HW ECC is not available/enabled) > for number of bytes not aligned to 4, the mis-aligned bytes are handled > first (by cpu copy method) before enabling the Prefetch engine to/from > 'p'(start of buffer 'buf'). Then it reads/writes rest of the bytes with > the help of Prefetch engine, if available, or again using cpu copy method. > Currently, reading/writing of rest of bytes, is not done correctly since > its trying to read/write again to/from begining of buffer 'buf', > overwriting the mis-aligned bytes. > > Read & write using prefetch engine got broken in commit '2c01946c'. > We never hit a scenario of not getting 'gpmc_prefetch_enable' call > success. So, problem did not get caught up. > > Signed-off-by: Kishore Kadiyala > Signed-off-by: Vimal Singh > Reported-by: Bryan DE FARIA > --- > drivers/mtd/nand/omap2.c | 12 +++++------- > 1 files changed, 5 insertions(+), 7 deletions(-) This is not the correct way to submit patches for inclusion in the stable kernel tree. Please read Documentation/stable_kernel_rules.txt for how to do this properly.