linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: ezequiel.garcia@free-electrons.com (Ezequiel Garcia)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] Partially revert "mtd: nand: pxa3xx: Introduce 'marvell,armada370-nand' compatible string"
Date: Tue, 17 Dec 2013 21:42:45 -0300	[thread overview]
Message-ID: <20131218004245.GB10369@localhost> (raw)
In-Reply-To: <20131212230715.GX27149@ld-irv-0074.broadcom.com>

Hi Brian,

On Thu, Dec 12, 2013 at 03:07:15PM -0800, Brian Norris wrote:
> On Mon, Dec 09, 2013 at 06:36:26PM -0300, Ezequiel Garcia wrote:
> > This reverts c0f3b8643a6fa2461d70760ec49d21d2b031d611.
> 
> Modified to say "partially reverts".
> 
> > 
> > The "armada370-nand" compatible support is not complete, and it was mistake
> > to add it. Revert it and pospone the support until the infrastructure is
> > in place.
> > 
> > Cc: stable at vger.kernel.org
> > Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> > ---
> > Note this won't apply to v3.12 as it is, although fixing it should be easy.
> 
> Pushed this patch to linux-mtd.git, noting -stable for 3.12+.
> 

Just to have some context, this compatible was removed because it was
wrongly added and it doesn't belong at v3.12 and v3.13.

Now the revert is in place in v3.13-rc4 and it should be queued for
v3.12.x, so we are good. Thanks for taking care of it!

On the other side, all the work needed for the driver to work OK on
Armada 370/XP SoC is already in l2-mtd.git master, and I have some fixes
ready to support 2048B pages.

In conclusion, I think it's safe to add the compatible on top of such work,
and queue it for v3.14.

How should I send a patch to re-add this compatible? Do you plan to rebase
l2-mtd.git master on top of v3.13-rc4 or so?
-- 
Ezequiel Garc?a, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com

  reply	other threads:[~2013-12-18  0:42 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-09 21:36 [PATCH] Partially revert "mtd: nand: pxa3xx: Introduce 'marvell, armada370-nand' compatible string" Ezequiel Garcia
2013-12-09 21:40 ` [PATCH] Partially revert "mtd: nand: pxa3xx: Introduce 'marvell,armada370-nand' " Jason Cooper
2013-12-11 20:22   ` Ezequiel Garcia
2013-12-12 23:07 ` Brian Norris
2013-12-18  0:42   ` Ezequiel Garcia [this message]
2013-12-18  7:12     ` Brian Norris
2013-12-18 21:46       ` 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=20131218004245.GB10369@localhost \
    --to=ezequiel.garcia@free-electrons.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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;
as well as URLs for NNTP newsgroup(s).