The seq_printf return value, because it's frequently misused,
will eventually be converted to void.

See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
     seq_has_overflowed() and make public")

Signed-off-by: Joe Perches <j...@perches.com>
---
 arch/openrisc/kernel/setup.c | 50 +++++++++++++++++++++++---------------------
 1 file changed, 26 insertions(+), 24 deletions(-)

diff --git a/arch/openrisc/kernel/setup.c b/arch/openrisc/kernel/setup.c
index 4fc7ccc..b4ed8b3 100644
--- a/arch/openrisc/kernel/setup.c
+++ b/arch/openrisc/kernel/setup.c
@@ -329,30 +329,32 @@ static int show_cpuinfo(struct seq_file *m, void *v)
        version = (vr & SPR_VR_VER) >> 24;
        revision = vr & SPR_VR_REV;
 
-       return seq_printf(m,
-                         "cpu\t\t: OpenRISC-%x\n"
-                         "revision\t: %d\n"
-                         "frequency\t: %ld\n"
-                         "dcache size\t: %d bytes\n"
-                         "dcache block size\t: %d bytes\n"
-                         "icache size\t: %d bytes\n"
-                         "icache block size\t: %d bytes\n"
-                         "immu\t\t: %d entries, %lu ways\n"
-                         "dmmu\t\t: %d entries, %lu ways\n"
-                         "bogomips\t: %lu.%02lu\n",
-                         version,
-                         revision,
-                         loops_per_jiffy * HZ,
-                         cpuinfo.dcache_size,
-                         cpuinfo.dcache_block_size,
-                         cpuinfo.icache_size,
-                         cpuinfo.icache_block_size,
-                         1 << ((mfspr(SPR_DMMUCFGR) & SPR_DMMUCFGR_NTS) >> 2),
-                         1 + (mfspr(SPR_DMMUCFGR) & SPR_DMMUCFGR_NTW),
-                         1 << ((mfspr(SPR_IMMUCFGR) & SPR_IMMUCFGR_NTS) >> 2),
-                         1 + (mfspr(SPR_IMMUCFGR) & SPR_IMMUCFGR_NTW),
-                         (loops_per_jiffy * HZ) / 500000,
-                         ((loops_per_jiffy * HZ) / 5000) % 100);
+       seq_printf(m,
+                  "cpu\t\t: OpenRISC-%x\n"
+                  "revision\t: %d\n"
+                  "frequency\t: %ld\n"
+                  "dcache size\t: %d bytes\n"
+                  "dcache block size\t: %d bytes\n"
+                  "icache size\t: %d bytes\n"
+                  "icache block size\t: %d bytes\n"
+                  "immu\t\t: %d entries, %lu ways\n"
+                  "dmmu\t\t: %d entries, %lu ways\n"
+                  "bogomips\t: %lu.%02lu\n",
+                  version,
+                  revision,
+                  loops_per_jiffy * HZ,
+                  cpuinfo.dcache_size,
+                  cpuinfo.dcache_block_size,
+                  cpuinfo.icache_size,
+                  cpuinfo.icache_block_size,
+                  1 << ((mfspr(SPR_DMMUCFGR) & SPR_DMMUCFGR_NTS) >> 2),
+                  1 + (mfspr(SPR_DMMUCFGR) & SPR_DMMUCFGR_NTW),
+                  1 << ((mfspr(SPR_IMMUCFGR) & SPR_IMMUCFGR_NTS) >> 2),
+                  1 + (mfspr(SPR_IMMUCFGR) & SPR_IMMUCFGR_NTW),
+                  (loops_per_jiffy * HZ) / 500000,
+                  ((loops_per_jiffy * HZ) / 5000) % 100);
+
+       return 0;
 }
 
 static void *c_start(struct seq_file *m, loff_t * pos)
-- 
2.1.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to