From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
To: Brian Norris <computersforpeace@gmail.com>
Cc: "Thomas Petazzoni" <thomas.petazzoni@free-electrons.com>,
"Andrew Lunn" <andrew@lunn.ch>,
"Jason Cooper" <jason@lakedaemon.net>,
"Emilio López" <emilio@elopez.com.ar>,
"Lior Amsalem" <alior@marvell.com>,
linux-mtd@lists.infradead.org,
"Gregory Clement" <gregory.clement@free-electrons.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] mtd: nand: pxa3xx: Disable "armada370-nand" compatible support
Date: Mon, 9 Dec 2013 18:08:56 -0300 [thread overview]
Message-ID: <20131209210855.GB31944@localhost> (raw)
In-Reply-To: <20131209204607.GV27149@ld-irv-0074.broadcom.com>
On Mon, Dec 09, 2013 at 12:46:07PM -0800, Brian Norris wrote:
> Hi Ezequiel,
>
> On Sat, Dec 07, 2013 at 08:40:49PM -0500, Jason Cooper wrote:
> > On Sat, Dec 07, 2013 at 10:35:26PM -0300, Ezequiel Garcia wrote:
> > > If we agree to remove the lines, shall we also revert the binding
> > > documentation, removing the compatible from there as well?
> >
> > I think this is unnecessary. We're trying to prevent breaking
> > bisection. No one in their right mind should be developing (and
> > referring to docs) at a random commit. As long as the docs are correct,
> > leave them alone.
>
> I agree, we don't need to drop all the docs, just the lines with the
> 'compatible' property. And #if 0 is a no-go; let's just delete the
> lines.
>
> Yes, the docs will appear in 3.12 and 3.13 even though the driver
> support won't be there until 3.14, but that's OK IMO.
>
> Do you think this should go to 3.12.y stable? If so, please add the
> appropriate Cc tag. BTW, you moved these lines around in 3.13 so far,
> so the patch probably won't apply 100% clean.
>
> Can you refresh this patch according to Jason's comments and resend?
>
Sure!
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: ezequiel.garcia@free-electrons.com (Ezequiel Garcia)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] mtd: nand: pxa3xx: Disable "armada370-nand" compatible support
Date: Mon, 9 Dec 2013 18:08:56 -0300 [thread overview]
Message-ID: <20131209210855.GB31944@localhost> (raw)
In-Reply-To: <20131209204607.GV27149@ld-irv-0074.broadcom.com>
On Mon, Dec 09, 2013 at 12:46:07PM -0800, Brian Norris wrote:
> Hi Ezequiel,
>
> On Sat, Dec 07, 2013 at 08:40:49PM -0500, Jason Cooper wrote:
> > On Sat, Dec 07, 2013 at 10:35:26PM -0300, Ezequiel Garcia wrote:
> > > If we agree to remove the lines, shall we also revert the binding
> > > documentation, removing the compatible from there as well?
> >
> > I think this is unnecessary. We're trying to prevent breaking
> > bisection. No one in their right mind should be developing (and
> > referring to docs) at a random commit. As long as the docs are correct,
> > leave them alone.
>
> I agree, we don't need to drop all the docs, just the lines with the
> 'compatible' property. And #if 0 is a no-go; let's just delete the
> lines.
>
> Yes, the docs will appear in 3.12 and 3.13 even though the driver
> support won't be there until 3.14, but that's OK IMO.
>
> Do you think this should go to 3.12.y stable? If so, please add the
> appropriate Cc tag. BTW, you moved these lines around in 3.13 so far,
> so the patch probably won't apply 100% clean.
>
> Can you refresh this patch according to Jason's comments and resend?
>
Sure!
--
Ezequiel Garc?a, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
next prev parent reply other threads:[~2013-12-09 21:08 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-07 15:39 [PATCH] mtd: nand: pxa3xx: Disable "armada370-nand" compatible support Ezequiel Garcia
2013-12-07 15:39 ` Ezequiel Garcia
2013-12-07 23:23 ` Jason Cooper
2013-12-07 23:23 ` Jason Cooper
2013-12-08 1:35 ` Ezequiel Garcia
2013-12-08 1:35 ` Ezequiel Garcia
2013-12-08 1:40 ` Jason Cooper
2013-12-08 1:40 ` Jason Cooper
2013-12-09 20:46 ` Brian Norris
2013-12-09 20:46 ` Brian Norris
2013-12-09 21:08 ` Ezequiel Garcia [this message]
2013-12-09 21:08 ` Ezequiel Garcia
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=20131209210855.GB31944@localhost \
--to=ezequiel.garcia@free-electrons.com \
--cc=alior@marvell.com \
--cc=andrew@lunn.ch \
--cc=computersforpeace@gmail.com \
--cc=emilio@elopez.com.ar \
--cc=gregory.clement@free-electrons.com \
--cc=jason@lakedaemon.net \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-mtd@lists.infradead.org \
--cc=thomas.petazzoni@free-electrons.com \
/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.