From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/4] nand: davinci: enable driver model for NAND
Date: Wed, 3 Jul 2019 09:10:26 -0400 [thread overview]
Message-ID: <20190703131026.GI8554@bill-the-cat> (raw)
In-Reply-To: <CAMRc=MfYf+TKsQAfyJV4cyqdmkHMaPS2aSCt0V2L3mSbau-7Jw@mail.gmail.com>
On Wed, Jul 03, 2019 at 03:01:15PM +0200, Bartosz Golaszewski wrote:
> pt., 7 cze 2019 o 10:16 Bartosz Golaszewski <brgl@bgdev.pl> napisał(a):
> >
> > czw., 6 cze 2019 o 17:58 Adam Ford <aford173@gmail.com> napisał(a):
> > >
> > > On Thu, Jun 6, 2019 at 10:50 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> > > >
> > > > From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> > > >
> > > > This series enables the driver model for NAND on davinci da850 boards.
> > > >
> > > > The first patch is a simple tweak for an unnecessarily exported function.
> > > >
> > > > The second patch extends the NAND driver to support both legacy and
> > > > driver-model users. For now we don't parse the device-tree as it's a
> > > > bit complicated on da850, namely: the nand node is a child of the
> > > > aemif node for which we don't have a driver in u-boot (unlike on linux
> > > > where the aemif driver populates all its subnodes). In order for the
> > > > nand device to be probed, we're adding a dummy node to the u-boot dts
> > > > extensions with an appropriate compatible.
> > > >
> > > > Two last patches enable driver model for NAND on da850-lcdk and da850-evm.
> > > > We don't enable the driver model in SPL as first: the patches enabling
> > > > CONFIG_SPL_DM are still waiting to be merged, and second: the
> > > > nand_spl_simple driver used by davinci SPL doesn't support driver model
> > > > users. This is planned for future series once some basic support is
> > > > merged.
> > >
> > > What happens when the board is booting SPL from NAND? Will it still
> > > work as expected? I know the da850-evm has at least one configuration
> > > which this is intended to boot from NAND instead of the default SPI
> > > flash.
> > >
> >
> > The driver works just like before in SPL. The support for the driver
> > model is added incrementally. In fact I tested it on da850-lcdk with
> > SPL loading u-boot from NAND.
> >
> > On da850-evm we could probably enable driver-model in SPL as well for
> > NAND, but I prefer to do it at once for both boards.
> >
> > Bart
> >
> > > adam
> > >
> > > >
> > > > Bartosz Golaszewski (4):
> > > > nand: davinci: make davinci_nand_init() static
> > > > nand: davinci: add support for driver model
> > > > davinci: omapl138-lcdk: enable driver model for NAND
> > > > davinci: da850-evm: enable driver model for NAND
> > > >
> > > > arch/arm/dts/da850-evm-u-boot.dtsi | 4 ++
> > > > arch/arm/dts/da850-lcdk-u-boot.dtsi | 4 ++
> > > > arch/arm/include/asm/ti-common/davinci_nand.h | 2 -
> > > > configs/da850evm_defconfig | 1 +
> > > > configs/da850evm_direct_nor_defconfig | 1 +
> > > > configs/da850evm_nand_defconfig | 1 +
> > > > configs/omapl138_lcdk_defconfig | 1 +
> > > > drivers/mtd/nand/raw/davinci_nand.c | 48 ++++++++++++++++++-
> > > > include/configs/da850evm.h | 4 ++
> > > > include/configs/omapl138_lcdk.h | 4 ++
> > > > 10 files changed, 66 insertions(+), 4 deletions(-)
> > > >
> > > > --
> > > > 2.21.0
> > > >
>
> Hi Tom,
>
> can you pick up this series?
Post-release, yes, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190703/440bc541/attachment.sig>
prev parent reply other threads:[~2019-07-03 13:10 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-06 15:49 [U-Boot] [PATCH 0/4] nand: davinci: enable driver model for NAND Bartosz Golaszewski
2019-06-06 15:49 ` [U-Boot] [PATCH 1/4] nand: davinci: make davinci_nand_init() static Bartosz Golaszewski
2019-06-06 15:49 ` [U-Boot] [PATCH 2/4] nand: davinci: add support for driver model Bartosz Golaszewski
2019-06-06 15:49 ` [U-Boot] [PATCH 3/4] davinci: omapl138-lcdk: enable driver model for NAND Bartosz Golaszewski
2019-06-06 15:50 ` [U-Boot] [PATCH 4/4] davinci: da850-evm: " Bartosz Golaszewski
2019-06-07 19:58 ` Adam Ford
2019-06-06 15:58 ` [U-Boot] [PATCH 0/4] nand: davinci: " Adam Ford
2019-06-07 8:16 ` Bartosz Golaszewski
2019-07-03 13:01 ` Bartosz Golaszewski
2019-07-03 13:10 ` Tom Rini [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=20190703131026.GI8554@bill-the-cat \
--to=trini@konsulko.com \
--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