All of lore.kernel.org
 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 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.