linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [patch 38/43] Various powerpc 32bit ppc64 build fixes
@ 2005-10-29  0:46 akpm
  2005-10-29  5:41 ` Paul Mackerras
  0 siblings, 1 reply; 3+ messages in thread
From: akpm @ 2005-10-29  0:46 UTC (permalink / raw)
  To: paulus; +Cc: akpm, linuxppc-dev, luther, linuxppc64-dev


From: Sven Luther <sven.luther@wanadoo.fr>

Find here attached two small build fixes for the 32bit ppc64 kernels.  I
know the support for those kernels will soon go away, and debian is not
using them anymore, but until then it is nice to not have it broken, since
other users may need it.

Signed-off-by: Sven Luther <luther@debian.org>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
---

 arch/ppc/boot/simple/misc-prep.c |    2 ++
 arch/ppc/platforms/Makefile      |    6 ++++++
 2 files changed, 8 insertions(+)

diff -puN arch/ppc/boot/simple/misc-prep.c~various-powerpc-32bit-ppc64-build-fixes arch/ppc/boot/simple/misc-prep.c
--- devel/arch/ppc/boot/simple/misc-prep.c~various-powerpc-32bit-ppc64-build-fixes	2005-10-28 17:44:04.000000000 -0700
+++ devel-akpm/arch/ppc/boot/simple/misc-prep.c	2005-10-28 17:44:04.000000000 -0700
@@ -152,9 +152,11 @@ load_kernel(unsigned long load_addr, int
 		hold_residual->VitalProductData.Reserved5 = 0xdeadbeef;
 	}
 
+#if defined(CONFIG_6xx)
 	/* Now go and clear out the BATs and ensure that our MSR is
 	 * correct .*/
 	disable_6xx_mmu();
+#endif
 
 	/* Make r3 be a pointer to the residual data. */
 	return (unsigned long)hold_residual;
diff -puN arch/ppc/platforms/Makefile~various-powerpc-32bit-ppc64-build-fixes arch/ppc/platforms/Makefile
--- devel/arch/ppc/platforms/Makefile~various-powerpc-32bit-ppc64-build-fixes	2005-10-28 17:44:04.000000000 -0700
+++ devel-akpm/arch/ppc/platforms/Makefile	2005-10-28 17:44:04.000000000 -0700
@@ -9,9 +9,15 @@ obj-$(CONFIG_APUS)		+= apus_setup.o
 ifeq ($(CONFIG_APUS),y)
 obj-$(CONFIG_PCI)		+= apus_pci.o
 endif
+ifeq ($(CONFIG_6xx),y)
 obj-$(CONFIG_PPC_PMAC)		+= pmac_pic.o pmac_setup.o pmac_time.o \
 					pmac_feature.o pmac_pci.o pmac_sleep.o \
 					pmac_low_i2c.o pmac_cache.o
+else
+obj-$(CONFIG_PPC_PMAC)		+= pmac_pic.o pmac_setup.o pmac_time.o \
+					pmac_feature.o pmac_pci.o pmac_sleep.o \
+					pmac_low_i2c.o
+endif
 obj-$(CONFIG_PPC_CHRP)		+= chrp_setup.o chrp_time.o chrp_pci.o \
 					chrp_pegasos_eth.o
 ifeq ($(CONFIG_PPC_CHRP),y)
_

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

* Re: [patch 38/43] Various powerpc 32bit ppc64 build fixes
  2005-10-29  0:46 [patch 38/43] Various powerpc 32bit ppc64 build fixes akpm
@ 2005-10-29  5:41 ` Paul Mackerras
  2005-10-29  6:27   ` Sven Luther
  0 siblings, 1 reply; 3+ messages in thread
From: Paul Mackerras @ 2005-10-29  5:41 UTC (permalink / raw)
  To: sven.luther; +Cc: akpm, linuxppc-dev, luther, linuxppc64-dev

Sven Luther writes:

> +ifeq ($(CONFIG_6xx),y)
>  obj-$(CONFIG_PPC_PMAC)		+= pmac_pic.o pmac_setup.o pmac_time.o \
>  					pmac_feature.o pmac_pci.o pmac_sleep.o \
>  					pmac_low_i2c.o pmac_cache.o
> +else
> +obj-$(CONFIG_PPC_PMAC)		+= pmac_pic.o pmac_setup.o pmac_time.o \
> +					pmac_feature.o pmac_pci.o pmac_sleep.o \
> +					pmac_low_i2c.o
> +endif

This is a bit gross.  How about:

pmac-$(CONFIG_6xx)		:= pmac_cache.o
obj-$(CONFIG_PPC_PMAC)		+= pmac_pic.o pmac_setup.o pmac_time.o \
					pmac_feature.o pmac_pci.o pmac_sleep.o \
					pmac_low_i2c.o $(pmac-y)

Care to send a revised patch?

Paul.

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

* Re: [patch 38/43] Various powerpc 32bit ppc64 build fixes
  2005-10-29  5:41 ` Paul Mackerras
@ 2005-10-29  6:27   ` Sven Luther
  0 siblings, 0 replies; 3+ messages in thread
From: Sven Luther @ 2005-10-29  6:27 UTC (permalink / raw)
  To: Paul Mackerras; +Cc: akpm, linuxppc-dev, luther, linuxppc64-dev

On Sat, Oct 29, 2005 at 03:41:59PM +1000, Paul Mackerras wrote:
> Sven Luther writes:
> 
> > +ifeq ($(CONFIG_6xx),y)
> >  obj-$(CONFIG_PPC_PMAC)		+= pmac_pic.o pmac_setup.o pmac_time.o \
> >  					pmac_feature.o pmac_pci.o pmac_sleep.o \
> >  					pmac_low_i2c.o pmac_cache.o
> > +else
> > +obj-$(CONFIG_PPC_PMAC)		+= pmac_pic.o pmac_setup.o pmac_time.o \
> > +					pmac_feature.o pmac_pci.o pmac_sleep.o \
> > +					pmac_low_i2c.o
> > +endif
> 
> This is a bit gross.  How about:

Well, i sent to linuxppc-dev for comments too :)

> pmac-$(CONFIG_6xx)		:= pmac_cache.o
> obj-$(CONFIG_PPC_PMAC)		+= pmac_pic.o pmac_setup.o pmac_time.o \
> 					pmac_feature.o pmac_pci.o pmac_sleep.o \
> 					pmac_low_i2c.o $(pmac-y)
> 
> Care to send a revised patch?

Sure, i will do so early next week.

Friendly,

Sven Luther

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

end of thread, other threads:[~2005-10-29  6:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-10-29  0:46 [patch 38/43] Various powerpc 32bit ppc64 build fixes akpm
2005-10-29  5:41 ` Paul Mackerras
2005-10-29  6:27   ` Sven Luther

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).