Hi Ville,

Thank you for the patch.

On Tue, Jul 01, 2025 at 12:07:04PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> 
> Decouple .get_format_info() from struct drm_mode_fb_cmd2 and just
> pass the pixel format+modifier combo in by hand.
> 
> We may want to use .get_format_info() outside of the normal
> addfb paths where we won't have a struct drm_mode_fb_cmd2, and
> creating a temporary one just for this seems silly.
> 
> v2: Fix intel_fb_get_format_info() docs (Laurent)
> 
> Cc: Harry Wentland <harry.wentl...@amd.com>
> Cc: Leo Li <sunpeng...@amd.com>
> Cc: Rodrigo Siqueira <sique...@igalia.com>
> Cc: Alex Deucher <alexander.deuc...@amd.com>
> Cc: amd-...@lists.freedesktop.org
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Reviewed-by: Thomas Zimmermann <tzimmerm...@suse.de>
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>

> ---
>  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c   |  4 ++--
>  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.h   |  2 +-
>  drivers/gpu/drm/drm_fourcc.c                          |  3 ++-
>  drivers/gpu/drm/i915/display/intel_fb.c               | 11 ++++++-----
>  drivers/gpu/drm/i915/display/intel_fb.h               |  2 +-
>  include/drm/drm_mode_config.h                         |  2 +-
>  6 files changed, 13 insertions(+), 11 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c 
> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c
> index b7c6e8d13435..eef51652ca35 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c
> @@ -92,9 +92,9 @@ enum dm_micro_swizzle {
>       MICRO_SWIZZLE_R = 3
>  };
>  
> -const struct drm_format_info *amdgpu_dm_plane_get_format_info(const struct 
> drm_mode_fb_cmd2 *cmd)
> +const struct drm_format_info *amdgpu_dm_plane_get_format_info(u32 
> pixel_format, u64 modifier)
>  {
> -     return amdgpu_lookup_format_info(cmd->pixel_format, cmd->modifier[0]);
> +     return amdgpu_lookup_format_info(pixel_format, modifier);
>  }
>  
>  void amdgpu_dm_plane_fill_blending_from_plane_state(const struct 
> drm_plane_state *plane_state,
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.h 
> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.h
> index 615d2ab2b803..ea2619b507db 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.h
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.h
> @@ -58,7 +58,7 @@ int amdgpu_dm_plane_init(struct amdgpu_display_manager *dm,
>                        unsigned long possible_crtcs,
>                        const struct dc_plane_cap *plane_cap);
>  
> -const struct drm_format_info *amdgpu_dm_plane_get_format_info(const struct 
> drm_mode_fb_cmd2 *cmd);
> +const struct drm_format_info *amdgpu_dm_plane_get_format_info(u32 
> pixel_format, u64 modifier);
>  
>  void amdgpu_dm_plane_fill_blending_from_plane_state(const struct 
> drm_plane_state *plane_state,
>                                   bool *per_pixel_alpha, bool 
> *pre_multiplied_alpha,
> diff --git a/drivers/gpu/drm/drm_fourcc.c b/drivers/gpu/drm/drm_fourcc.c
> index 2890e889dd15..4b4444f6d504 100644
> --- a/drivers/gpu/drm/drm_fourcc.c
> +++ b/drivers/gpu/drm/drm_fourcc.c
> @@ -430,7 +430,8 @@ drm_get_format_info(struct drm_device *dev,
>       const struct drm_format_info *info = NULL;
>  
>       if (dev->mode_config.funcs->get_format_info)
> -             info = dev->mode_config.funcs->get_format_info(mode_cmd);
> +             info = 
> dev->mode_config.funcs->get_format_info(mode_cmd->pixel_format,
> +                                                            
> mode_cmd->modifier[0]);
>  
>       if (!info)
>               info = drm_format_info(mode_cmd->pixel_format);
> diff --git a/drivers/gpu/drm/i915/display/intel_fb.c 
> b/drivers/gpu/drm/i915/display/intel_fb.c
> index 6158031821fd..52c4901dc072 100644
> --- a/drivers/gpu/drm/i915/display/intel_fb.c
> +++ b/drivers/gpu/drm/i915/display/intel_fb.c
> @@ -422,21 +422,22 @@ unsigned int intel_fb_modifier_to_tiling(u64 
> fb_modifier)
>  
>  /**
>   * intel_fb_get_format_info: Get a modifier specific format information
> - * @cmd: FB add command structure
> + * @pixel_format: pixel format
> + * @modifier: modifier
>   *
>   * Returns:
> - * Returns the format information for @cmd->pixel_format specific to 
> @cmd->modifier[0],
> + * Returns the format information for @pixel_format specific to @modifier,
>   * or %NULL if the modifier doesn't override the format.
>   */
>  const struct drm_format_info *
> -intel_fb_get_format_info(const struct drm_mode_fb_cmd2 *cmd)
> +intel_fb_get_format_info(u32 pixel_format, u64 modifier)
>  {
> -     const struct intel_modifier_desc *md = 
> lookup_modifier_or_null(cmd->modifier[0]);
> +     const struct intel_modifier_desc *md = 
> lookup_modifier_or_null(modifier);
>  
>       if (!md || !md->formats)
>               return NULL;
>  
> -     return lookup_format_info(md->formats, md->format_count, 
> cmd->pixel_format);
> +     return lookup_format_info(md->formats, md->format_count, pixel_format);
>  }
>  
>  static bool plane_caps_contain_any(u8 caps, u8 mask)
> diff --git a/drivers/gpu/drm/i915/display/intel_fb.h 
> b/drivers/gpu/drm/i915/display/intel_fb.h
> index bdd76b372957..7d1267fbeee2 100644
> --- a/drivers/gpu/drm/i915/display/intel_fb.h
> +++ b/drivers/gpu/drm/i915/display/intel_fb.h
> @@ -47,7 +47,7 @@ u64 *intel_fb_plane_get_modifiers(struct intel_display 
> *display,
>  bool intel_fb_plane_supports_modifier(struct intel_plane *plane, u64 
> modifier);
>  
>  const struct drm_format_info *
> -intel_fb_get_format_info(const struct drm_mode_fb_cmd2 *cmd);
> +intel_fb_get_format_info(u32 pixel_format, u64 modifier);
>  
>  bool
>  intel_format_info_is_yuv_semiplanar(const struct drm_format_info *info,
> diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h
> index 9e524b51a001..e971e1b8a850 100644
> --- a/include/drm/drm_mode_config.h
> +++ b/include/drm/drm_mode_config.h
> @@ -95,7 +95,7 @@ struct drm_mode_config_funcs {
>        * The format information specific to the given fb metadata, or
>        * NULL if none is found.
>        */
> -     const struct drm_format_info *(*get_format_info)(const struct 
> drm_mode_fb_cmd2 *mode_cmd);
> +     const struct drm_format_info *(*get_format_info)(u32 pixel_format, u64 
> modifier);
>  
>       /**
>        * @mode_valid:

-- 
Regards,

Laurent Pinchart

Reply via email to