linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Russell King - ARM Linux <linux@armlinux.org.uk>
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org,
	Eric Miao <eric.y.miao@gmail.com>,
	Haojian Zhuang <haojian.zhuang@gmail.com>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ARM: mmp: replace NO_IRQ
Date: Tue, 6 Sep 2016 21:41:35 +0100	[thread overview]
Message-ID: <20160906204135.GE1041@n2100.armlinux.org.uk> (raw)
In-Reply-To: <4510913.a46LvNLeqZ@wuerfel>

On Tue, Sep 06, 2016 at 10:19:18PM +0200, Arnd Bergmann wrote:
> The patch was correct, the only problem that you pointed out already
> was that it needs to be applied on top of your patch. I didn't
> check when the file was touched the last time but only looked at
> the current state in linux-next that happened to be from your
> patch last week.

The patch is not correct on its own - and, as far as I'm aware, my
patch to ucb1x00-core.c should not be in linux-next - it is part of
a series which is still - partly - a work-in-progress, and the patch
has dependencies on other changes in the series which are not obvious.

I see now that Lee has picked it up - that's just great, because it
does have a dependency on gpio changes.  That's _why_ it's part of
the large series.  Had Lee even bothered to read the fscking covering
message to the sub-series that he was covered on, he'd have realised
that he shouldn't have taken it yet.

The series was posted to get review comments and testing, not for
people to start cherry-picking patches from it as they damn-well
wish.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

      reply	other threads:[~2016-09-06 20:41 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-06 14:07 [PATCH] ARM: mmp: replace NO_IRQ Arnd Bergmann
2016-09-06 14:24 ` Russell King - ARM Linux
2016-09-06 19:28   ` Arnd Bergmann
2016-09-06 19:44     ` Russell King - ARM Linux
2016-09-06 20:03       ` Linus Torvalds
2016-09-06 21:22         ` Russell King - ARM Linux
2016-09-08 20:16           ` Arnd Bergmann
2016-09-06 20:19       ` Arnd Bergmann
2016-09-06 20:41         ` Russell King - ARM Linux [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=20160906204135.GE1041@n2100.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=arnd@arndb.de \
    --cc=eric.y.miao@gmail.com \
    --cc=haojian.zhuang@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.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).