diff options
author | Sam Ravnborg <sam@ravnborg.org> | 2009-01-18 02:10:35 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-01-18 02:10:35 -0500 |
commit | c275ce44929402664902cbb09d4f054b315623d7 (patch) | |
tree | 4016c6a4c82b55c55ab833df6f36c7635876761e /arch/sparc | |
parent | f3b8436ad9a8ad36b3c9fa1fe030c7f38e5d3d0b (diff) |
sparc64: fix readout of cpu/fpu type
Meelis reported that on his box /proc/cpuinfo started
to reported "Unknow CPU" and the same did the boot messages.
It was a stupid bug I introduced when merging
cpu.c for 32 and 64 bit.
The code did an array reference where it had to search
for the right index.
Reported-by: Meelis Roos <mroos@linux.ee>
Tested-by: Meelis Roos <mroos@linux.ee>
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'arch/sparc')
-rw-r--r-- | arch/sparc/kernel/cpu.c | 18 |
1 files changed, 15 insertions, 3 deletions
diff --git a/arch/sparc/kernel/cpu.c b/arch/sparc/kernel/cpu.c index f0b825505da5..32d32b4824f5 100644 --- a/arch/sparc/kernel/cpu.c +++ b/arch/sparc/kernel/cpu.c | |||
@@ -239,14 +239,26 @@ unsigned int fsr_storage; | |||
239 | 239 | ||
240 | static void set_cpu_and_fpu(int psr_impl, int psr_vers, int fpu_vers) | 240 | static void set_cpu_and_fpu(int psr_impl, int psr_vers, int fpu_vers) |
241 | { | 241 | { |
242 | const struct manufacturer_info *manuf; | ||
243 | int i; | ||
244 | |||
242 | sparc_cpu_type = NULL; | 245 | sparc_cpu_type = NULL; |
243 | sparc_fpu_type = NULL; | 246 | sparc_fpu_type = NULL; |
244 | if (psr_impl < ARRAY_SIZE(manufacturer_info)) | 247 | manuf = NULL; |
248 | |||
249 | for (i = 0; i < ARRAY_SIZE(manufacturer_info); i++) | ||
250 | { | ||
251 | if (psr_impl == manufacturer_info[i].psr_impl) { | ||
252 | manuf = &manufacturer_info[i]; | ||
253 | break; | ||
254 | } | ||
255 | } | ||
256 | if (manuf != NULL) | ||
245 | { | 257 | { |
246 | const struct cpu_info *cpu; | 258 | const struct cpu_info *cpu; |
247 | const struct fpu_info *fpu; | 259 | const struct fpu_info *fpu; |
248 | 260 | ||
249 | cpu = &manufacturer_info[psr_impl].cpu_info[0]; | 261 | cpu = &manuf->cpu_info[0]; |
250 | while (cpu->psr_vers != -1) | 262 | while (cpu->psr_vers != -1) |
251 | { | 263 | { |
252 | if (cpu->psr_vers == psr_vers) { | 264 | if (cpu->psr_vers == psr_vers) { |
@@ -256,7 +268,7 @@ static void set_cpu_and_fpu(int psr_impl, int psr_vers, int fpu_vers) | |||
256 | } | 268 | } |
257 | cpu++; | 269 | cpu++; |
258 | } | 270 | } |
259 | fpu = &manufacturer_info[psr_impl].fpu_info[0]; | 271 | fpu = &manuf->fpu_info[0]; |
260 | while (fpu->fp_vers != -1) | 272 | while (fpu->fp_vers != -1) |
261 | { | 273 | { |
262 | if (fpu->fp_vers == fpu_vers) { | 274 | if (fpu->fp_vers == fpu_vers) { |