From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> Date: Tue, 2 May 2017 14:01:17 +0200 A few single characters 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@xxxxxxxxxxxxxxxxxxxxx> --- drivers/md/md.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/md/md.c b/drivers/md/md.c index 82f798be964f..7f0a35ee192a 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -7498,7 +7498,7 @@ static void status_unused(struct seq_file *seq) if (!i) seq_printf(seq, "<none>"); - seq_printf(seq, "\n"); + seq_putc(seq, '\n'); } static int status_resync(struct seq_file *seq, struct mddev *mddev) @@ -7552,12 +7552,13 @@ static int status_resync(struct seq_file *seq, struct mddev *mddev) per_milli = res; { int i, x = per_milli/50, y = 20-x; - seq_printf(seq, "["); + + seq_putc(seq, '['); for (i = 0; i < x; i++) - seq_printf(seq, "="); - seq_printf(seq, ">"); + seq_putc(seq, '='); + seq_putc(seq, '>'); for (i = 0; i < y; i++) - seq_printf(seq, "."); + seq_putc(seq, '.'); seq_printf(seq, "] "); } seq_printf(seq, " %s =%3u.%u%% (%llu/%llu)", @@ -7678,7 +7679,7 @@ static int md_seq_show(struct seq_file *seq, void *v) seq_printf(seq, "[%s] ", pers->name); spin_unlock(&pers_lock); - seq_printf(seq, "\n"); + seq_putc(seq, '\n'); seq->poll_event = atomic_read(&md_event_count); return 0; } @@ -7754,8 +7755,7 @@ static int md_seq_show(struct seq_file *seq, void *v) seq_printf(seq, "\n "); bitmap_status(seq, mddev->bitmap); - - seq_printf(seq, "\n"); + seq_putc(seq, '\n'); } spin_unlock(&mddev->lock); -- 2.12.2 -- 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