diff options
author | Paul Mackerras <paulus@samba.org> | 2005-10-20 06:48:19 -0400 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2005-10-20 06:48:19 -0400 |
commit | 0dd194d02d2584c34e06ddd26c7a7896a5fa1974 (patch) | |
tree | f5441cf5497d78d0c3ed5ce9d9ae47328426f4f8 /arch/powerpc/kernel/setup_32.c | |
parent | a9c59264690aea9d0df2d2d76683bc39ec6b7288 (diff) |
powerpc: Fix places where ppc_md.show_[per]cpuinfo was treated as int
I missed a few places where ppc code was still assuming that the
ppc_md.show_[per]cpuinfo functions returned int.
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/powerpc/kernel/setup_32.c')
-rw-r--r-- | arch/powerpc/kernel/setup_32.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/arch/powerpc/kernel/setup_32.c b/arch/powerpc/kernel/setup_32.c index b95f7cf693e6..b9269c038af3 100644 --- a/arch/powerpc/kernel/setup_32.c +++ b/arch/powerpc/kernel/setup_32.c | |||
@@ -150,7 +150,6 @@ extern u32 cpu_temp_both(unsigned long cpu); | |||
150 | int show_cpuinfo(struct seq_file *m, void *v) | 150 | int show_cpuinfo(struct seq_file *m, void *v) |
151 | { | 151 | { |
152 | int i = (int) v - 1; | 152 | int i = (int) v - 1; |
153 | int err = 0; | ||
154 | unsigned int pvr; | 153 | unsigned int pvr; |
155 | unsigned short maj, min; | 154 | unsigned short maj, min; |
156 | unsigned long lpj; | 155 | unsigned long lpj; |
@@ -167,8 +166,8 @@ int show_cpuinfo(struct seq_file *m, void *v) | |||
167 | #endif /* CONFIG_SMP */ | 166 | #endif /* CONFIG_SMP */ |
168 | 167 | ||
169 | if (ppc_md.show_cpuinfo != NULL) | 168 | if (ppc_md.show_cpuinfo != NULL) |
170 | err = ppc_md.show_cpuinfo(m); | 169 | ppc_md.show_cpuinfo(m); |
171 | return err; | 170 | return 0; |
172 | } | 171 | } |
173 | 172 | ||
174 | #ifdef CONFIG_SMP | 173 | #ifdef CONFIG_SMP |
@@ -210,15 +209,12 @@ int show_cpuinfo(struct seq_file *m, void *v) | |||
210 | } | 209 | } |
211 | #endif /* CONFIG_TAU */ | 210 | #endif /* CONFIG_TAU */ |
212 | 211 | ||
213 | if (ppc_md.show_percpuinfo != NULL) { | 212 | if (ppc_md.show_percpuinfo != NULL) |
214 | err = ppc_md.show_percpuinfo(m, i); | 213 | ppc_md.show_percpuinfo(m, i); |
215 | if (err) | ||
216 | return err; | ||
217 | } | ||
218 | 214 | ||
219 | /* If we are a Freescale core do a simple check so | 215 | /* If we are a Freescale core do a simple check so |
220 | * we dont have to keep adding cases in the future */ | 216 | * we dont have to keep adding cases in the future */ |
221 | if ((PVR_VER(pvr) & 0x8000) == 0x8000) { | 217 | if (PVR_VER(pvr) & 0x8000) { |
222 | maj = PVR_MAJ(pvr); | 218 | maj = PVR_MAJ(pvr); |
223 | min = PVR_MIN(pvr); | 219 | min = PVR_MIN(pvr); |
224 | } else { | 220 | } else { |