pinctrl: Use seq_putc() in three functions
authorMarkus Elfring <elfring@users.sourceforge.net>
Mon, 1 May 2017 20:24:29 +0000 (22:24 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 22 May 2017 08:39:24 +0000 (10:39 +0200)
A single character (line break) should be put into a sequence.
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>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinconf.c

index a02dba3..1a3fbdf 100644 (file)
@@ -232,7 +232,7 @@ static void pinconf_show_config(struct seq_file *s, struct pinctrl_dev *pctldev,
                                                            configs[i]);
                else
                        seq_printf(s, "%08lx", configs[i]);
-               seq_puts(s, "\n");
+               seq_putc(s, '\n');
        }
 }
 
@@ -325,8 +325,7 @@ static int pinconf_pins_show(struct seq_file *s, void *what)
                seq_printf(s, "pin %d (%s): ", pin, desc->name);
 
                pinconf_dump_pin(pctldev, s, pin);
-
-               seq_printf(s, "\n");
+               seq_putc(s, '\n');
        }
 
        mutex_unlock(&pctldev->mutex);
@@ -361,8 +360,7 @@ static int pinconf_groups_show(struct seq_file *s, void *what)
 
                seq_printf(s, "%u (%s): ", selector, gname);
                pinconf_dump_group(pctldev, s, selector, gname);
-               seq_printf(s, "\n");
-
+               seq_putc(s, '\n');
                selector++;
        }