public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [U-Boot, 1/4, v2] mtd: nand: fsmc: Add BCH4 SW ECC support for SPEAr600
Date: Mon, 14 Sep 2015 07:30:56 +0200	[thread overview]
Message-ID: <55F65B90.8000606@denx.de> (raw)
In-Reply-To: <20150912152031.GU26226@bill-the-cat>

On 12.09.2015 17:20, Tom Rini wrote:
> On Sat, Sep 12, 2015 at 09:46:51AM -0500, Scott Wood wrote:
>> On Sat, 2015-09-12 at 10:46 -0400, Tom Rini wrote:
>>> On Sat, Sep 12, 2015 at 09:36:54AM -0500, Scott Wood wrote:
>>>> On Sat, 2015-09-12 at 08:50 -0400, Tom Rini wrote:
>>>>> On Wed, Sep 02, 2015 at 02:29:12PM +0200, Stefan Roese wrote:
>>>>>
>>>>>> This patch adds support for 4-bit ECC BCH4 for the SPEAr600 SoC. This
>>>>>> can
>>>>>> be used by boards equipped with a NAND chip that requires 4-bit ECC
>>>>>> strength.
>>>>>> The SPEAr600 HW ECC only supports 1-bit ECC strength.
>>>>>>
>>>>>> To enable SW BCH4, you need to specify this in your config header:
>>>>>>
>>>>>> #define CONFIG_NAND_ECC_BCH
>>>>>> #define CONFIG_BCH
>>>>>>
>>>>>> And use the command "nandecc bch4" to select this ECC scheme upon
>>>>>> runtime.
>>>>>>
>>>>>> Tested on SPEAr600 x600 board.
>>>>>>
>>>>>> Signed-off-by: Stefan Roese <sr@denx.de>
>>>>>> Cc: Scott Wood <scottwood@freescale.com>
>>>>>> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
>>>>>
>>>>> Applied to u-boot/master, thanks!
>>>>>
>>>>
>>>> There's a v3, and some minor comments even on that one...
>>>
>>> Mutter, sorry.  Would you rather a revert of the series or just
>>> incremental on top?
>>
>> Either is fine -- I just wanted to make sure it didn't get forgotten.
>
> OK, Stefan, please do an incremental on top of master now to cover
> what's been noted in v3, thanks!

Sure.

Thanks,
Stefan

      reply	other threads:[~2015-09-14  5:30 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-02 12:29 [U-Boot] [PATCH 1/4 v2] mtd: nand: fsmc: Add BCH4 SW ECC support for SPEAr600 Stefan Roese
2015-09-10 22:31 ` Scott Wood
2015-09-11  5:56   ` Stefan Roese
2015-09-11  6:23 ` [U-Boot] [PATCH 1/4 v3] " Stefan Roese
2015-09-11 18:02   ` Scott Wood
2015-09-12 12:50 ` [U-Boot] [U-Boot, 1/4, v2] " Tom Rini
2015-09-12 14:36   ` Scott Wood
2015-09-12 14:46     ` Tom Rini
2015-09-12 14:46       ` Scott Wood
2015-09-12 15:20         ` Tom Rini
2015-09-14  5:30           ` Stefan Roese [this message]

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=55F65B90.8000606@denx.de \
    --to=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /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