From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> Date: Mon, 1 Jul 2019 13:07:55 +0200 A single character (depending on a condition check) 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-multipath.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/md/md-multipath.c b/drivers/md/md-multipath.c index 6780938d2991..3bf6f97ea264 100644 --- a/drivers/md/md-multipath.c +++ b/drivers/md/md-multipath.c @@ -146,7 +146,8 @@ static void multipath_status(struct seq_file *seq, struct mddev *mddev) rcu_read_lock(); for (i = 0; i < conf->raid_disks; i++) { struct md_rdev *rdev = rcu_dereference(conf->multipaths[i].rdev); - seq_printf (seq, "%s", rdev && test_bit(In_sync, &rdev->flags) ? "U" : "_"); + seq_putc(seq, + rdev && test_bit(In_sync, &rdev->flags) ? 'U' : '_'); } rcu_read_unlock(); seq_putc(seq, ']'); -- 2.22.0