public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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>

      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