| Message ID | af602b2e-4d92-4c54-9884-7db84700aa93@web.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm/dp_mst: Simplify character output in drm_dp_mst_dump_topology() | expand |
Reviewed-by: Lyude Paul <lyude@redhat.com> Will push to the kernel in just a moment On Mon, 2024-07-15 at 13:26 +0200, Markus Elfring wrote: > From: Markus Elfring <elfring@users.sourceforge.net> > Date: Mon, 15 Jul 2024 13:12:10 +0200 > > 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> > --- > drivers/gpu/drm/display/drm_dp_mst_topology.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c b/drivers/gpu/drm/display/drm_dp_mst_topology.c > index fc2ceae61db2..65c1700f3e40 100644 > --- a/drivers/gpu/drm/display/drm_dp_mst_topology.c > +++ b/drivers/gpu/drm/display/drm_dp_mst_topology.c > @@ -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)) > -- > 2.45.2 >
diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c b/drivers/gpu/drm/display/drm_dp_mst_topology.c index fc2ceae61db2..65c1700f3e40 100644 --- a/drivers/gpu/drm/display/drm_dp_mst_topology.c +++ b/drivers/gpu/drm/display/drm_dp_mst_topology.c @@ -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))