[PATCH v2 04/19] drm: Pass the format info to .fb_create()
Ville Syrjälä
ville.syrjala at linux.intel.com
Fri Jul 4 09:56:07 UTC 2025
On Tue, Jul 01, 2025 at 10:18:01PM +0300, Laurent Pinchart wrote:
> 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,
None of the existing parameters are documented (for any of the
mode_config hooks actually), so I think we'd need a separate
patch to deal with this.
>
> 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
--
Ville Syrjälä
Intel
More information about the dri-devel
mailing list