From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 00/15] nand: device model bringup on am335x evm and am437x gpevm
Date: Tue, 7 Feb 2017 12:19:27 -0500 [thread overview]
Message-ID: <20170207171927.GA26129@bill-the-cat> (raw)
In-Reply-To: <22033665-079e-3fc0-e062-576d56102257@ti.com>
On Mon, Feb 06, 2017 at 12:50:25PM -0600, Grygorii Strashko wrote:
> Hi Tom,
>
> On 02/05/2017 07:56 PM, Tom Rini wrote:
> > On Tue, Jan 31, 2017 at 03:37:02PM -0600, Grygorii Strashko wrote:
> >
> >> This patch seires adds nand uclass driver and enables omap_gpmc to
> >> adopt driver model. This has been tested on AM335x GP EVM and
> >> AM437x GP EVM.
> >>
> >> As preparation for introducing nand uclass driver the new API
> >> get_nand_dev_by_index() was added to avoid direct acces to nand_info array and
> >> u-boot core files were updated to use it.
> >>
> >> Patches can be found at:
> >> git at git.ti.com:~gragst/ti-u-boot/gragsts-ti-u-boot.git dm-nand-v2
> >
> > My conceptual concern here is that we need to address (as I've stated
> > elsewhere) the need to get DM/DT support within SPL on these families
> > (and yes, figure out what we'll do with respect to HS and the further
> > size constraints) before we push farther down this path of DM/DT
> > conversion as it's adding more #if/#else/#endif code that we need to be
> > able to drop. Thanks!
> >
>
> I understand your concern and see it's discussed now in [1], but I'd
> like to understand how to proceed with this series.
>
> The original patches from from Mugunthan are at about 1 year old and
> and it took me pretty a lot of time to just re-base them properly,
> so will it be acceptable if i will re-send only patches 1-10 as they are
> do not depend on TI SPL and will unblock DM conversation of other drivers?
> Really want to reduce existed patch queue :(
>
> [1] https://www.mail-archive.com/u-boot at lists.denx.de/msg237938.html
We can take up some of the underlying patches now, yes. But we really
need to do the DM conversion fully, in order to really unblock progress.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170207/875a20cb/attachment.sig>
prev parent reply other threads:[~2017-02-07 17:19 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-31 21:37 [U-Boot] [PATCH v2 00/15] nand: device model bringup on am335x evm and am437x gpevm Grygorii Strashko
2017-01-31 21:37 ` [U-Boot] [PATCH v2 01/15] cmd: bootm: fix build when CONFIG_CMD_IMLS_NAND Grygorii Strashko
2017-02-06 1:56 ` Tom Rini
2017-02-06 15:33 ` Simon Glass
2017-02-09 3:03 ` [U-Boot] [U-Boot, v2, " Tom Rini
2017-01-31 21:37 ` [U-Boot] [PATCH v2 02/15] cmd: nand: abstract global variable usage for dm conversion Grygorii Strashko
2017-02-06 1:57 ` Tom Rini
2017-02-09 17:51 ` Grygorii Strashko
2017-02-09 18:12 ` Tom Rini
2017-02-06 15:33 ` Simon Glass
2017-02-08 21:23 ` [U-Boot] [U-Boot, v2, " Tom Rini
2017-01-31 21:37 ` [U-Boot] [PATCH v2 03/15] common: env_nand: use get_nand_dev_by_index() Grygorii Strashko
2017-02-06 1:57 ` Tom Rini
2017-02-06 15:33 ` Simon Glass
2017-01-31 21:37 ` [U-Boot] [PATCH v2 04/15] dfu: dfu_nand: " Grygorii Strashko
2017-02-06 1:57 ` Tom Rini
2017-02-06 15:34 ` Simon Glass
2017-01-31 21:37 ` [U-Boot] [PATCH v2 05/15] cmd: bootm: " Grygorii Strashko
2017-02-06 1:57 ` Tom Rini
2017-02-06 15:34 ` Simon Glass
2017-01-31 21:37 ` [U-Boot] [PATCH v2 06/15] cmd: jffs2: " Grygorii Strashko
2017-02-06 1:57 ` Tom Rini
2017-02-06 15:34 ` Simon Glass
2017-01-31 21:37 ` [U-Boot] [PATCH v2 07/15] common: " Grygorii Strashko
2017-02-06 1:57 ` Tom Rini
2017-02-06 15:33 ` Simon Glass
2017-01-31 21:37 ` [U-Boot] [PATCH v2 08/15] fs: " Grygorii Strashko
2017-02-06 1:57 ` Tom Rini
2017-02-06 15:34 ` Simon Glass
2017-01-31 21:37 ` [U-Boot] [PATCH v2 09/15] cmd: nand: remove direct access to struct mtd_info->priv Grygorii Strashko
2017-02-06 1:57 ` Tom Rini
2017-02-06 15:34 ` Simon Glass
2017-01-31 21:37 ` [U-Boot] [PATCH v2 10/15] drivers: nand: implement a NAND uclass Grygorii Strashko
2017-02-06 15:34 ` Simon Glass
2017-02-06 18:39 ` Grygorii Strashko
2017-02-10 16:22 ` Simon Glass
2017-02-10 18:23 ` Grygorii Strashko
2017-01-31 21:37 ` [U-Boot] [PATCH v2 11/15] drivers: nand: omap_gpmc: convert driver to adopt driver model Grygorii Strashko
2017-02-06 1:57 ` Tom Rini
2017-02-06 18:40 ` Grygorii Strashko
2017-02-10 16:22 ` Simon Glass
2017-01-31 21:37 ` [U-Boot] [PATCH v2 12/15] am43xx_evm: nand: do not define DM_NAND for spl Grygorii Strashko
2017-01-31 21:37 ` [U-Boot] [PATCH v2 13/15] defconfig: am43xx_evm: enable NAND driver model Grygorii Strashko
2017-01-31 21:37 ` [U-Boot] [PATCH v2 14/15] am335x_evm: nand: do not define DM_NAND for spl Grygorii Strashko
2017-01-31 21:37 ` [U-Boot] [PATCH v2 15/15] defconfig: am335x_evm: enable NAND driver model Grygorii Strashko
2017-02-06 1:56 ` [U-Boot] [PATCH v2 00/15] nand: device model bringup on am335x evm and am437x gpevm Tom Rini
2017-02-06 18:50 ` Grygorii Strashko
2017-02-07 17:19 ` 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=20170207171927.GA26129@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