From: Rob Herring <robherring2@gmail.com>
To: Olof Johansson <olof@lixom.net>, David Brown <davidb@codeaurora.org>
Cc: arm@kernel.org, Arnd Bergmann <arnd@arndb.de>,
Nicolas Pitre <nicolas.pitre@linaro.org>,
Thomas Gleixner <tglx@linutronix.de>,
Russell King <rmk@arm.linux.org.uk>,
linux-arm-msm@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, grant.likely@secretlab.ca
Subject: Re: [GIT PULL] msm core for 3.10
Date: Mon, 15 Apr 2013 07:15:22 -0500 [thread overview]
Message-ID: <516BEF5A.7000101@gmail.com> (raw)
In-Reply-To: <20130415034905.GC10949@quad.lixom.net>
On 04/14/2013 10:49 PM, Olof Johansson wrote:
> Hi,
>
> On Fri, Apr 12, 2013 at 03:43:15PM -0700, David Brown wrote:
>> The following changes since commit f6161aa153581da4a3867a2d1a7caf4be19b6ec9:
>>
>> Linux 3.9-rc2 (2013-03-10 16:54:19 -0700)
>>
>> are available in the git repository at:
>>
>> git://git.kernel.org/pub/scm/linux/kernel/git/davidb/linux-msm.git tags/msm-core-3.10
>> .../devicetree/bindings/arm/msm/timer.txt | 41 ++++----
>> arch/arm/boot/dts/msm8660-surf.dts | 20 ++--
>> arch/arm/boot/dts/msm8960-cdp.dts | 22 ++---
>> arch/arm/mach-msm/timer.c | 110 ++++++++++-----------
>> 4 files changed, 86 insertions(+), 107 deletions(-)
>
> I didn't see either of the device tree maintainers cc:d on the bindings
> update, nor an ack from them. It looks reasonable enough to me, but I
> also do know that in your private downstream tree you have a lot of new
> bindings. Please make sure these go through proper review channels as
> you start to upstream them; i.e. Grant Likely and Rob Herring should
> either merge or ack them.
>
> I've cc:d Rob and Grant on this reply and I'll merge it for now but to avoid
> delays in getting your code merged in the future please make sure they are in
> the loop from the beginning.
Other than breaking the DTB-kernel ABI, looks fine to me. It's a
platform decision whether they are okay with that or not.
Rob
WARNING: multiple messages have this Message-ID (diff)
From: robherring2@gmail.com (Rob Herring)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] msm core for 3.10
Date: Mon, 15 Apr 2013 07:15:22 -0500 [thread overview]
Message-ID: <516BEF5A.7000101@gmail.com> (raw)
In-Reply-To: <20130415034905.GC10949@quad.lixom.net>
On 04/14/2013 10:49 PM, Olof Johansson wrote:
> Hi,
>
> On Fri, Apr 12, 2013 at 03:43:15PM -0700, David Brown wrote:
>> The following changes since commit f6161aa153581da4a3867a2d1a7caf4be19b6ec9:
>>
>> Linux 3.9-rc2 (2013-03-10 16:54:19 -0700)
>>
>> are available in the git repository at:
>>
>> git://git.kernel.org/pub/scm/linux/kernel/git/davidb/linux-msm.git tags/msm-core-3.10
>> .../devicetree/bindings/arm/msm/timer.txt | 41 ++++----
>> arch/arm/boot/dts/msm8660-surf.dts | 20 ++--
>> arch/arm/boot/dts/msm8960-cdp.dts | 22 ++---
>> arch/arm/mach-msm/timer.c | 110 ++++++++++-----------
>> 4 files changed, 86 insertions(+), 107 deletions(-)
>
> I didn't see either of the device tree maintainers cc:d on the bindings
> update, nor an ack from them. It looks reasonable enough to me, but I
> also do know that in your private downstream tree you have a lot of new
> bindings. Please make sure these go through proper review channels as
> you start to upstream them; i.e. Grant Likely and Rob Herring should
> either merge or ack them.
>
> I've cc:d Rob and Grant on this reply and I'll merge it for now but to avoid
> delays in getting your code merged in the future please make sure they are in
> the loop from the beginning.
Other than breaking the DTB-kernel ABI, looks fine to me. It's a
platform decision whether they are okay with that or not.
Rob
next prev parent reply other threads:[~2013-04-15 12:15 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-12 22:40 [GIT PULL] msm cleanup for 3.10 David Brown
2013-04-12 22:40 ` David Brown
2013-04-12 22:43 ` [GIT PULL] msm core " David Brown
2013-04-12 22:43 ` David Brown
2013-04-15 3:49 ` Olof Johansson
2013-04-15 3:49 ` Olof Johansson
2013-04-15 12:15 ` Rob Herring [this message]
2013-04-15 12:15 ` Rob Herring
2013-04-12 22:43 ` [GIT PULL] msm defconfig " David Brown
2013-04-12 22:43 ` David Brown
2013-04-12 22:43 ` [GIT PULL] msm fixes " David Brown
2013-04-12 22:43 ` David Brown
2013-04-15 3:56 ` Olof Johansson
2013-04-15 3:56 ` Olof Johansson
2013-04-15 4:01 ` [GIT PULL] msm cleanup " Olof Johansson
2013-04-15 4:01 ` Olof Johansson
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=516BEF5A.7000101@gmail.com \
--to=robherring2@gmail.com \
--cc=arm@kernel.org \
--cc=arnd@arndb.de \
--cc=davidb@codeaurora.org \
--cc=grant.likely@secretlab.ca \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nicolas.pitre@linaro.org \
--cc=olof@lixom.net \
--cc=rmk@arm.linux.org.uk \
--cc=tglx@linutronix.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.