[PATCH v2 04/19] drm: Pass the format info to .fb_create()

Laurent Pinchart laurent.pinchart at ideasonboard.com
Tue Jul 1 19:18:01 UTC 2025


Hi Ville,

Thank you for the patch.

On Tue, Jul 01, 2025 at 12:07:07PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> Pass along the format information from the top to .fb_create()
> so that we can avoid redundant (and somewhat expensive) lookups
> in the drivers.
> 
> Done with cocci (with some manual fixups):
> @@
> identifier func =~ ".*create.*";
> identifier dev, file, mode_cmd;
> @@
> struct drm_framebuffer *func(
>        struct drm_device *dev,
>        struct drm_file *file,
> +      const struct drm_format_info *info,
>        const struct drm_mode_fb_cmd2 *mode_cmd)
> {
> ...
> (
> - const struct drm_format_info *info = drm_get_format_info(...);
> |
> - const struct drm_format_info *info;
> ...
> - info = drm_get_format_info(...);
> )
> <...
> - if (!info)
> -    return ...;
> ...>
> }
> 
> @@
> identifier func =~ ".*create.*";
> identifier dev, file, mode_cmd;
> @@
> struct drm_framebuffer *func(
>        struct drm_device *dev,
>        struct drm_file *file,
> +      const struct drm_format_info *info,
>        const struct drm_mode_fb_cmd2 *mode_cmd)
> {
> ...
> }
> 
> @find@
> identifier fb_create_func =~ ".*create.*";
> identifier dev, file, mode_cmd;
> @@
> struct drm_framebuffer *fb_create_func(
>        struct drm_device *dev,
>        struct drm_file *file,
> +      const struct drm_format_info *info,
>        const struct drm_mode_fb_cmd2 *mode_cmd);
> 
> @@
> identifier find.fb_create_func;
> expression dev, file, mode_cmd;
> @@
> fb_create_func(dev, file
> +	       ,info
> 	       ,mode_cmd)
> 
> @@
> expression dev, file, mode_cmd;
> @@
> drm_gem_fb_create(dev, file
> +	       ,info
> 	       ,mode_cmd)
> 
> @@
> expression dev, file, mode_cmd;
> @@
> drm_gem_fb_create_with_dirty(dev, file
> +	       ,info
> 	       ,mode_cmd)
> 
> @@
> expression dev, file_priv, mode_cmd;
> identifier info, fb;
> @@
> info = drm_get_format_info(...);
> ...
> fb = dev->mode_config.funcs->fb_create(dev, file_priv
> +                                      ,info
>                                        ,mode_cmd);
> 
> @@
> identifier dev, file_priv, mode_cmd;
> @@
> struct drm_mode_config_funcs {
> ...
> struct drm_framebuffer *(*fb_create)(struct drm_device *dev,
>                                      struct drm_file *file_priv,
> +                                     const struct drm_format_info *info,
>                                      const struct drm_mode_fb_cmd2 *mode_cmd);
> ...
> };
> 
> v2: Fix kernel docs (Laurent)
>     Fix commit msg (Geert)
> 
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Cc: Liviu Dudau <liviu.dudau at arm.com>
> Cc: Maxime Ripard <mripard at kernel.org>
> Cc: Russell King <linux at armlinux.org.uk>
> Cc: Inki Dae <inki.dae at samsung.com>
> Cc: Seung-Woo Kim <sw0312.kim at samsung.com>
> Cc: Kyungmin Park <kyungmin.park at samsung.com>
> Cc: Patrik Jakobsson <patrik.r.jakobsson at gmail.com>
> Cc: Chun-Kuang Hu <chunkuang.hu at kernel.org>
> Cc: Philipp Zabel <p.zabel at pengutronix.de>
> Cc: Rob Clark <robdclark at gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk at quicinc.com>
> Cc: Dmitry Baryshkov <lumag at kernel.org>
> Cc: Sean Paul <sean at poorly.run>
> Cc: Marijn Suijten <marijn.suijten at somainline.org>
> Cc: Marek Vasut <marex at denx.de>
> Cc: Stefan Agner <stefan at agner.ch>
> Cc: Lyude Paul <lyude at redhat.com>
> Cc: Danilo Krummrich <dakr at kernel.org>
> Cc: Tomi Valkeinen <tomi.valkeinen at ideasonboard.com>
> Cc: Dave Airlie <airlied at redhat.com>
> Cc: Gerd Hoffmann <kraxel at redhat.com>
> Cc: Kieran Bingham <kieran.bingham+renesas at ideasonboard.com>
> Cc: Biju Das <biju.das.jz at bp.renesas.com>
> Cc: Sandy Huang <hjc at rock-chips.com>
> Cc: "Heiko Stübner" <heiko at sntech.de>
> Cc: Andy Yan <andy.yan at rock-chips.com>
> Cc: Thierry Reding <thierry.reding at gmail.com>
> Cc: Mikko Perttunen <mperttunen at nvidia.com>
> Cc: Dave Stevenson <dave.stevenson at raspberrypi.com>
> Cc: "Maíra Canal" <mcanal at igalia.com>
> Cc: Raspberry Pi Kernel Maintenance <kernel-list at raspberrypi.com>
> Cc: Dmitry Osipenko <dmitry.osipenko at collabora.com>
> Cc: Gurchetan Singh <gurchetansingh at chromium.org>
> Cc: Chia-I Wu <olvaffe at gmail.com>
> Cc: Zack Rusin <zack.rusin at broadcom.com>
> Cc: Broadcom internal kernel review list <bcm-kernel-feedback-list at broadcom.com>
> Cc: Oleksandr Andrushchenko <oleksandr_andrushchenko at epam.com>
> Cc: amd-gfx at lists.freedesktop.org
> Cc: linux-arm-msm at vger.kernel.org
> Cc: freedreno at lists.freedesktop.org
> Cc: nouveau at lists.freedesktop.org
> Cc: virtualization at lists.linux.dev
> Cc: spice-devel at lists.freedesktop.org
> Cc: linux-renesas-soc at vger.kernel.org
> Cc: linux-tegra at vger.kernel.org
> Cc: Laurent Pinchart <laurent.pinchart+renesas at ideasonboard.com>
> Reviewed-by: Geert Uytterhoeven <geert+renesas at glider.be>
> Reviewed-by: Thomas Zimmermann <tzimmermann at suse.de>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c            |  1 +
>  drivers/gpu/drm/amd/amdgpu/amdgpu_display.h            |  1 +
>  .../gpu/drm/arm/display/komeda/komeda_framebuffer.c    |  1 +
>  .../gpu/drm/arm/display/komeda/komeda_framebuffer.h    |  1 +
>  drivers/gpu/drm/arm/malidp_drv.c                       |  3 ++-
>  drivers/gpu/drm/armada/armada_fb.c                     |  6 ++----
>  drivers/gpu/drm/armada/armada_fb.h                     |  3 ++-
>  drivers/gpu/drm/drm_framebuffer.c                      |  2 +-
>  drivers/gpu/drm/drm_gem_framebuffer_helper.c           |  4 ++++
>  drivers/gpu/drm/exynos/exynos_drm_fb.c                 |  4 +---
>  drivers/gpu/drm/gma500/framebuffer.c                   |  1 +
>  drivers/gpu/drm/i915/display/intel_fb.c                |  1 +
>  drivers/gpu/drm/i915/display/intel_fb.h                |  1 +
>  drivers/gpu/drm/ingenic/ingenic-drm-drv.c              |  5 +++--
>  drivers/gpu/drm/mediatek/mtk_drm_drv.c                 |  7 ++-----
>  drivers/gpu/drm/msm/msm_drv.h                          |  3 ++-
>  drivers/gpu/drm/msm/msm_fb.c                           |  6 ++----
>  drivers/gpu/drm/mxsfb/mxsfb_drv.c                      | 10 ++--------
>  drivers/gpu/drm/nouveau/nouveau_display.c              |  1 +
>  drivers/gpu/drm/nouveau/nouveau_display.h              |  1 +
>  drivers/gpu/drm/omapdrm/omap_fb.c                      |  6 ++----
>  drivers/gpu/drm/omapdrm/omap_fb.h                      |  3 ++-
>  drivers/gpu/drm/qxl/qxl_display.c                      |  1 +
>  drivers/gpu/drm/radeon/radeon_display.c                |  1 +
>  drivers/gpu/drm/renesas/rcar-du/rcar_du_kms.c          |  3 ++-
>  drivers/gpu/drm/renesas/rz-du/rzg2l_du_kms.c           |  3 ++-
>  drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c       |  3 ++-
>  drivers/gpu/drm/rockchip/rockchip_drm_fb.c             |  7 +------
>  drivers/gpu/drm/tegra/drm.h                            |  1 +
>  drivers/gpu/drm/tegra/fb.c                             |  4 +---
>  drivers/gpu/drm/tests/drm_framebuffer_test.c           |  1 +
>  drivers/gpu/drm/vc4/vc4_kms.c                          |  3 ++-
>  drivers/gpu/drm/virtio/virtgpu_display.c               |  1 +
>  drivers/gpu/drm/vmwgfx/vmwgfx_kms.c                    |  1 +
>  drivers/gpu/drm/xen/xen_drm_front_kms.c                |  1 +
>  drivers/gpu/drm/xlnx/zynqmp_kms.c                      |  3 ++-
>  include/drm/drm_gem_framebuffer_helper.h               |  3 +++
>  include/drm/drm_mode_config.h                          |  1 +
>  38 files changed, 59 insertions(+), 49 deletions(-)

[snip]

> diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h
> index e971e1b8a850..2e848b816218 100644
> --- a/include/drm/drm_mode_config.h
> +++ b/include/drm/drm_mode_config.h
> @@ -82,6 +82,7 @@ struct drm_mode_config_funcs {
>  	 */

It would be nice to document the new parameter here. With that
addressed,

Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas at ideasonboard.com>

>  	struct drm_framebuffer *(*fb_create)(struct drm_device *dev,
>  					     struct drm_file *file_priv,
> +					     const struct drm_format_info *info,
>  					     const struct drm_mode_fb_cmd2 *mode_cmd);
>  
>  	/**

-- 
Regards,

Laurent Pinchart


More information about the dri-devel mailing list