* [PATCH] [POWERPC] Always build setup-bus.c on powerpc
@ 2008-01-24 6:07 Kumar Gala
2008-01-24 6:28 ` Benjamin Herrenschmidt
2008-01-24 6:29 ` [PATCH] [POWERPC] " Greg KH
0 siblings, 2 replies; 6+ messages in thread
From: Kumar Gala @ 2008-01-24 6:07 UTC (permalink / raw)
To: Greg KH; +Cc: linuxppc-dev, linux-pci, Paul Mackerras, linux-kernel
The common arch/powerpc code calls in to functions in setup-bus.c
so some builds of ppc32 would fail.
Note, ppc32 usage of setup-irq.c is limited to arch/ppc and should be
removed when arch/ppc goes away.
Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
---
Greg, recent changes that BenH has made to the arch/powerpc pci code make
this necessary. If you don't have an issue I'd like this patch to go via
the paulus's powerpc.git tree.
- k
drivers/pci/Makefile | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/pci/Makefile b/drivers/pci/Makefile
index 945bf02..869d689 100644
--- a/drivers/pci/Makefile
+++ b/drivers/pci/Makefile
@@ -31,8 +31,8 @@ obj-$(CONFIG_ALPHA) += setup-bus.o setup-irq.o
obj-$(CONFIG_ARM) += setup-bus.o setup-irq.o
obj-$(CONFIG_PARISC) += setup-bus.o
obj-$(CONFIG_SUPERH) += setup-bus.o setup-irq.o
-obj-$(CONFIG_PPC32) += setup-irq.o setup-bus.o
-obj-$(CONFIG_PPC64) += setup-bus.o
+obj-$(CONFIG_PPC) += setup-bus.o
+obj-$(CONFIG_PPC32) += setup-irq.o
obj-$(CONFIG_MIPS) += setup-bus.o setup-irq.o
obj-$(CONFIG_X86_VISWS) += setup-irq.o
--
1.5.3.7
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] [POWERPC] Always build setup-bus.c on powerpc
2008-01-24 6:07 [PATCH] [POWERPC] Always build setup-bus.c on powerpc Kumar Gala
@ 2008-01-24 6:28 ` Benjamin Herrenschmidt
2008-01-24 6:33 ` Kumar Gala
2008-01-24 6:29 ` [PATCH] [POWERPC] " Greg KH
1 sibling, 1 reply; 6+ messages in thread
From: Benjamin Herrenschmidt @ 2008-01-24 6:28 UTC (permalink / raw)
To: Kumar Gala; +Cc: Greg KH, linux-pci, Paul Mackerras, linux-kernel, linuxppc-dev
On Thu, 2008-01-24 at 00:07 -0600, Kumar Gala wrote:
> The common arch/powerpc code calls in to functions in setup-bus.c
> so some builds of ppc32 would fail.
>
> Note, ppc32 usage of setup-irq.c is limited to arch/ppc and should be
> removed when arch/ppc goes away.
I don't understand... the old code would build setup-bus.o for both
PPC32 and PPC64 cases, how did you make it fail ?
Ben.
> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
> ---
>
> Greg, recent changes that BenH has made to the arch/powerpc pci code make
> this necessary. If you don't have an issue I'd like this patch to go via
> the paulus's powerpc.git tree.
>
> - k
>
> drivers/pci/Makefile | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pci/Makefile b/drivers/pci/Makefile
> index 945bf02..869d689 100644
> --- a/drivers/pci/Makefile
> +++ b/drivers/pci/Makefile
> @@ -31,8 +31,8 @@ obj-$(CONFIG_ALPHA) += setup-bus.o setup-irq.o
> obj-$(CONFIG_ARM) += setup-bus.o setup-irq.o
> obj-$(CONFIG_PARISC) += setup-bus.o
> obj-$(CONFIG_SUPERH) += setup-bus.o setup-irq.o
> -obj-$(CONFIG_PPC32) += setup-irq.o setup-bus.o
> -obj-$(CONFIG_PPC64) += setup-bus.o
> +obj-$(CONFIG_PPC) += setup-bus.o
> +obj-$(CONFIG_PPC32) += setup-irq.o
> obj-$(CONFIG_MIPS) += setup-bus.o setup-irq.o
> obj-$(CONFIG_X86_VISWS) += setup-irq.o
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] [POWERPC] Always build setup-bus.c on powerpc
2008-01-24 6:28 ` Benjamin Herrenschmidt
@ 2008-01-24 6:33 ` Kumar Gala
2008-01-24 6:38 ` [PATCH v2][POWERPC] " Kumar Gala
0 siblings, 1 reply; 6+ messages in thread
From: Kumar Gala @ 2008-01-24 6:33 UTC (permalink / raw)
To: benh; +Cc: Greg KH, linux-pci, Paul Mackerras, linux-kernel, linuxppc-dev
On Jan 24, 2008, at 12:28 AM, Benjamin Herrenschmidt wrote:
>
> On Thu, 2008-01-24 at 00:07 -0600, Kumar Gala wrote:
>> The common arch/powerpc code calls in to functions in setup-bus.c
>> so some builds of ppc32 would fail.
>>
>> Note, ppc32 usage of setup-irq.c is limited to arch/ppc and should be
>> removed when arch/ppc goes away.
>
> I don't understand... the old code would build setup-bus.o for both
> PPC32 and PPC64 cases, how did you make it fail ?
The patch is bogus. The old makefile rules looked like:
obj-$(CONFIG_PPC64) += setup-bus.o
obj-$(CONFIG_PPC32) += setup-irq.o
pmac most like builds because CONFIG_HOTPLUG pulls in setup-bus.o
(I'll fix my foobar'd patch and send this via Paul).
- k
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2][POWERPC] Always build setup-bus.c on powerpc
2008-01-24 6:33 ` Kumar Gala
@ 2008-01-24 6:38 ` Kumar Gala
2008-01-24 7:06 ` Benjamin Herrenschmidt
0 siblings, 1 reply; 6+ messages in thread
From: Kumar Gala @ 2008-01-24 6:38 UTC (permalink / raw)
To: benh; +Cc: Greg KH, linux-pci, Paul Mackerras, linux-kernel, linuxppc-dev
The common arch/powerpc code calls in to functions in setup-bus.c
so some builds of ppc32 would fail.
Note, ppc32 usage of setup-irq.c is limited to arch/ppc and should be
removed when arch/ppc goes away.
Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
Here's the proper diff, will send this via paulus.
drivers/pci/Makefile | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/pci/Makefile b/drivers/pci/Makefile
index 5550556..f697f3d 100644
--- a/drivers/pci/Makefile
+++ b/drivers/pci/Makefile
@@ -32,7 +32,7 @@ obj-$(CONFIG_ARM) += setup-bus.o setup-irq.o
obj-$(CONFIG_PARISC) += setup-bus.o
obj-$(CONFIG_SUPERH) += setup-bus.o setup-irq.o
obj-$(CONFIG_PPC32) += setup-irq.o
-obj-$(CONFIG_PPC64) += setup-bus.o
+obj-$(CONFIG_PPC) += setup-bus.o
obj-$(CONFIG_MIPS) += setup-bus.o setup-irq.o
obj-$(CONFIG_X86_VISWS) += setup-irq.o
--
1.5.3.7
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2][POWERPC] Always build setup-bus.c on powerpc
2008-01-24 6:38 ` [PATCH v2][POWERPC] " Kumar Gala
@ 2008-01-24 7:06 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 6+ messages in thread
From: Benjamin Herrenschmidt @ 2008-01-24 7:06 UTC (permalink / raw)
To: Kumar Gala; +Cc: Greg KH, linux-pci, Paul Mackerras, linux-kernel, linuxppc-dev
On Thu, 2008-01-24 at 00:38 -0600, Kumar Gala wrote:
> The common arch/powerpc code calls in to functions in setup-bus.c
> so some builds of ppc32 would fail.
>
> Note, ppc32 usage of setup-irq.c is limited to arch/ppc and should be
> removed when arch/ppc goes away.
>
> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Ack. Funny I didn't catch it on my 4xx testing, I must have always had
HOTPLUG enabled.
> ---
>
> Here's the proper diff, will send this via paulus.
>
> drivers/pci/Makefile | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/pci/Makefile b/drivers/pci/Makefile
> index 5550556..f697f3d 100644
> --- a/drivers/pci/Makefile
> +++ b/drivers/pci/Makefile
> @@ -32,7 +32,7 @@ obj-$(CONFIG_ARM) += setup-bus.o setup-irq.o
> obj-$(CONFIG_PARISC) += setup-bus.o
> obj-$(CONFIG_SUPERH) += setup-bus.o setup-irq.o
> obj-$(CONFIG_PPC32) += setup-irq.o
> -obj-$(CONFIG_PPC64) += setup-bus.o
> +obj-$(CONFIG_PPC) += setup-bus.o
> obj-$(CONFIG_MIPS) += setup-bus.o setup-irq.o
> obj-$(CONFIG_X86_VISWS) += setup-irq.o
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] [POWERPC] Always build setup-bus.c on powerpc
2008-01-24 6:07 [PATCH] [POWERPC] Always build setup-bus.c on powerpc Kumar Gala
2008-01-24 6:28 ` Benjamin Herrenschmidt
@ 2008-01-24 6:29 ` Greg KH
1 sibling, 0 replies; 6+ messages in thread
From: Greg KH @ 2008-01-24 6:29 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev, linux-pci, Paul Mackerras, linux-kernel
On Thu, Jan 24, 2008 at 12:07:32AM -0600, Kumar Gala wrote:
> The common arch/powerpc code calls in to functions in setup-bus.c
> so some builds of ppc32 would fail.
>
> Note, ppc32 usage of setup-irq.c is limited to arch/ppc and should be
> removed when arch/ppc goes away.
>
> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
> ---
>
> Greg, recent changes that BenH has made to the arch/powerpc pci code make
> this necessary. If you don't have an issue I'd like this patch to go via
> the paulus's powerpc.git tree.
No objection at all. Feel free to add my:
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
to this patch and send it through Paul.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2008-01-24 7:07 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-01-24 6:07 [PATCH] [POWERPC] Always build setup-bus.c on powerpc Kumar Gala
2008-01-24 6:28 ` Benjamin Herrenschmidt
2008-01-24 6:33 ` Kumar Gala
2008-01-24 6:38 ` [PATCH v2][POWERPC] " Kumar Gala
2008-01-24 7:06 ` Benjamin Herrenschmidt
2008-01-24 6:29 ` [PATCH] [POWERPC] " Greg KH
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).