All of lore.kernel.org
 help / color / mirror / Atom feed
* [RFC PATCH v3 0/3] Add support modifiers for drivers whose planes only support linear layout
@ 2022-01-14 10:17 ` Tomohito Esaki
  0 siblings, 0 replies; 72+ messages in thread
From: Tomohito Esaki @ 2022-01-14 10:17 UTC (permalink / raw)
  To: dri-devel
  Cc: Alex Deucher, Christian König, Pan, Xinhui, David Airlie,
	Daniel Vetter, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, Ben Skeggs, Michel Dänzer, Simon Ser,
	Qingqing Zhuo, Bas Nieuwenhuizen, Mark Yacoub, Sean Paul,
	Evan Quan, Andy Shevchenko, Petr Mladek, Sakari Ailus, Lee Jones,
	Abhinav Kumar, Dmitry Baryshkov, Rob Clark, amd-gfx,
	linux-kernel, nouveau, Daniel Stone, Laurent Pinchart,
	Tomohito Esaki, Damian Hobson-Garcia, Takanari Hayama

Some drivers whose planes only support linear layout fb do not support format
modifiers.
These drivers should support modifiers, however the DRM core should handle this
rather than open-coding in every driver.

In this patch series, these drivers expose format modifiers based on the
following suggestion[1].

On Thu, Nov 18, 2021 at 01:02:11PM +0000, Daniel Stone wrote:
> I think the best way forward here is:
>   - add a new mode_config.cannot_support_modifiers flag, and enable
> this in radeon (plus any other drivers in the same boat)
>   - change drm_universal_plane_init() to advertise the LINEAR modifier
> when NULL is passed as the modifier list (including installing a
> default .format_mod_supported hook)
>   - remove the mode_config.allow_fb_modifiers hook and always
> advertise modifier support, unless
> mode_config.cannot_support_modifiers is set


[1] https://patchwork.kernel.org/project/linux-renesas-soc/patch/20190509054518.10781-1-etom@igel.co.jp/#24602575

v3:
* change the order as follows:
   1. add fb_modifiers_not_supported flag
   2. add default modifiers
   3. remove allow_fb_modifiers flag
* add a conditional disable in amdgpu_dm_plane_init()

v2: https://www.spinics.net/lists/dri-devel/msg328939.html
* rebase to the latest master branch (5.16.0+) + "drm/plane: Make format_mod_supported truly optional" patch [2]
  [2] https://patchwork.freedesktop.org/patch/467940/?series=98255&rev=3

v1: https://www.spinics.net/lists/dri-devel/msg327352.html
* The initial patch set

Tomohito Esaki (3):
  drm: introduce fb_modifiers_not_supported flag in mode_config
  drm: add support modifiers for drivers whose planes only support
    linear layout
  drm: remove allow_fb_modifiers

 drivers/gpu/drm/amd/amdgpu/amdgpu_display.c   |  6 ++---
 drivers/gpu/drm/amd/amdgpu/dce_v10_0.c        |  2 ++
 drivers/gpu/drm/amd/amdgpu/dce_v11_0.c        |  2 ++
 drivers/gpu/drm/amd/amdgpu/dce_v6_0.c         |  1 +
 drivers/gpu/drm/amd/amdgpu/dce_v8_0.c         |  2 ++
 .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |  3 +++
 drivers/gpu/drm/drm_framebuffer.c             |  6 ++---
 drivers/gpu/drm/drm_ioctl.c                   |  2 +-
 drivers/gpu/drm/drm_plane.c                   | 22 +++++++++----------
 drivers/gpu/drm/nouveau/nouveau_display.c     |  6 +++--
 drivers/gpu/drm/radeon/radeon_display.c       |  2 ++
 .../gpu/drm/selftests/test-drm_framebuffer.c  |  1 -
 include/drm/drm_mode_config.h                 | 18 +++++----------
 include/drm/drm_plane.h                       |  3 +++
 14 files changed, 43 insertions(+), 33 deletions(-)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 72+ messages in thread

end of thread, other threads:[~2022-01-26 18:21 UTC | newest]

Thread overview: 72+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-14 10:17 [RFC PATCH v3 0/3] Add support modifiers for drivers whose planes only support linear layout Tomohito Esaki
2022-01-14 10:17 ` [Nouveau] " Tomohito Esaki
2022-01-14 10:17 ` Tomohito Esaki
2022-01-14 10:17 ` Tomohito Esaki
2022-01-14 10:17 ` [RFC PATCH v3 1/3] drm: introduce fb_modifiers_not_supported flag in mode_config Tomohito Esaki
2022-01-14 10:17   ` [Nouveau] " Tomohito Esaki
2022-01-14 10:17   ` Tomohito Esaki
2022-01-14 10:17   ` Tomohito Esaki
2022-01-14 14:43   ` Harry Wentland
2022-01-14 14:43     ` Harry Wentland
2022-01-14 14:43     ` Harry Wentland
2022-01-14 14:43     ` [Nouveau] " Harry Wentland
2022-01-23 22:50   ` Laurent Pinchart
2022-01-23 22:50     ` Laurent Pinchart
2022-01-23 22:50     ` Laurent Pinchart
2022-01-23 22:50     ` [Nouveau] " Laurent Pinchart
2022-01-25  5:19     ` Esaki Tomohito
2022-01-25  5:19       ` [Nouveau] " Esaki Tomohito
2022-01-25  5:19       ` Esaki Tomohito
2022-01-25  5:19       ` Esaki Tomohito
2022-01-14 10:17 ` [RFC PATCH v3 2/3] drm: add support modifiers for drivers whose planes only support linear layout Tomohito Esaki
2022-01-14 10:17   ` [Nouveau] " Tomohito Esaki
2022-01-14 10:17   ` Tomohito Esaki
2022-01-14 10:17   ` Tomohito Esaki
2022-01-14 14:16   ` Andy Shevchenko
2022-01-14 14:16     ` Andy Shevchenko
2022-01-14 14:16     ` Andy Shevchenko
2022-01-14 14:16     ` [Nouveau] " Andy Shevchenko
2022-01-14 15:07     ` Simon Ser
2022-01-14 15:07       ` Simon Ser
2022-01-14 15:07       ` Simon Ser
2022-01-14 15:07       ` [Nouveau] " Simon Ser
2022-01-14 15:17       ` Andy Shevchenko
2022-01-14 15:17         ` Andy Shevchenko
2022-01-14 15:17         ` Andy Shevchenko
2022-01-14 15:17         ` [Nouveau] " Andy Shevchenko
2022-01-14 15:42         ` Simon Ser
2022-01-14 15:42           ` Simon Ser
2022-01-14 15:42           ` Simon Ser
2022-01-14 15:42           ` [Nouveau] " Simon Ser
2022-01-14 16:06           ` Andy Shevchenko
2022-01-14 16:06             ` Andy Shevchenko
2022-01-14 16:06             ` Andy Shevchenko
2022-01-14 16:06             ` [Nouveau] " Andy Shevchenko
2022-01-17  5:15     ` Esaki Tomohito
2022-01-17  5:15       ` [Nouveau] " Esaki Tomohito
2022-01-17  5:15       ` Esaki Tomohito
2022-01-17  5:15       ` Esaki Tomohito
2022-01-18  9:53       ` Andy Shevchenko
2022-01-18  9:53         ` Andy Shevchenko
2022-01-18  9:53         ` [Nouveau] " Andy Shevchenko
2022-01-18  9:53         ` Andy Shevchenko
2022-01-19  2:35         ` Esaki Tomohito
2022-01-19  2:35           ` [Nouveau] " Esaki Tomohito
2022-01-19  2:35           ` Esaki Tomohito
2022-01-19  2:35           ` Esaki Tomohito
2022-01-19 19:17           ` Andy Shevchenko
2022-01-19 19:17             ` Andy Shevchenko
2022-01-19 19:17             ` Andy Shevchenko
2022-01-19 19:17             ` [Nouveau] " Andy Shevchenko
2022-01-23 22:37   ` Laurent Pinchart
2022-01-23 22:37     ` Laurent Pinchart
2022-01-23 22:37     ` Laurent Pinchart
2022-01-23 22:37     ` [Nouveau] " Laurent Pinchart
2022-01-14 10:17 ` [RFC PATCH v3 3/3] drm: remove allow_fb_modifiers Tomohito Esaki
2022-01-14 10:17   ` [Nouveau] " Tomohito Esaki
2022-01-14 10:17   ` Tomohito Esaki
2022-01-14 10:17   ` Tomohito Esaki
2022-01-23 22:44   ` Laurent Pinchart
2022-01-23 22:44     ` Laurent Pinchart
2022-01-23 22:44     ` Laurent Pinchart
2022-01-23 22:44     ` [Nouveau] " Laurent Pinchart

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.