All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] mtd: nand: vf610_nfc: resync with upstream Linux version
Date: Thu, 15 Oct 2015 10:13:16 +0200	[thread overview]
Message-ID: <561F601C.9040901@denx.de> (raw)
In-Reply-To: <1444881617.5185.235.camel@freescale.com>

On 15/10/2015 06:00, Scott Wood wrote:
> On Wed, 2015-10-14 at 13:27 +0200, Albert ARIBAUD wrote:
>> On Wed, 14 Oct 2015 13:21:41 +0200, Albert ARIBAUD (3ADEV)
>> <albert.aribaud@3adev.fr> wrote:
>>> Hello Stefan,
>>>
>>> On Tue, 13 Oct 2015 22:11:42 -0700, Stefan Agner <stefan@agner.ch>
>>> wrote:
>>>> This resyncs the driver changes with the Linux version of the
>>>> driver. The driver received some feedback in the LKML and got
>>>> recently acceppted, the latest version can be found here:
>>>> https://lkml.org/lkml/2015/9/2/678
>>>>
>>>> Notable changes are:
>>>> - On ECC error, reread OOB and count bit flips in OOB too.
>>>>   If flipped bits are below threshold, also return an empty
>>>>   OOB buffer.
>>>> - Return the amount of bit flips in vf610_nfc_read_page.
>>>> - Use endianness aware vf610_nfc_read to read ECC status.
>>>> - Do not enable IDLE IRQ (since we do not operate with an
>>>>   interrupt service routine).
>>>> - Use type safe struct for buffer variants (vf610_nfc_alt_buf).
>>>> - Renamed variables in struct vf610_nfc (column and page_sz)
>>>>   to reflect better what they really representing.
>>>>
>>>> The U-Boot version currently does not support RAW NAND write
>>>> when using the HW ECC engine.
>>>>
>>>> Signed-off-by: Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>
>>>> Signed-off-by: Stefan Agner <stefan@agner.ch>
>>>
>>> Tested-by: Albert ARIBAUD (3ADEV) <albert.aribaud@3adev.fr>
>>
>> Putting Stefano Babic in To: list since this is IMX-like stuff.
>>
>> Stefano,
>>
>> I can pick this plus Stefan's other two and Anthony's vf610-related
>> patches if you want, or you can. Either way, Tom will take them as they
>> are essentially bugfixes.
>
> Acked-by: Scott Wood <scottwood@freescale.com>
>


Thanks Scott. I am picking this one and a couple of other patches and I
will send my PR to Tom.

Best regards,
Stefano



-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

  reply	other threads:[~2015-10-15  8:13 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-14  5:11 [U-Boot] [PATCH] mtd: nand: vf610_nfc: resync with upstream Linux version Stefan Agner
2015-10-14  8:43 ` Albert ARIBAUD
2015-10-14  8:51   ` Albert ARIBAUD
2015-10-14 11:21 ` Albert ARIBAUD
2015-10-14 11:27   ` Albert ARIBAUD
2015-10-15  4:00     ` Scott Wood
2015-10-15  8:13       ` Stefano Babic [this message]
2015-10-14 16:19   ` Stefan Agner

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=561F601C.9040901@denx.de \
    --to=sbabic@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.