linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* ppc32 power3/4 pmac_cache.S build breakage fix.
@ 2005-03-27 12:37 Sven Luther
  2005-03-27 22:44 ` Benjamin Herrenschmidt
  0 siblings, 1 reply; 3+ messages in thread
From: Sven Luther @ 2005-03-27 12:37 UTC (permalink / raw)
  To: linuxppc-dev, benh

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

Hello,

Find attached a trivial fix for the 32bit power3 and power4 introduced by
benh's latest round of pmac sleep fixes.

I am not sure if this is the best way to solve this problem, but it fixes the
issue.

Friendly,

Sven Luther

[-- Attachment #2: pmac-cache.diff --]
[-- Type: text/plain, Size: 727 bytes --]

--- kernel-source-2.6.11/arch/ppc/platforms/Makefile.orig	2005-03-27 11:38:25.000000000 +0200
+++ kernel-source-2.6.11/arch/ppc/platforms/Makefile	2005-03-27 11:39:23.000000000 +0200
@@ -9,9 +9,15 @@
 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
 obj-$(CONFIG_PPC_PREP)		+= prep_pci.o prep_setup.o

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

* Re: ppc32 power3/4 pmac_cache.S build breakage fix.
  2005-03-27 12:37 ppc32 power3/4 pmac_cache.S build breakage fix Sven Luther
@ 2005-03-27 22:44 ` Benjamin Herrenschmidt
  2005-03-28  7:17   ` Sven Luther
  0 siblings, 1 reply; 3+ messages in thread
From: Benjamin Herrenschmidt @ 2005-03-27 22:44 UTC (permalink / raw)
  To: Sven Luther; +Cc: linuxppc-dev list

On Sun, 2005-03-27 at 14:37 +0200, Sven Luther wrote:
> Hello,
> 
> Find attached a trivial fix for the 32bit power3 and power4 introduced by
> benh's latest round of pmac sleep fixes.
> 
> I am not sure if this is the best way to solve this problem, but it fixes the
> issue.

Bah, fixe plustot dans le code ... ajout un #ifdef CONFIG_ALTIVEC autour
du dssall (enfin, de la section entiere avec le BEGIN/END inclus) pour
le moment, quoi que ici, le mieux est encore de dire a l'asm de pas
moufter. En ce qui concerne pmac_sleep vs. pmac_cache, #ifdef aussi pour
le moment.

Ben.

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

* Re: ppc32 power3/4 pmac_cache.S build breakage fix.
  2005-03-27 22:44 ` Benjamin Herrenschmidt
@ 2005-03-28  7:17   ` Sven Luther
  0 siblings, 0 replies; 3+ messages in thread
From: Sven Luther @ 2005-03-28  7:17 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: linuxppc-dev list

On Mon, Mar 28, 2005 at 08:44:47AM +1000, Benjamin Herrenschmidt wrote:
> On Sun, 2005-03-27 at 14:37 +0200, Sven Luther wrote:
> > Hello,
> > 
> > Find attached a trivial fix for the 32bit power3 and power4 introduced by
> > benh's latest round of pmac sleep fixes.
> > 
> > I am not sure if this is the best way to solve this problem, but it fixes the
> > issue.
> 
> Bah, fixe plustot dans le code ... ajout un #ifdef CONFIG_ALTIVEC autour
> du dssall (enfin, de la section entiere avec le BEGIN/END inclus) pour
> le moment, quoi que ici, le mieux est encore de dire a l'asm de pas
> moufter. En ce qui concerne pmac_sleep vs. pmac_cache, #ifdef aussi pour
> le moment.

Le truc, c'est que l'ensemble du code de pmac_cache, il ne devrait pas etre
compiler si on est pas CONFIG_6xx, car la fonction n'est jamais appelle depuis
pmac_sleep si on est pas CONFIG_6xx.

Je te laisse gerer cela pour upstream ? 

Amicalement,

Sven Luther

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

end of thread, other threads:[~2005-03-28  7:19 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-03-27 12:37 ppc32 power3/4 pmac_cache.S build breakage fix Sven Luther
2005-03-27 22:44 ` Benjamin Herrenschmidt
2005-03-28  7:17   ` 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).