From: Tony Lindgren <tony@atomide.com>
To: pekon <pekon@pek-sem.com>
Cc: Roger Quadros <rogerq@ti.com>,
computersforpeace@gmail.com, dwmw2@infradead.org,
ezequiel@vanguardiasur.com.ar, linux-mtd@lists.infradead.org,
linux-omap@vger.kernel.org, stable@vger.kernel.org
Subject: Re: [PATCH] mtd: nand: omap: Fix NAND enumeration on 3430 LDP
Date: Wed, 12 Nov 2014 10:02:54 -0800 [thread overview]
Message-ID: <20141112180253.GJ26481@atomide.com> (raw)
In-Reply-To: <545FC08D.8000008@pek-sem.com>
* pekon <pekon@pek-sem.com> [141109 11:31]:
> On Saturday 08 November 2014 04:18 AM, Tony Lindgren wrote:
> >
> >Right. I doubt anybody has bch8 rootfs on LDP.. And considering u-boot
> >must be ham1 to boot at all, that's what we should change for the
> >devices that do not have not standardized on bch8.
> >
> My view on this is slightly different:
> - Lately, everyone seems to have migrated to BCH8.
>
> - Also HAM1 does not have strength to fix bitflips in aging NAND. So if
> someone already has OMAP3-LDP deployed on field then its NAND would have
> already aged to such an extend that HAM1 may not be sufficient.
OK so it makes sense to keep it as BCH8 then. Would be nice to get
the writing u-boot from kernel issue fixed somehow though as people
are hitting that for sure.
> My question is..
> Moving back to HAM1 should be decided based on statistics rather than rule
> of breaking backward compatibility. So please review:
> (1) How many user of OMAP3-Zoom or other old boards complaining about using
> BCH8 in mainline kernel? OR
0
> (2) How many users of OMAP3 legacy boards are migrating to newer kernel?
Quite a few for sure, but are probably also using rootfs on MMC instead.
> At-least I have not, rather most of the OMAP3 users I interacted via TI's
> E2E forum wanted to migrate to BCH8 or even BCH16, as HAM1 was not
> sufficient for their usage.
>
> So whatever you decide on this topic, please be cautious that you don't
> re-invent work for yourself, as I did. It took me lot of time and testing
> effort on multiple boards to migrate HAM1 to BCH8, And add BCH16 for newer
> devices.
Right no objections to using BCH8 for rootfs, except it stopped working
over past year or so.
And it seems the settings should be partition specific because of u-boot
requiring HAM1.
Regards,
Tony
next prev parent reply other threads:[~2014-11-12 18:03 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-05 11:00 [PATCH] mtd: nand: omap: Fix NAND enumeration on 3430 LDP Roger Quadros
2014-11-06 18:03 ` Tony Lindgren
2014-11-07 9:35 ` Roger Quadros
2014-11-07 9:58 ` Roger Quadros
2014-11-07 22:48 ` Tony Lindgren
2014-11-09 19:29 ` pekon
2014-11-12 18:02 ` Tony Lindgren [this message]
2014-11-13 11:29 ` Roger Quadros
2014-11-13 16:00 ` Tom Rini
2014-11-13 17:54 ` Tony Lindgren
2014-11-15 11:12 ` pekon
2014-11-13 12:00 ` Roger Quadros
2014-11-19 12:22 ` [PATCH v2] " Roger Quadros
2014-11-26 6:47 ` Brian Norris
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=20141112180253.GJ26481@atomide.com \
--to=tony@atomide.com \
--cc=computersforpeace@gmail.com \
--cc=dwmw2@infradead.org \
--cc=ezequiel@vanguardiasur.com.ar \
--cc=linux-mtd@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=pekon@pek-sem.com \
--cc=rogerq@ti.com \
--cc=stable@vger.kernel.org \
/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).