From: Kishore Kadiyala <kishorek.kadiyala@gmail.com>
To: Greg KH <greg@kroah.com>
Cc: dedekind1@gmail.com, Kishore Kadiyala <kishore.kadiyala@ti.com>,
Vimal Singh <vimal.newwork@gmail.com>,
linux-mtd@lists.infradead.org, linux-omap@vger.kernel.org,
stable@kernel.org
Subject: Re: [PATCH RESEND] omap : nand : fix subpage ecc issue with prefetch
Date: Wed, 11 May 2011 21:07:55 +0530 [thread overview]
Message-ID: <BANLkTinsZsBhVbcGRnx0fzWwA2Q0f4Fs3w@mail.gmail.com> (raw)
In-Reply-To: <20110511145431.GA18317@kroah.com>
On Wed, May 11, 2011 at 8:24 PM, Greg KH <greg@kroah.com> wrote:
> 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 <kishore.kadiyala@ti.com>
>> Signed-off-by: Vimal Singh <vimal.newwork@gmail.com>
>> Reported-by: Bryan DE FARIA <bdefaria@adeneo-embedded.com>
>> ---
>> drivers/mtd/nand/omap2.c | 12 +++++-------
>> 1 files changed, 5 insertions(+), 7 deletions(-)
>
>
> <formletter>
>
> 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.
Ok, will post it properly.
Regards,
Kishore
>
> </formletter>
>
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/
>
_______________________________________________
stable mailing list
stable@linux.kernel.org
http://linux.kernel.org/mailman/listinfo/stable
next prev parent reply other threads:[~2011-05-11 15:37 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-11 14:24 [PATCH RESEND] omap : nand : fix subpage ecc issue with prefetch Kishore Kadiyala
2011-05-11 14:54 ` Greg KH
2011-05-11 15:37 ` Kishore Kadiyala [this message]
-- strict thread matches above, loose matches on Subject: below --
2011-05-11 15:47 Kishore Kadiyala
2011-05-12 12:56 ` Artem Bityutskiy
2011-05-09 14:21 Kishore Kadiyala
2011-05-11 14:15 ` Kishore Kadiyala
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=BANLkTinsZsBhVbcGRnx0fzWwA2Q0f4Fs3w@mail.gmail.com \
--to=kishorek.kadiyala@gmail.com \
--cc=dedekind1@gmail.com \
--cc=greg@kroah.com \
--cc=kishore.kadiyala@ti.com \
--cc=linux-mtd@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=stable@kernel.org \
--cc=vimal.newwork@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).