mfd: ab8500-debugfs: Remove extraneous seq_putc
authorLinus Torvalds <torvalds@linux-foundation.org>
Sat, 26 Dec 2020 17:19:49 +0000 (09:19 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 26 Dec 2020 17:19:49 +0000 (09:19 -0800)
Commit c9a3c4e637ac ("mfd: ab8500-debugfs: Remove extraneous curly
brace") removed a left-over curly brace that caused build failures, but
Joe Perches points out that the subsequent 'seq_putc()' should also be
removed, because the commit that caused all these problems already added
the final '\n' to the seq_printf() above it.

Reported-by: Joe Perches <joe@perches.com>
Fixes: 886c8121659d ("mfd: ab8500-debugfs: Remove the racy fiddling with irq_desc")
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Nathan Chancellor <natechancellor@gmail.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/mfd/ab8500-debugfs.c

index 1cf8456..e43dea8 100644 (file)
@@ -1521,7 +1521,6 @@ static int ab8500_interrupts_show(struct seq_file *s, void *p)
                           line + irq_first,
                           num_interrupts[line],
                           num_wake_interrupts[line]);
-               seq_putc(s, '\n');
        }
 
        return 0;