From: Boris Brezillon <boris.brezillon@free-electrons.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 5/7] mtd: nand: Add+use mtd_to/from_nand and nand_get/set_controller_data
Date: Wed, 15 Jun 2016 09:59:15 +0200 [thread overview]
Message-ID: <20160615095915.6fa18fd8@bbrezillon> (raw)
In-Reply-To: <1465524420.22191.136.camel@buserror.net>
Hi Scott,
On Thu, 09 Jun 2016 21:07:00 -0500
Scott Wood <oss@buserror.net> wrote:
> On Mon, 2016-06-06 at 18:02 +0200, Boris Brezillon wrote:
> > On Mon, 30 May 2016 13:57:56 -0500
> > Scott Wood <oss@buserror.net> wrote:
> >
> > > These functions are part of the Linux 4.6 sync. They are being added
> > > before the main sync patch in order to make it easier to address the
> > > issue across all NAND drivers (many/most of which do not closely track
> > > their Linux counterparts) separately from other merge issues.
> >
> > Some drivers are still directly accessing chip->mtd. Here is a patch
> > fixing those places [1] (not tested).
> >
> > [1] http://code.bulix.org/inu69a-100476
>
> Could you send this as a proper patch?
Sure, see below. You can squash the changes into you patch (hence the
!fixup prefix).
>
> Though some of the changes seem to be removing SPL mtd pointers rather than
> converting chip->mtd. I don't care much whether those pointers are retained
> but it's not "directly accessing chip->mtd".
Yes, I dropped all the unnecessary changes.
Regards,
Boris
>
> -Scott
>
--->8---
next prev parent reply other threads:[~2016-06-15 7:59 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-30 18:57 [U-Boot] [PATCH 1/7] mtd: nand: Remove jz4740 driver Scott Wood
2016-05-30 18:57 ` [U-Boot] [PATCH 2/7] mtd: nand: Remove docg4 driver and palmtreo680 flashing tool Scott Wood
2016-05-30 18:57 ` [U-Boot] [PATCH 3/7] mtd: nand: Remove nand_info_t typedef Scott Wood
2016-05-30 18:57 ` [U-Boot] [PATCH 4/7] nand: Embed mtd_info in struct nand_chip Scott Wood
2016-05-30 18:57 ` [U-Boot] [PATCH 5/7] mtd: nand: Add+use mtd_to/from_nand and nand_get/set_controller_data Scott Wood
2016-06-06 16:02 ` Boris Brezillon
2016-06-10 2:07 ` Scott Wood
2016-06-15 7:59 ` Boris Brezillon [this message]
2016-06-15 8:07 ` Boris Brezillon
2016-06-15 17:39 ` Scott Wood
2016-06-15 18:38 ` Boris Brezillon
2016-05-30 18:57 ` [U-Boot] [PATCH 6/7] mtd: nand: Add page argument to write_page() etc Scott Wood
2016-05-30 18:57 ` [U-Boot] [PATCH 7/7] mtd: nand: Sync with Linux v4.6 Scott Wood
2016-05-31 6:48 ` Heiko Schocher
2016-05-31 7:19 ` Scott Wood
2016-05-31 7:10 ` Boris Brezillon
2016-06-06 16:04 ` Boris Brezillon
2016-06-07 4:20 ` Heiko Schocher
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=20160615095915.6fa18fd8@bbrezillon \
--to=boris.brezillon@free-electrons.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.