AVR32-clock: Use seq_putc() in two functions
authorMarkus Elfring <elfring@users.sourceforge.net>
Sun, 16 Oct 2016 19:51:09 +0000 (21:51 +0200)
committerHans-Christian Noren Egtvedt <egtvedt@samfundet.no>
Mon, 12 Dec 2016 08:05:46 +0000 (09:05 +0100)
A single character (line break) should be put into two sequences.
Thus use the corresponding function "seq_putc".

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
arch/avr32/mach-at32ap/clock.c

index 52c179b..a912a72 100644 (file)
@@ -242,7 +242,7 @@ dump_clock(struct clk *parent, struct clkinf *r)
                clk_get_rate(parent));
        if (parent->dev)
                seq_printf(r->s, ", for %s", dev_name(parent->dev));
-       seq_printf(r->s, "\n");
+       seq_putc(r->s, '\n');
 
        /* cost of this scan is small, but not linear... */
        r->nest = nest + NEST_DELTA;
@@ -276,8 +276,7 @@ static int clk_show(struct seq_file *s, void *unused)
                seq_printf(s, "GCCTRL%d = %8x\n", i, pm_readl(GCCTRL(i)));
        }
 
-       seq_printf(s, "\n");
-
+       seq_putc(s, '\n');
        r.s = s;
        r.nest = 0;
        /* protected from changes on the list while dumping */