From: Boris Brezillon <boris.brezillon@bootlin.com>
To: Benjamin Lindqvist <benjamin.lindqvist@endian.se>
Cc: "Stefan Agner" <stefan@agner.ch>,
"Dmitry Osipenko" <digetx@gmail.com>,
"David Woodhouse" <dwmw2@infradead.org>,
"Brian Norris" <computersforpeace@gmail.com>,
"Marek Vašut" <marek.vasut@gmail.com>,
robh+dt@kernel.org, mark.rutland@arm.com,
"Thierry Reding" <thierry.reding@gmail.com>,
"Michael Turquette" <mturquette@baylibre.com>,
sboyd@kernel.org, "Lucas Stach" <dev@lynxeye.de>,
"Miquel Raynal" <miquel.raynal@bootlin.com>,
richard@nod.at, "Marcel Ziswiler" <marcel@ziswiler.com>,
krzk@kernel.org, jonathanh@nvidia.com, pdeschrijver@nvidia.com,
"Prashant Gaikwad" <pgaikwad@nvidia.com>,
"Mirza Krak" <mirza.krak@gmail.com>,
linux-mtd@lists.infradead.org, linux-tegra@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-clk@vger.kernel.org
Subject: Re: [PATCH v2 3/6] mtd: rawnand: add NVIDIA Tegra NAND Flash controller driver
Date: Mon, 28 May 2018 18:57:14 +0200 [thread overview]
Message-ID: <20180528185714.0a33bdc7@bbrezillon> (raw)
In-Reply-To: <CAGefG56pyKd=rCjArVtvO1mnXzacrD+BW4_AkJSDRtw-2Gf3Nw@mail.gmail.com>
On Mon, 28 May 2018 18:41:26 +0200
Benjamin Lindqvist <benjamin.lindqvist@endian.se> wrote:
> Note that it's certainly possible to encode U-Boot and kernel with
> RS[4] and still use RS[8] for the rootfs even if the boot rom doesn't
> support it.
Not if you want to read/write from/to the uboot partition from Linux.
Per-partition ECC setup is not supported, so the only solutions we have
right now are:
1/ use the same ECC config as the one use by the bootrom
2/ only access uboot partition in raw mode from Linux (that implies
generating images containing the ECC bytes so that they can be
flashed to the device in raw mode)
3/ never access uboot partitions from Linux (not sure we want that)
> This whole 'use-bootable-ecc-only' business seems a bit
> overengineered.
>
Hm, I don't find it over-engineered.
next prev parent reply other threads:[~2018-05-28 16:57 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-27 21:54 [PATCH v2 3/6] mtd: rawnand: add NVIDIA Tegra NAND Flash controller driver Stefan Agner
2018-05-27 22:19 ` Miquel Raynal
2018-05-28 12:41 ` Stefan Agner
2018-05-28 11:57 ` Dmitry Osipenko
2018-05-28 11:57 ` Dmitry Osipenko
2018-05-28 12:43 ` Stefan Agner
2018-05-28 16:41 ` Benjamin Lindqvist
2018-05-28 16:41 ` Benjamin Lindqvist
2018-05-28 16:57 ` Boris Brezillon [this message]
2018-05-31 9:37 ` Stefan Agner
2018-05-31 9:55 ` Stefan Agner
2018-06-07 9:58 ` Miquel Raynal
2018-06-07 9:58 ` Miquel Raynal
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=20180528185714.0a33bdc7@bbrezillon \
--to=boris.brezillon@bootlin.com \
--cc=benjamin.lindqvist@endian.se \
--cc=computersforpeace@gmail.com \
--cc=dev@lynxeye.de \
--cc=devicetree@vger.kernel.org \
--cc=digetx@gmail.com \
--cc=dwmw2@infradead.org \
--cc=jonathanh@nvidia.com \
--cc=krzk@kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=linux-tegra@vger.kernel.org \
--cc=marcel@ziswiler.com \
--cc=marek.vasut@gmail.com \
--cc=mark.rutland@arm.com \
--cc=miquel.raynal@bootlin.com \
--cc=mirza.krak@gmail.com \
--cc=mturquette@baylibre.com \
--cc=pdeschrijver@nvidia.com \
--cc=pgaikwad@nvidia.com \
--cc=richard@nod.at \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
--cc=stefan@agner.ch \
--cc=thierry.reding@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 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.