]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
media: renesas: vsp1: Use %p4cc printk modifier to print FourCC codes
authorBiju Das <biju.das.jz@bp.renesas.com>
Sun, 30 Mar 2025 09:17:35 +0000 (10:17 +0100)
committerHans Verkuil <hverkuil@xs4all.nl>
Fri, 2 May 2025 08:16:44 +0000 (10:16 +0200)
Replace '%08x->%p4cc' printk modifier to print FourCC codes for pixel
formats.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Link: https://lore.kernel.org/r/20250330091738.27052-1-biju.das.jz@bp.renesas.com
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
drivers/media/platform/renesas/vsp1/vsp1_drm.c

index b5d1f238f7beeab0615aa7fb815cdd61e79616b6..f8a575f6188a49f004957aa56662ee9a712ebaa1 100644 (file)
@@ -593,8 +593,8 @@ static int vsp1_du_pipeline_set_rwpf_format(struct vsp1_device *vsp1,
 
        fmtinfo = vsp1_get_format_info(vsp1, pixelformat);
        if (!fmtinfo) {
-               dev_dbg(vsp1->dev, "Unsupported pixel format %08x\n",
-                       pixelformat);
+               dev_dbg(vsp1->dev, "Unsupported pixel format %p4cc\n",
+                       &pixelformat);
                return -EINVAL;
        }
 
@@ -849,11 +849,11 @@ int vsp1_du_atomic_update(struct device *dev, unsigned int pipe_index,
        }
 
        dev_dbg(vsp1->dev,
-               "%s: RPF%u: (%u,%u)/%ux%u -> (%u,%u)/%ux%u (%08x), pitch %u dma { %pad, %pad, %pad } zpos %u\n",
+               "%s: RPF%u: (%u,%u)/%ux%u -> (%u,%u)/%ux%u (%p4cc), pitch %u dma { %pad, %pad, %pad } zpos %u\n",
                __func__, rpf_index,
                cfg->src.left, cfg->src.top, cfg->src.width, cfg->src.height,
                cfg->dst.left, cfg->dst.top, cfg->dst.width, cfg->dst.height,
-               cfg->pixelformat, cfg->pitch, &cfg->mem[0], &cfg->mem[1],
+               &cfg->pixelformat, cfg->pitch, &cfg->mem[0], &cfg->mem[1],
                &cfg->mem[2], cfg->zpos);
 
        /*