Re: [PATCH] pinctrl: sh-pfc: Use seq_puts() in sh_pfc_pin_dbg_show()
On Sat, Jan 6, 2018 at 9:56 PM, SF Markus Elfringwrote: > From: Markus Elfring > Date: Sat, 6 Jan 2018 21:50:20 +0100 > > A string which did not contain a data format specification should be put > into a sequence. Thus use the corresponding function "seq_puts". > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring Reviewed-by: Geert Uytterhoeven i.e. will queue in sh-pfc-for-v4.17. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Re: [PATCH] pinctrl: sh-pfc: Use seq_puts() in sh_pfc_pin_dbg_show()
On Sat, Jan 6, 2018 at 9:56 PM, SF Markus Elfringwrote: > From: Markus Elfring > Date: Sat, 6 Jan 2018 21:50:20 +0100 > > A string which did not contain a data format specification should be put > into a sequence. Thus use the corresponding function "seq_puts". > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring Leaving to Geert to pick this up. (Looks good.) Yours, Linus Walleij
[PATCH] pinctrl: sh-pfc: Use seq_puts() in sh_pfc_pin_dbg_show()
From: Markus ElfringDate: Sat, 6 Jan 2018 21:50:20 +0100 A string which did not contain a data format specification should be put into a sequence. Thus use the corresponding function "seq_puts". This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/pinctrl/sh-pfc/pinctrl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/sh-pfc/pinctrl.c b/drivers/pinctrl/sh-pfc/pinctrl.c index 736634aee500..70db21638901 100644 --- a/drivers/pinctrl/sh-pfc/pinctrl.c +++ b/drivers/pinctrl/sh-pfc/pinctrl.c @@ -75,7 +75,7 @@ static int sh_pfc_get_group_pins(struct pinctrl_dev *pctldev, unsigned selector, static void sh_pfc_pin_dbg_show(struct pinctrl_dev *pctldev, struct seq_file *s, unsigned offset) { - seq_printf(s, "%s", DRV_NAME); + seq_puts(s, DRV_NAME); } #ifdef CONFIG_OF -- 2.15.1