From: Benoit Cousson <bcousson@baylibre.com>
To: Olof Johansson <olof@lixom.net>
Cc: Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
Kevin Hilman <khilman@linaro.org>,
Javier Martinez Canillas <javier@dowhile0.org>,
Stephen Rothwell <sfr@canb.auug.org.au>, Greg KH <greg@kroah.com>,
Arnd Bergmann <arnd@arndb.de>,
linux-kernel@vger.kernel.org, Felipe Balbi <balbi@ti.com>,
linux-next@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: linux-next: manual merge of the arm-soc tree with the usb tree
Date: Wed, 28 Aug 2013 15:01:38 +0200 [thread overview]
Message-ID: <521DF4B2.6070205@baylibre.com> (raw)
In-Reply-To: <20130827161203.GA22852@quad.lixom.net>
Hi Olof,
On 27/08/2013 18:12, Olof Johansson wrote:
> On Tue, Aug 27, 2013 at 05:25:23PM +0200, Sebastian Andrzej Siewior wrote:
>> On 08/27/2013 05:01 PM, Kevin Hilman wrote:
>>>>> What do we do now?
>>>>
>>>> Cannot you just merge the stable arm-soc/dt branch into your branch
>>>> before applying your patches?
>>>
>>> Unfortunately, the next/dt branch of arm-soc is not necessarily stable
>>> so should *not* be merged. In fact none of the arm-soc branches should
>>> be considered stable.
>>>
>>> As was already mentioned, this should be split up into driver changes
>>> and DTS changes through arm-soc. They'll both merge for v3.12.
>>
>> But splitting will break the driver until .dts & code is in sync again.
>>
>>> BTW, how did this patch get merged without a signoff/ack from the OMAP
>>> DT maintainer in the first place? Hmm, looks like Benoit was not copied
>>> nor was linux-omap or linux-arm-kernel copied in the original mails.
>>
>> Hmm. I had Benoit's okay [0] to do the change "as long as Felipe is
>> fine with it". I indeed forgot to Cc Benoit on the dts changes.
>> For the phy-rename Felipe pinged you and we did the topic-branch, here
>> I forgot.
>
> No. Read that email again. What Benoit said was that if Felipe was fine
> with the change _HE_ would take it. Huge difference, and one that would have
> avoided this situation.
>
> The only way to solve these things in the future is to make the driver handle
> both the new and the old binding. Bindings are not supposed to change in
> incompatible ways any more, unless for special circumstances and/or when the
> old binding was completely broken.
>
>
> The only way forward here, since Greg runs a stable tree that he doesn't
> rebase, is for us to rebuild without the OMAP DT branch, and ask Benoit to take
> out the conflicting changes.
>
> Benoit, I know this is none of your fault, but would you mind preparing a new
> copy of the DT branch without the conflicting patches, and hold those to 3.13?
> I haven't looked to see how many those were.
OK, I'll do that ASAP and check how many should be removed to avoid a
conflict with usb-next.
Regards,
Benoit
WARNING: multiple messages have this Message-ID (diff)
From: bcousson@baylibre.com (Benoit Cousson)
To: linux-arm-kernel@lists.infradead.org
Subject: linux-next: manual merge of the arm-soc tree with the usb tree
Date: Wed, 28 Aug 2013 15:01:38 +0200 [thread overview]
Message-ID: <521DF4B2.6070205@baylibre.com> (raw)
In-Reply-To: <20130827161203.GA22852@quad.lixom.net>
Hi Olof,
On 27/08/2013 18:12, Olof Johansson wrote:
> On Tue, Aug 27, 2013 at 05:25:23PM +0200, Sebastian Andrzej Siewior wrote:
>> On 08/27/2013 05:01 PM, Kevin Hilman wrote:
>>>>> What do we do now?
>>>>
>>>> Cannot you just merge the stable arm-soc/dt branch into your branch
>>>> before applying your patches?
>>>
>>> Unfortunately, the next/dt branch of arm-soc is not necessarily stable
>>> so should *not* be merged. In fact none of the arm-soc branches should
>>> be considered stable.
>>>
>>> As was already mentioned, this should be split up into driver changes
>>> and DTS changes through arm-soc. They'll both merge for v3.12.
>>
>> But splitting will break the driver until .dts & code is in sync again.
>>
>>> BTW, how did this patch get merged without a signoff/ack from the OMAP
>>> DT maintainer in the first place? Hmm, looks like Benoit was not copied
>>> nor was linux-omap or linux-arm-kernel copied in the original mails.
>>
>> Hmm. I had Benoit's okay [0] to do the change "as long as Felipe is
>> fine with it". I indeed forgot to Cc Benoit on the dts changes.
>> For the phy-rename Felipe pinged you and we did the topic-branch, here
>> I forgot.
>
> No. Read that email again. What Benoit said was that if Felipe was fine
> with the change _HE_ would take it. Huge difference, and one that would have
> avoided this situation.
>
> The only way to solve these things in the future is to make the driver handle
> both the new and the old binding. Bindings are not supposed to change in
> incompatible ways any more, unless for special circumstances and/or when the
> old binding was completely broken.
>
>
> The only way forward here, since Greg runs a stable tree that he doesn't
> rebase, is for us to rebuild without the OMAP DT branch, and ask Benoit to take
> out the conflicting changes.
>
> Benoit, I know this is none of your fault, but would you mind preparing a new
> copy of the DT branch without the conflicting patches, and hold those to 3.13?
> I haven't looked to see how many those were.
OK, I'll do that ASAP and check how many should be removed to avoid a
conflict with usb-next.
Regards,
Benoit
next prev parent reply other threads:[~2013-08-28 13:01 UTC|newest]
Thread overview: 109+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-27 8:13 linux-next: manual merge of the arm-soc tree with the usb tree Stephen Rothwell
2013-08-27 8:13 ` Stephen Rothwell
2013-08-27 8:13 ` Stephen Rothwell
2013-08-27 8:54 ` Sebastian Andrzej Siewior
2013-08-27 8:54 ` Sebastian Andrzej Siewior
2013-08-27 13:02 ` Javier Martinez Canillas
2013-08-27 13:02 ` Javier Martinez Canillas
2013-08-27 13:24 ` Benoit Cousson
2013-08-27 13:24 ` Benoit Cousson
2013-08-27 13:53 ` Sebastian Andrzej Siewior
2013-08-27 13:53 ` Sebastian Andrzej Siewior
2013-08-27 13:57 ` Benoit Cousson
2013-08-27 13:57 ` Benoit Cousson
2013-08-27 14:02 ` Sebastian Andrzej Siewior
2013-08-27 14:02 ` Sebastian Andrzej Siewior
2013-08-27 14:05 ` Benoit Cousson
2013-08-27 14:05 ` Benoit Cousson
2013-08-27 14:13 ` Sebastian Andrzej Siewior
2013-08-27 14:13 ` Sebastian Andrzej Siewior
2013-08-27 17:37 ` Greg KH
2013-08-27 17:37 ` Greg KH
2013-08-27 18:37 ` Felipe Balbi
2013-08-27 18:37 ` Felipe Balbi
2013-08-27 18:37 ` Felipe Balbi
2013-08-27 19:30 ` Greg KH
2013-08-27 19:30 ` Greg KH
2013-08-27 19:56 ` Felipe Balbi
2013-08-27 19:56 ` Felipe Balbi
2013-08-27 19:56 ` Felipe Balbi
2013-08-29 10:06 ` Benoit Cousson
2013-08-29 10:06 ` Benoit Cousson
2013-08-29 13:44 ` Javier Martinez Canillas
2013-08-29 13:44 ` Javier Martinez Canillas
2013-08-29 14:23 ` Felipe Balbi
2013-08-29 14:23 ` Felipe Balbi
2013-08-29 14:23 ` Felipe Balbi
2013-08-29 14:47 ` Benoit Cousson
2013-08-29 14:47 ` Benoit Cousson
2013-08-27 19:23 ` Sebastian Andrzej Siewior
2013-08-27 19:23 ` Sebastian Andrzej Siewior
2013-08-27 15:01 ` Kevin Hilman
2013-08-27 15:01 ` Kevin Hilman
2013-08-27 15:25 ` Sebastian Andrzej Siewior
2013-08-27 15:25 ` Sebastian Andrzej Siewior
2013-08-27 16:12 ` Olof Johansson
2013-08-27 16:12 ` Olof Johansson
2013-08-27 16:30 ` Sebastian Andrzej Siewior
2013-08-27 16:30 ` Sebastian Andrzej Siewior
2013-08-28 13:01 ` Benoit Cousson [this message]
2013-08-28 13:01 ` Benoit Cousson
2013-08-27 16:20 ` Kevin Hilman
2013-08-27 16:20 ` Kevin Hilman
-- strict thread matches above, loose matches on Subject: below --
2013-08-28 6:45 Stephen Rothwell
2013-08-28 6:45 ` Stephen Rothwell
2013-08-28 6:45 ` Stephen Rothwell
2013-08-28 17:26 ` Greg KH
2013-08-28 17:26 ` Greg KH
2013-08-27 8:24 Stephen Rothwell
2013-08-27 8:24 ` Stephen Rothwell
2013-08-27 8:24 ` Stephen Rothwell
2013-08-27 8:21 Stephen Rothwell
2013-08-27 8:21 ` Stephen Rothwell
2013-08-27 8:21 ` Stephen Rothwell
2013-08-27 8:18 Stephen Rothwell
2013-08-27 8:18 ` Stephen Rothwell
2013-08-27 8:18 ` Stephen Rothwell
2013-06-18 6:12 Stephen Rothwell
2013-06-18 6:12 ` Stephen Rothwell
2013-06-18 6:12 ` Stephen Rothwell
2013-06-18 12:56 ` Sergei Shtylyov
2013-06-18 12:56 ` Sergei Shtylyov
2013-06-18 16:03 ` Greg KH
2013-06-18 16:03 ` Greg KH
2013-02-12 3:59 Stephen Rothwell
2013-02-12 3:59 ` Stephen Rothwell
2013-02-12 3:59 ` Stephen Rothwell
2013-02-11 6:06 Stephen Rothwell
2013-02-11 6:06 ` Stephen Rothwell
2013-02-11 6:06 ` Stephen Rothwell
2013-02-11 6:05 Stephen Rothwell
2013-02-11 6:05 ` Stephen Rothwell
2013-02-11 6:05 ` Stephen Rothwell
2012-11-27 4:57 Stephen Rothwell
2012-11-27 4:57 ` Stephen Rothwell
2012-11-27 4:57 ` Stephen Rothwell
2012-11-27 8:37 ` Arnd Bergmann
2012-11-27 8:37 ` Arnd Bergmann
2012-11-13 4:20 Stephen Rothwell
2012-11-13 4:20 ` Stephen Rothwell
2012-11-13 4:20 ` Stephen Rothwell
2012-11-13 8:48 ` Nicolas Ferre
2012-11-13 8:48 ` Nicolas Ferre
2012-11-13 8:48 ` Nicolas Ferre
2012-11-13 4:06 Stephen Rothwell
2012-11-13 4:06 ` Stephen Rothwell
2012-11-13 4:06 ` Stephen Rothwell
2012-11-13 8:47 ` Nicolas Ferre
2012-11-13 8:47 ` Nicolas Ferre
2012-11-13 8:47 ` Nicolas Ferre
2012-09-25 6:56 Stephen Rothwell
2012-09-25 6:56 ` Stephen Rothwell
2012-09-25 6:56 ` Stephen Rothwell
2012-09-25 7:22 ` Tony Prisk
2012-09-25 7:22 ` Tony Prisk
2012-09-06 5:42 Stephen Rothwell
2012-09-06 5:42 ` Stephen Rothwell
2012-09-06 5:42 ` Stephen Rothwell
2012-09-06 9:32 ` Roland Stigge
2012-09-06 9:32 ` Roland Stigge
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=521DF4B2.6070205@baylibre.com \
--to=bcousson@baylibre.com \
--cc=arnd@arndb.de \
--cc=balbi@ti.com \
--cc=bigeasy@linutronix.de \
--cc=greg@kroah.com \
--cc=javier@dowhile0.org \
--cc=khilman@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-next@vger.kernel.org \
--cc=olof@lixom.net \
--cc=sfr@canb.auug.org.au \
/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.