On 10/15/2016 12:56 PM, SF Markus Elfring wrote: > From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> > Date: Sat, 15 Oct 2016 21:31:16 +0200 > > A single character (line break) should be put into a sequence. > Thus use the corresponding function "seq_putc". Perhaps reword the changelog to say that seqc_putc is more efficient than seqc_printf to output a single char. I mean _printf is not wrong but not as efficient ? > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> > --- > arch/arc/kernel/setup.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/arch/arc/kernel/setup.c b/arch/arc/kernel/setup.c > index 3df7f9c..e3f5432 100644 > --- a/arch/arc/kernel/setup.c > +++ b/arch/arc/kernel/setup.c > @@ -494,8 +494,7 @@ static int show_cpuinfo(struct seq_file *m, void *v) > > free_page((unsigned long)str); > done: > - seq_printf(m, "\n"); > - > + seq_putc(m, '\n'); > return 0; > } > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
