All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tty: Remove leftover dependencies on PPC_OF
@ 2015-04-17  5:20 Guenter Roeck
  2015-04-17  6:01 ` Kevin Hao
  0 siblings, 1 reply; 8+ messages in thread
From: Guenter Roeck @ 2015-04-17  5:20 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: Jiri Slaby, linux-serial, linux-kernel, Guenter Roeck, Kevin Hao,
	Michael Ellerman

powerpc qemu runs fail with the current upstream kernel.
Bisect points to commit 52d996270032 ("powerpc: kill PPC_OF").
Unfortunately, that commit did not remove all instances of PPC_OF.
Practical impact is that the serial driver used by powerpc qemu
targets is no longer built into the test kernel.

Fixes: 52d996270032 ("powerpc: kill PPC_OF")
Cc: Kevin Hao <haokexin@gmail.com>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---
 drivers/tty/serial/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
index d2501f01cd03..77471d3db8d3 100644
--- a/drivers/tty/serial/Kconfig
+++ b/drivers/tty/serial/Kconfig
@@ -835,7 +835,7 @@ config SERIAL_MCF_CONSOLE
 
 config SERIAL_PMACZILOG
 	tristate "Mac or PowerMac z85c30 ESCC support"
-	depends on (M68K && MAC) || (PPC_OF && PPC_PMAC)
+	depends on (M68K && MAC) || PPC_PMAC
 	select SERIAL_CORE
 	help
 	  This driver supports the Zilog z85C30 serial ports found on
@@ -1153,7 +1153,7 @@ config SERIAL_OMAP_CONSOLE
 
 config SERIAL_OF_PLATFORM_NWPSERIAL
 	tristate "NWP serial port driver"
-	depends on PPC_OF && PPC_DCR
+	depends on PPC_DCR
 	select SERIAL_OF_PLATFORM
 	select SERIAL_CORE_CONSOLE
 	select SERIAL_CORE
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] tty: Remove leftover dependencies on PPC_OF
  2015-04-17  5:20 [PATCH] tty: Remove leftover dependencies on PPC_OF Guenter Roeck
@ 2015-04-17  6:01 ` Kevin Hao
  2015-04-17  6:17   ` Guenter Roeck
  2015-04-17  6:58   ` Grant Likely
  0 siblings, 2 replies; 8+ messages in thread
From: Kevin Hao @ 2015-04-17  6:01 UTC (permalink / raw)
  To: Guenter Roeck
  Cc: Greg Kroah-Hartman, Jiri Slaby, linux-serial, linux-kernel,
	Michael Ellerman

[-- Attachment #1: Type: text/plain, Size: 1807 bytes --]

On Thu, Apr 16, 2015 at 10:20:59PM -0700, Guenter Roeck wrote:
> powerpc qemu runs fail with the current upstream kernel.
> Bisect points to commit 52d996270032 ("powerpc: kill PPC_OF").
> Unfortunately, that commit did not remove all instances of PPC_OF.
> Practical impact is that the serial driver used by powerpc qemu
> targets is no longer built into the test kernel.

Sorry for the break. This is a dependency issue. The patch 213dce3c17a6
("tty: kconfig: remove the superfluous dependency on PPC_OF") has already
been merged into tty-next, but still not propagate to upstream yet. I failed
to reminder Michael of this when the pulling request is sent to Linus.

Thanks,
Kevin
> 
> Fixes: 52d996270032 ("powerpc: kill PPC_OF")
> Cc: Kevin Hao <haokexin@gmail.com>
> Cc: Michael Ellerman <mpe@ellerman.id.au>
> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
> ---
>  drivers/tty/serial/Kconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
> index d2501f01cd03..77471d3db8d3 100644
> --- a/drivers/tty/serial/Kconfig
> +++ b/drivers/tty/serial/Kconfig
> @@ -835,7 +835,7 @@ config SERIAL_MCF_CONSOLE
>  
>  config SERIAL_PMACZILOG
>  	tristate "Mac or PowerMac z85c30 ESCC support"
> -	depends on (M68K && MAC) || (PPC_OF && PPC_PMAC)
> +	depends on (M68K && MAC) || PPC_PMAC
>  	select SERIAL_CORE
>  	help
>  	  This driver supports the Zilog z85C30 serial ports found on
> @@ -1153,7 +1153,7 @@ config SERIAL_OMAP_CONSOLE
>  
>  config SERIAL_OF_PLATFORM_NWPSERIAL
>  	tristate "NWP serial port driver"
> -	depends on PPC_OF && PPC_DCR
> +	depends on PPC_DCR
>  	select SERIAL_OF_PLATFORM
>  	select SERIAL_CORE_CONSOLE
>  	select SERIAL_CORE
> -- 
> 2.1.0
> 

[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] tty: Remove leftover dependencies on PPC_OF
  2015-04-17  6:01 ` Kevin Hao
@ 2015-04-17  6:17   ` Guenter Roeck
  2015-04-17  7:00     ` Grant Likely
  2015-04-17  6:58   ` Grant Likely
  1 sibling, 1 reply; 8+ messages in thread
From: Guenter Roeck @ 2015-04-17  6:17 UTC (permalink / raw)
  To: Kevin Hao
  Cc: Greg Kroah-Hartman, Jiri Slaby, linux-serial, linux-kernel,
	Michael Ellerman

On 04/16/2015 11:01 PM, Kevin Hao wrote:
> On Thu, Apr 16, 2015 at 10:20:59PM -0700, Guenter Roeck wrote:
>> powerpc qemu runs fail with the current upstream kernel.
>> Bisect points to commit 52d996270032 ("powerpc: kill PPC_OF").
>> Unfortunately, that commit did not remove all instances of PPC_OF.
>> Practical impact is that the serial driver used by powerpc qemu
>> targets is no longer built into the test kernel.
>
> Sorry for the break. This is a dependency issue. The patch 213dce3c17a6
> ("tty: kconfig: remove the superfluous dependency on PPC_OF") has already
> been merged into tty-next, but still not propagate to upstream yet. I failed
> to reminder Michael of this when the pulling request is sent to Linus.
>

Guess that explains why I don't see the breakage in linux-next.

This kind of problem seems to be happening a lot in this commit window.

Is there a new mechanism in place which requires splitting such series
into multiple parts ? Personally I preferred the "old" style, where
the entire series would have been handled by one maintainer, with Acks
from the others.

Guenter


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] tty: Remove leftover dependencies on PPC_OF
  2015-04-17  6:01 ` Kevin Hao
  2015-04-17  6:17   ` Guenter Roeck
@ 2015-04-17  6:58   ` Grant Likely
  2015-04-17  8:14     ` Kevin Hao
  1 sibling, 1 reply; 8+ messages in thread
From: Grant Likely @ 2015-04-17  6:58 UTC (permalink / raw)
  To: Kevin Hao
  Cc: Guenter Roeck, Greg Kroah-Hartman, Jiri Slaby, linux-serial,
	Linux Kernel Mailing List, Michael Ellerman

On Fri, Apr 17, 2015 at 7:01 AM, Kevin Hao <haokexin@gmail.com> wrote:
> On Thu, Apr 16, 2015 at 10:20:59PM -0700, Guenter Roeck wrote:
>> powerpc qemu runs fail with the current upstream kernel.
>> Bisect points to commit 52d996270032 ("powerpc: kill PPC_OF").
>> Unfortunately, that commit did not remove all instances of PPC_OF.
>> Practical impact is that the serial driver used by powerpc qemu
>> targets is no longer built into the test kernel.
>
> Sorry for the break. This is a dependency issue. The patch 213dce3c17a6
> ("tty: kconfig: remove the superfluous dependency on PPC_OF") has already
> been merged into tty-next, but still not propagate to upstream yet. I failed
> to reminder Michael of this when the pulling request is sent to Linus.

This means you're doing it wrong. If there are dependencies between
patches, then they *must* all be in the same tree. You cannot depend
on the order that branches get merged because it breaks bisecting.

Or, to put it another way, every branch must compile on it's own
without needing to be merged into mainline or linux-next first.

There are two ways to handle this. The easiest is to commit all the
patches to a single branch. Then there is guaranteed to be no
problems. However, if the patches really need to be split up between
several branches, then the way to handle it is to put the prerequisite
patches into a new clean branch, and then merge that branch into each
tree that needs the prerequisite. That way every single tree includes
the required commit without committing it more than once or breaking
bisectability.

g.

>
> Thanks,
> Kevin
>>
>> Fixes: 52d996270032 ("powerpc: kill PPC_OF")
>> Cc: Kevin Hao <haokexin@gmail.com>
>> Cc: Michael Ellerman <mpe@ellerman.id.au>
>> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
>> ---
>>  drivers/tty/serial/Kconfig | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
>> index d2501f01cd03..77471d3db8d3 100644
>> --- a/drivers/tty/serial/Kconfig
>> +++ b/drivers/tty/serial/Kconfig
>> @@ -835,7 +835,7 @@ config SERIAL_MCF_CONSOLE
>>
>>  config SERIAL_PMACZILOG
>>       tristate "Mac or PowerMac z85c30 ESCC support"
>> -     depends on (M68K && MAC) || (PPC_OF && PPC_PMAC)
>> +     depends on (M68K && MAC) || PPC_PMAC
>>       select SERIAL_CORE
>>       help
>>         This driver supports the Zilog z85C30 serial ports found on
>> @@ -1153,7 +1153,7 @@ config SERIAL_OMAP_CONSOLE
>>
>>  config SERIAL_OF_PLATFORM_NWPSERIAL
>>       tristate "NWP serial port driver"
>> -     depends on PPC_OF && PPC_DCR
>> +     depends on PPC_DCR
>>       select SERIAL_OF_PLATFORM
>>       select SERIAL_CORE_CONSOLE
>>       select SERIAL_CORE
>> --
>> 2.1.0
>>

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] tty: Remove leftover dependencies on PPC_OF
  2015-04-17  6:17   ` Guenter Roeck
@ 2015-04-17  7:00     ` Grant Likely
  2015-04-17  7:43       ` Greg Kroah-Hartman
  0 siblings, 1 reply; 8+ messages in thread
From: Grant Likely @ 2015-04-17  7:00 UTC (permalink / raw)
  To: Guenter Roeck
  Cc: Kevin Hao, Greg Kroah-Hartman, Jiri Slaby, linux-serial,
	Linux Kernel Mailing List, Michael Ellerman

On Fri, Apr 17, 2015 at 7:17 AM, Guenter Roeck <linux@roeck-us.net> wrote:
> On 04/16/2015 11:01 PM, Kevin Hao wrote:
>>
>> On Thu, Apr 16, 2015 at 10:20:59PM -0700, Guenter Roeck wrote:
>>>
>>> powerpc qemu runs fail with the current upstream kernel.
>>> Bisect points to commit 52d996270032 ("powerpc: kill PPC_OF").
>>> Unfortunately, that commit did not remove all instances of PPC_OF.
>>> Practical impact is that the serial driver used by powerpc qemu
>>> targets is no longer built into the test kernel.
>>
>>
>> Sorry for the break. This is a dependency issue. The patch 213dce3c17a6
>> ("tty: kconfig: remove the superfluous dependency on PPC_OF") has already
>> been merged into tty-next, but still not propagate to upstream yet. I
>> failed
>> to reminder Michael of this when the pulling request is sent to Linus.
>>
>
> Guess that explains why I don't see the breakage in linux-next.
>
> This kind of problem seems to be happening a lot in this commit window.
>
> Is there a new mechanism in place which requires splitting such series
> into multiple parts ? Personally I preferred the "old" style, where
> the entire series would have been handled by one maintainer, with Acks
> from the others.

The rules haven't changed. Maintainers are doing the wrong thing. If a
series is split up into multiple parts, then maintainers *must*
coordinate to put the prerequisites into a single branch that can be
merged into each branch handling it. However, it is still almost
always better to just merge the entire series via a single tree.

Make noise whenever you see this kind of breakage because it means a
maintainer has done the wrong thing.

g.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] tty: Remove leftover dependencies on PPC_OF
  2015-04-17  7:00     ` Grant Likely
@ 2015-04-17  7:43       ` Greg Kroah-Hartman
  2015-04-17  8:19         ` Grant Likely
  0 siblings, 1 reply; 8+ messages in thread
From: Greg Kroah-Hartman @ 2015-04-17  7:43 UTC (permalink / raw)
  To: Grant Likely
  Cc: Guenter Roeck, Kevin Hao, Jiri Slaby, linux-serial,
	Linux Kernel Mailing List, Michael Ellerman

On Fri, Apr 17, 2015 at 08:00:45AM +0100, Grant Likely wrote:
> On Fri, Apr 17, 2015 at 7:17 AM, Guenter Roeck <linux@roeck-us.net> wrote:
> > On 04/16/2015 11:01 PM, Kevin Hao wrote:
> >>
> >> On Thu, Apr 16, 2015 at 10:20:59PM -0700, Guenter Roeck wrote:
> >>>
> >>> powerpc qemu runs fail with the current upstream kernel.
> >>> Bisect points to commit 52d996270032 ("powerpc: kill PPC_OF").
> >>> Unfortunately, that commit did not remove all instances of PPC_OF.
> >>> Practical impact is that the serial driver used by powerpc qemu
> >>> targets is no longer built into the test kernel.
> >>
> >>
> >> Sorry for the break. This is a dependency issue. The patch 213dce3c17a6
> >> ("tty: kconfig: remove the superfluous dependency on PPC_OF") has already
> >> been merged into tty-next, but still not propagate to upstream yet. I
> >> failed
> >> to reminder Michael of this when the pulling request is sent to Linus.
> >>
> >
> > Guess that explains why I don't see the breakage in linux-next.
> >
> > This kind of problem seems to be happening a lot in this commit window.
> >
> > Is there a new mechanism in place which requires splitting such series
> > into multiple parts ? Personally I preferred the "old" style, where
> > the entire series would have been handled by one maintainer, with Acks
> > from the others.
> 
> The rules haven't changed. Maintainers are doing the wrong thing. If a
> series is split up into multiple parts, then maintainers *must*
> coordinate to put the prerequisites into a single branch that can be
> merged into each branch handling it. However, it is still almost
> always better to just merge the entire series via a single tree.
> 
> Make noise whenever you see this kind of breakage because it means a
> maintainer has done the wrong thing.

Well, the maintainer needs to be _told_ that the patch that is being
sent to them shouldn't go through their tree and that it depends on
other patches, so that they can properly just ack them.

Which is what happened here, someone sent me a patch, and I applied it.
Nothing broke that I could determine, and I never got a report of
something breaking, so how am I, the maintainer, doing the wrong thing?

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] tty: Remove leftover dependencies on PPC_OF
  2015-04-17  6:58   ` Grant Likely
@ 2015-04-17  8:14     ` Kevin Hao
  0 siblings, 0 replies; 8+ messages in thread
From: Kevin Hao @ 2015-04-17  8:14 UTC (permalink / raw)
  To: Grant Likely
  Cc: Guenter Roeck, Greg Kroah-Hartman, Jiri Slaby, linux-serial,
	Linux Kernel Mailing List, Michael Ellerman

[-- Attachment #1: Type: text/plain, Size: 3564 bytes --]

On Fri, Apr 17, 2015 at 07:58:10AM +0100, Grant Likely wrote:
> On Fri, Apr 17, 2015 at 7:01 AM, Kevin Hao <haokexin@gmail.com> wrote:
> > On Thu, Apr 16, 2015 at 10:20:59PM -0700, Guenter Roeck wrote:
> >> powerpc qemu runs fail with the current upstream kernel.
> >> Bisect points to commit 52d996270032 ("powerpc: kill PPC_OF").
> >> Unfortunately, that commit did not remove all instances of PPC_OF.
> >> Practical impact is that the serial driver used by powerpc qemu
> >> targets is no longer built into the test kernel.
> >
> > Sorry for the break. This is a dependency issue. The patch 213dce3c17a6
> > ("tty: kconfig: remove the superfluous dependency on PPC_OF") has already
> > been merged into tty-next, but still not propagate to upstream yet. I failed
> > to reminder Michael of this when the pulling request is sent to Linus.
> 
> This means you're doing it wrong.

Definitely.

> If there are dependencies between
> patches, then they *must* all be in the same tree. You cannot depend
> on the order that branches get merged because it breaks bisecting.

We don't try to depend on the order that branches get merged. We just want to
merge the patch which kill the PPC_OF completely only after all the patches for
the specific drivers have been merged. I should took the responsibility to
make sure of this, but of course I missed something. Yes. putting all the
patches into the same tree should be a better plan. I will make sure of this
next time. :-)

> 
> Or, to put it another way, every branch must compile on it's own
> without needing to be merged into mainline or linux-next first.

This issue doesn't break compile. That is another reason we didn't find it
earlier.

Thanks,
Kevin

> 
> There are two ways to handle this. The easiest is to commit all the
> patches to a single branch. Then there is guaranteed to be no
> problems. However, if the patches really need to be split up between
> several branches, then the way to handle it is to put the prerequisite
> patches into a new clean branch, and then merge that branch into each
> tree that needs the prerequisite. That way every single tree includes
> the required commit without committing it more than once or breaking
> bisectability.
> 
> g.
> 
> >
> > Thanks,
> > Kevin
> >>
> >> Fixes: 52d996270032 ("powerpc: kill PPC_OF")
> >> Cc: Kevin Hao <haokexin@gmail.com>
> >> Cc: Michael Ellerman <mpe@ellerman.id.au>
> >> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
> >> ---
> >>  drivers/tty/serial/Kconfig | 4 ++--
> >>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
> >> index d2501f01cd03..77471d3db8d3 100644
> >> --- a/drivers/tty/serial/Kconfig
> >> +++ b/drivers/tty/serial/Kconfig
> >> @@ -835,7 +835,7 @@ config SERIAL_MCF_CONSOLE
> >>
> >>  config SERIAL_PMACZILOG
> >>       tristate "Mac or PowerMac z85c30 ESCC support"
> >> -     depends on (M68K && MAC) || (PPC_OF && PPC_PMAC)
> >> +     depends on (M68K && MAC) || PPC_PMAC
> >>       select SERIAL_CORE
> >>       help
> >>         This driver supports the Zilog z85C30 serial ports found on
> >> @@ -1153,7 +1153,7 @@ config SERIAL_OMAP_CONSOLE
> >>
> >>  config SERIAL_OF_PLATFORM_NWPSERIAL
> >>       tristate "NWP serial port driver"
> >> -     depends on PPC_OF && PPC_DCR
> >> +     depends on PPC_DCR
> >>       select SERIAL_OF_PLATFORM
> >>       select SERIAL_CORE_CONSOLE
> >>       select SERIAL_CORE
> >> --
> >> 2.1.0
> >>

[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] tty: Remove leftover dependencies on PPC_OF
  2015-04-17  7:43       ` Greg Kroah-Hartman
@ 2015-04-17  8:19         ` Grant Likely
  0 siblings, 0 replies; 8+ messages in thread
From: Grant Likely @ 2015-04-17  8:19 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: Guenter Roeck, Kevin Hao, Jiri Slaby, linux-serial,
	Linux Kernel Mailing List, Michael Ellerman

On Fri, Apr 17, 2015 at 8:43 AM, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
> On Fri, Apr 17, 2015 at 08:00:45AM +0100, Grant Likely wrote:
>> On Fri, Apr 17, 2015 at 7:17 AM, Guenter Roeck <linux@roeck-us.net> wrote:
>> > On 04/16/2015 11:01 PM, Kevin Hao wrote:
>> >>
>> >> On Thu, Apr 16, 2015 at 10:20:59PM -0700, Guenter Roeck wrote:
>> >>>
>> >>> powerpc qemu runs fail with the current upstream kernel.
>> >>> Bisect points to commit 52d996270032 ("powerpc: kill PPC_OF").
>> >>> Unfortunately, that commit did not remove all instances of PPC_OF.
>> >>> Practical impact is that the serial driver used by powerpc qemu
>> >>> targets is no longer built into the test kernel.
>> >>
>> >>
>> >> Sorry for the break. This is a dependency issue. The patch 213dce3c17a6
>> >> ("tty: kconfig: remove the superfluous dependency on PPC_OF") has already
>> >> been merged into tty-next, but still not propagate to upstream yet. I
>> >> failed
>> >> to reminder Michael of this when the pulling request is sent to Linus.
>> >>
>> >
>> > Guess that explains why I don't see the breakage in linux-next.
>> >
>> > This kind of problem seems to be happening a lot in this commit window.
>> >
>> > Is there a new mechanism in place which requires splitting such series
>> > into multiple parts ? Personally I preferred the "old" style, where
>> > the entire series would have been handled by one maintainer, with Acks
>> > from the others.
>>
>> The rules haven't changed. Maintainers are doing the wrong thing. If a
>> series is split up into multiple parts, then maintainers *must*
>> coordinate to put the prerequisites into a single branch that can be
>> merged into each branch handling it. However, it is still almost
>> always better to just merge the entire series via a single tree.
>>
>> Make noise whenever you see this kind of breakage because it means a
>> maintainer has done the wrong thing.
>
> Well, the maintainer needs to be _told_ that the patch that is being
> sent to them shouldn't go through their tree and that it depends on
> other patches, so that they can properly just ack them.
>
> Which is what happened here, someone sent me a patch, and I applied it.
> Nothing broke that I could determine, and I never got a report of
> something breaking, so how am I, the maintainer, doing the wrong thing?

My apologies, Yes of course. s/a maintainer/someone/

g.

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2015-04-17  8:19 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-17  5:20 [PATCH] tty: Remove leftover dependencies on PPC_OF Guenter Roeck
2015-04-17  6:01 ` Kevin Hao
2015-04-17  6:17   ` Guenter Roeck
2015-04-17  7:00     ` Grant Likely
2015-04-17  7:43       ` Greg Kroah-Hartman
2015-04-17  8:19         ` Grant Likely
2015-04-17  6:58   ` Grant Likely
2015-04-17  8:14     ` Kevin Hao

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.