linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Fix sleep/wakeup problems on iBook G3
@ 2005-03-27 22:55 Benjamin Herrenschmidt
  2005-03-29 19:07 ` Graham Wilson
  0 siblings, 1 reply; 4+ messages in thread
From: Benjamin Herrenschmidt @ 2005-03-27 22:55 UTC (permalink / raw)
  To: linuxppc-dev list, debian-powerpc@lists.debian.org

Hi !

There have been various reports lately of sleep beeing broken on iBook
G3. Frank (no name provided here) found a typo in my code that cleans
the L1 cache on these CPU models. This patch fixes it, I'd appreciate
some feedback about wether that fixes the problem or not.

Ben.

Index: linux-work/arch/ppc/platforms/pmac_cache.S
===================================================================
--- linux-work.orig/arch/ppc/platforms/pmac_cache.S	2005-03-15 11:56:42.000000000 +1100
+++ linux-work/arch/ppc/platforms/pmac_cache.S	2005-03-28 08:51:46.000000000 +1000
@@ -73,7 +73,7 @@
 
 	/* disable / invalidate / enable L1 data */
 	mfspr	r3,SPRN_HID0
-	rlwinm	r0,r0,0,~HID0_DCE
+	rlwinm	r3,r3,0,~HID0_DCE
 	mtspr	SPRN_HID0,r3
 	sync
 	isync

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

* Re: [PATCH] Fix sleep/wakeup problems on iBook G3
  2005-03-27 22:55 [PATCH] Fix sleep/wakeup problems on iBook G3 Benjamin Herrenschmidt
@ 2005-03-29 19:07 ` Graham Wilson
  2005-03-29 22:04   ` Benjamin Herrenschmidt
  0 siblings, 1 reply; 4+ messages in thread
From: Graham Wilson @ 2005-03-29 19:07 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: linuxppc-dev, debian-powerpc

On Mon, Mar 28, 2005 at 08:55:11AM +1000, Benjamin Herrenschmidt wrote:
> There have been various reports lately of sleep beeing broken on iBook
> G3. Frank (no name provided here) found a typo in my code that cleans
> the L1 cache on these CPU models. This patch fixes it, I'd appreciate
> some feedback about wether that fixes the problem or not.

I've been having various trouble with my iBook lately, including random
freezes, not waking up after going to sleep sometimes, and sometimes not
going to sleep. Since applying the patch you posted, everything seems
fine, though I may be speaking too soon.

My /proc/cpuinfo for reference:
 processor	: 0
 cpu		: 750FX
 temperature 	: 0-3 C (uncalibrated)
 clock		: 400MHz
 revision	: 2.3 (pvr 7000 0203)
 bogomips	: 793.71
 machine		: PowerBook4,3
 motherboard	: PowerBook4,3 MacRISC2 MacRISC Power Macintosh
 detected as	: 257 (iBook 2 rev. 2)
 pmac flags	: 0000001b
 L2 cache	: 512K unified
 memory		: 256MB
 pmac-generation	: NewWorld

Note that the maximum CPU speed is 900 MHz.

-- 
gram

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

* Re: [PATCH] Fix sleep/wakeup problems on iBook G3
  2005-03-29 19:07 ` Graham Wilson
@ 2005-03-29 22:04   ` Benjamin Herrenschmidt
  2005-03-29 22:45     ` Graham Wilson
  0 siblings, 1 reply; 4+ messages in thread
From: Benjamin Herrenschmidt @ 2005-03-29 22:04 UTC (permalink / raw)
  To: Graham Wilson; +Cc: linuxppc-dev list, debian-powerpc@lists.debian.org


> I've been having various trouble with my iBook lately, including random
> freezes, not waking up after going to sleep sometimes, and sometimes not
> going to sleep. Since applying the patch you posted, everything seems
> fine, though I may be speaking too soon.
> 
> My /proc/cpuinfo for reference:
>  processor	: 0
>  cpu		: 750FX
>  temperature 	: 0-3 C (uncalibrated)
>  clock		: 400MHz
>  revision	: 2.3 (pvr 7000 0203)
>  bogomips	: 793.71
>  machine		: PowerBook4,3
>  motherboard	: PowerBook4,3 MacRISC2 MacRISC Power Macintosh
>  detected as	: 257 (iBook 2 rev. 2)
>  pmac flags	: 0000001b
>  L2 cache	: 512K unified
>  memory		: 256MB
>  pmac-generation	: NewWorld
> 
> Note that the maximum CPU speed is 900 MHz.

Can you also check my other patch which fixes cpufreq as well ? And test
if it's reliable wether you sleep at low or high speed ?

Thanks,
Ben.

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

* Re: [PATCH] Fix sleep/wakeup problems on iBook G3
  2005-03-29 22:04   ` Benjamin Herrenschmidt
@ 2005-03-29 22:45     ` Graham Wilson
  0 siblings, 0 replies; 4+ messages in thread
From: Graham Wilson @ 2005-03-29 22:45 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: linuxppc-dev, debian-powerpc

On Wed, Mar 30, 2005 at 08:04:37AM +1000, Benjamin Herrenschmidt wrote:
> > I've been having various trouble with my iBook lately, including random
> > freezes, not waking up after going to sleep sometimes, and sometimes not
> > going to sleep. Since applying the patch you posted, everything seems
> > fine, though I may be speaking too soon.
> 
> Can you also check my other patch which fixes cpufreq as well ? And test
> if it's reliable wether you sleep at low or high speed ?

I haven't had any trouble with cpufreq (and I use powernowd with the
userspace governor), but I will apply your other patch this evening and
see what happens.

-- 
gram

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

end of thread, other threads:[~2005-03-29 22:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-03-27 22:55 [PATCH] Fix sleep/wakeup problems on iBook G3 Benjamin Herrenschmidt
2005-03-29 19:07 ` Graham Wilson
2005-03-29 22:04   ` Benjamin Herrenschmidt
2005-03-29 22:45     ` Graham Wilson

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