cris fasttimer: remove use of seq_printf return value

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")

Miscellanea:

o Coalesce formats, realign arguments

Signed-off-by: Joe Perches <joe@perches.com>
Cc: Mikael Starvik <starvik@axis.com>
Cc: Jesper Nilsson <jesper.nilsson@axis.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

1 files changed, 39 insertions, 46 deletions

diff --git a/arch/cris/arch-v32/kernel/fasttimer.c b/arch/cris/arch-v32/kernel/fasttimer.c
index b130c2c5fdd8..5c84dbb99f30 100644
--- a/arch/cris/arch-v32/kernel/fasttimer.c
+++ b/arch/cris/arch-v32/kernel/fasttimer.c
@@ -501,7 +501,8 @@ static int proc_fasttimer_show(struct seq_file *m, void *v)
                         i = debug_log_cnt;
 
                 while ((i != end_i || debug_log_cnt_wrapped)) {
-                        if (seq_printf(m, debug_log_string[i], debug_log_value[i]) < 0)
+                        seq_printf(m, debug_log_string[i], debug_log_value[i]);
+                        if (seq_has_overflowed(m))
                                 return 0;
                         i = (i+1) % DEBUG_LOG_MAX;
                 }
@@ -516,23 +517,21 @@ static int proc_fasttimer_show(struct seq_file *m, void *v)
                 int cur = (fast_timers_started - i - 1) % NUM_TIMER_STATS;
 
 #if 1 //ndef FAST_TIMER_LOG
-                seq_printf(m, "div: %i delay: %i"
-                           "\n",
+                seq_printf(m, "div: %i delay: %i\n",
                            timer_div_settings[cur],
                            timer_delay_settings[cur]);
 #endif
 #ifdef FAST_TIMER_LOG
                 t = &timer_started_log[cur];
-                if (seq_printf(m, "%-14s s: %6lu.%06lu e: %6lu.%06lu "
-                               "d: %6li us data: 0x%08lX"
-                               "\n",
-                               t->name,
-                               (unsigned long)t->tv_set.tv_jiff,
-                               (unsigned long)t->tv_set.tv_usec,
-                               (unsigned long)t->tv_expires.tv_jiff,
-                               (unsigned long)t->tv_expires.tv_usec,
-                               t->delay_us,
-                               t->data) < 0)
+                seq_printf(m, "%-14s s: %6lu.%06lu e: %6lu.%06lu d: %6li us data: 0x%08lX\n",
+                           t->name,
+                           (unsigned long)t->tv_set.tv_jiff,
+                           (unsigned long)t->tv_set.tv_usec,
+                           (unsigned long)t->tv_expires.tv_jiff,
+                           (unsigned long)t->tv_expires.tv_usec,
+                           t->delay_us,
+                           t->data);
+                if (seq_has_overflowed(m))
                         return 0;
 #endif
         }
@@ -544,16 +543,15 @@ static int proc_fasttimer_show(struct seq_file *m, void *v)
         seq_printf(m, "Timers added: %i\n", fast_timers_added);
         for (i = 0; i < num_to_show; i++) {
                 t = &timer_added_log[(fast_timers_added - i - 1) % NUM_TIMER_STATS];
-                if (seq_printf(m, "%-14s s: %6lu.%06lu e: %6lu.%06lu "
-                               "d: %6li us data: 0x%08lX"
-                               "\n",
-                               t->name,
-                               (unsigned long)t->tv_set.tv_jiff,
-                               (unsigned long)t->tv_set.tv_usec,
-                               (unsigned long)t->tv_expires.tv_jiff,
-                               (unsigned long)t->tv_expires.tv_usec,
-                               t->delay_us,
-                               t->data) < 0)
+                seq_printf(m, "%-14s s: %6lu.%06lu e: %6lu.%06lu d: %6li us data: 0x%08lX\n",
+                           t->name,
+                           (unsigned long)t->tv_set.tv_jiff,
+                           (unsigned long)t->tv_set.tv_usec,
+                           (unsigned long)t->tv_expires.tv_jiff,
+                           (unsigned long)t->tv_expires.tv_usec,
+                           t->delay_us,
+                           t->data);
+                if (seq_has_overflowed(m))
                         return 0;
         }
         seq_putc(m, '\n');
@@ -563,16 +561,15 @@ static int proc_fasttimer_show(struct seq_file *m, void *v)
         seq_printf(m, "Timers expired: %i\n", fast_timers_expired);
         for (i = 0; i < num_to_show; i++){
                 t = &timer_expired_log[(fast_timers_expired - i - 1) % NUM_TIMER_STATS];
-                if (seq_printf(m, "%-14s s: %6lu.%06lu e: %6lu.%06lu "
-                               "d: %6li us data: 0x%08lX"
-                               "\n",
-                               t->name,
-                               (unsigned long)t->tv_set.tv_jiff,
-                               (unsigned long)t->tv_set.tv_usec,
-                               (unsigned long)t->tv_expires.tv_jiff,
-                               (unsigned long)t->tv_expires.tv_usec,
-                               t->delay_us,
-                               t->data) < 0)
+                seq_printf(m, "%-14s s: %6lu.%06lu e: %6lu.%06lu d: %6li us data: 0x%08lX\n",
+                           t->name,
+                           (unsigned long)t->tv_set.tv_jiff,
+                           (unsigned long)t->tv_set.tv_usec,
+                           (unsigned long)t->tv_expires.tv_jiff,
+                           (unsigned long)t->tv_expires.tv_usec,
+                           t->delay_us,
+                           t->data);
+                if (seq_has_overflowed(m))
                         return 0;
         }
         seq_putc(m, '\n');
@@ -584,19 +581,15 @@ static int proc_fasttimer_show(struct seq_file *m, void *v)
         while (t != NULL){
                 nextt = t->next;
                 local_irq_restore(flags);
-                if (seq_printf(m, "%-14s s: %6lu.%06lu e: %6lu.%06lu "
-                               "d: %6li us data: 0x%08lX"
-/*                      " func: 0x%08lX" */
-                               "\n",
-                               t->name,
-                               (unsigned long)t->tv_set.tv_jiff,
-                               (unsigned long)t->tv_set.tv_usec,
-                               (unsigned long)t->tv_expires.tv_jiff,
-                               (unsigned long)t->tv_expires.tv_usec,
-                               t->delay_us,
-                               t->data
-/*                      , t->function */
-                               ) < 0)
+                seq_printf(m, "%-14s s: %6lu.%06lu e: %6lu.%06lu d: %6li us data: 0x%08lX\n",
+                           t->name,
+                           (unsigned long)t->tv_set.tv_jiff,
+                           (unsigned long)t->tv_set.tv_usec,
+                           (unsigned long)t->tv_expires.tv_jiff,
+                           (unsigned long)t->tv_expires.tv_usec,
+                           t->delay_us,
+                           t->data);
+                if (seq_has_overflowed(m))
                         return 0;
                 local_irq_save(flags);
                 if (t->next != nextt)