]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
drm/dp_mst: Simplify character output in drm_dp_mst_dump_topology()
authorMarkus Elfring <elfring@users.sourceforge.net>
Mon, 15 Jul 2024 11:12:10 +0000 (13:12 +0200)
committerLyude Paul <lyude@redhat.com>
Tue, 23 Jul 2024 19:45:39 +0000 (15:45 -0400)
Single characters should be put into a sequence.
Thus use the corresponding function “seq_putc” for one selected call.

This issue was transformed by using the Coccinelle software.

Suggested-by: Christophe Jaillet <christophe.jaillet@wanadoo.fr>
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Reviewed-by: Lyude Paul <lyude@redhat.com>
Signed-off-by: Lyude Paul <lyude@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/af602b2e-4d92-4c54-9884-7db84700aa93@web.de
drivers/gpu/drm/display/drm_dp_mst_topology.c

index fc2ceae61db2d0f6c31f6e6e1da35b2fce6a79db..65c1700f3e40763674a52fccbf45977491ee7553 100644 (file)
@@ -4963,7 +4963,7 @@ void drm_dp_mst_dump_topology(struct seq_file *m,
                seq_printf(m, "branch oui: %*phN devid: ", 3, buf);
 
                for (i = 0x3; i < 0x8 && buf[i]; i++)
-                       seq_printf(m, "%c", buf[i]);
+                       seq_putc(m, buf[i]);
                seq_printf(m, " revision: hw: %x.%x sw: %x.%x\n",
                           buf[0x9] >> 4, buf[0x9] & 0xf, buf[0xa], buf[0xb]);
                if (dump_dp_payload_table(mgr, buf))