Linux MIPS Architecture development
 help / color / mirror / Atom feed
* MIPS 20Kc and WAIT instruction
@ 2007-06-11  9:54 Aurelien Jarno
  2007-06-12 10:18 ` Ralf Baechle
  0 siblings, 1 reply; 2+ messages in thread
From: Aurelien Jarno @ 2007-06-11  9:54 UTC (permalink / raw)
  To: linux-mips

Hi all,

The latest kernels from kernel.org or linux-mips.org do not enter low
power mode when the CPU is idle on a MIPS 20Kc CPU.

Looking at the code in arch/mips/kernel/cpu-probe.c, the "case"
corresponding to the 20Kc CPU is commented out:

        case CPU_5KC:
/*      case CPU_20KC:*/
        case CPU_24K:

According to the datasheet this CPU supports the WAIT instruction, so I
don't really understand why it is disabled in the kernel. Does anybody
know why?

Thanks,
Aurelien

-- 
  .''`.  Aurelien Jarno	            | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   aurel32@debian.org         | aurelien@aurel32.net
   `-    people.debian.org/~aurel32 | www.aurel32.net

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

* Re: MIPS 20Kc and WAIT instruction
  2007-06-11  9:54 MIPS 20Kc and WAIT instruction Aurelien Jarno
@ 2007-06-12 10:18 ` Ralf Baechle
  0 siblings, 0 replies; 2+ messages in thread
From: Ralf Baechle @ 2007-06-12 10:18 UTC (permalink / raw)
  To: Aurelien Jarno; +Cc: linux-mips

On Mon, Jun 11, 2007 at 11:54:07AM +0200, Aurelien Jarno wrote:

> The latest kernels from kernel.org or linux-mips.org do not enter low
> power mode when the CPU is idle on a MIPS 20Kc CPU.
> 
> Looking at the code in arch/mips/kernel/cpu-probe.c, the "case"
> corresponding to the 20Kc CPU is commented out:
> 
>         case CPU_5KC:
> /*      case CPU_20KC:*/
>         case CPU_24K:
> 
> According to the datasheet this CPU supports the WAIT instruction, so I
> don't really understand why it is disabled in the kernel. Does anybody
> know why?

CPU_20KC is MIPS64 so by definition supports a WAIT instruction - even
though an implementation would be legal as per architecture spec.

As for why it's commented out, I don't know.  The original patch to add
support was submitted by Carsten Langgaard (ex-carstenl@mips.com) in 2002
and already had the CPU_20KC commented out.  As for why one would do
something like that - my guess is as good as your's :-)  Trying to find
out more.

  Ralf

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

end of thread, other threads:[~2007-06-12 10:20 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-06-11  9:54 MIPS 20Kc and WAIT instruction Aurelien Jarno
2007-06-12 10:18 ` Ralf Baechle

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox