On Mon Oct 28 15:59:26 2024 +0000, Sean Young wrote:
> There is no need to pass constants via stack. The width may be explicitly
> specified in the format.
> 
> Signed-off-by: Sean Young <s...@mess.org>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+hua...@kernel.org>

Patch committed.

Thanks,
Mauro Carvalho Chehab

 drivers/media/rc/mceusb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

---

diff --git a/drivers/media/rc/mceusb.c b/drivers/media/rc/mceusb.c
index cd7af4d88b7f..d5ea3868d5a3 100644
--- a/drivers/media/rc/mceusb.c
+++ b/drivers/media/rc/mceusb.c
@@ -658,8 +658,8 @@ static void mceusb_dev_printdata(struct mceusb_dev *ir, u8 
*buf, int buf_len,
                        if (len == 2)
                                dev_dbg(dev, "Get hw/sw rev?");
                        else
-                               dev_dbg(dev, "hw/sw rev %*ph",
-                                       4, &buf[offset + 2]);
+                               dev_dbg(dev, "hw/sw rev %4ph",
+                                       &buf[offset + 2]);
                        break;
                case MCE_CMD_RESUME:
                        dev_dbg(dev, "Device resume requested");

Reply via email to