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>
---
 drivers/char/i8k.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/drivers/char/i8k.c b/drivers/char/i8k.c
index 24cc4ed..a43048b 100644
--- a/drivers/char/i8k.c
+++ b/drivers/char/i8k.c
@@ -510,13 +510,15 @@ static int i8k_proc_show(struct seq_file *seq, void 
*offset)
         * 9)  AC power
         * 10) Fn Key status
         */
-       return seq_printf(seq, "%s %s %s %d %d %d %d %d %d %d\n",
-                         I8K_PROC_FMT,
-                         bios_version,
-                         i8k_get_dmi_data(DMI_PRODUCT_SERIAL),
-                         cpu_temp,
-                         left_fan, right_fan, left_speed, right_speed,
-                         ac_power, fn_key);
+       seq_printf(seq, "%s %s %s %d %d %d %d %d %d %d\n",
+                  I8K_PROC_FMT,
+                  bios_version,
+                  i8k_get_dmi_data(DMI_PRODUCT_SERIAL),
+                  cpu_temp,
+                  left_fan, right_fan, left_speed, right_speed,
+                  ac_power, fn_key);
+
+       return 0;
 }
 
 static int i8k_open_fs(struct inode *inode, struct file *file)
-- 
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