aboutsummaryrefslogtreecommitdiffstats
path: root/arch/powerpc/kernel/setup_32.c
diff options
context:
space:
mode:
authorPaul Mackerras <paulus@samba.org>2005-10-20 06:48:19 -0400
committerPaul Mackerras <paulus@samba.org>2005-10-20 06:48:19 -0400
commit0dd194d02d2584c34e06ddd26c7a7896a5fa1974 (patch)
treef5441cf5497d78d0c3ed5ce9d9ae47328426f4f8 /arch/powerpc/kernel/setup_32.c
parenta9c59264690aea9d0df2d2d76683bc39ec6b7288 (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.c14
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);
150int show_cpuinfo(struct seq_file *m, void *v) 150int 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 {