Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/2] drm: Add plane SIZE_HINTS property
@ 2024-02-27 19:35 Ville Syrjala
  2024-02-27 19:35 ` [PATCH v2 1/2] drm: Introduce " Ville Syrjala
                   ` (4 more replies)
  0 siblings, 5 replies; 10+ messages in thread
From: Ville Syrjala @ 2024-02-27 19:35 UTC (permalink / raw)
  To: dri-devel
  Cc: intel-gfx, Simon Ser, Jonas Ådahl, Daniel Stone,
	Sameer Lattannavar, Sebastian Wick, Harry Wentland,
	Pekka Paalanen

From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Final version for a new plane SIZE_HINTS property to
essentially replace the cursor size caps, based on recent
discussion in this gitlab bug:
https://gitlab.freedesktop.org/drm/intel/-/issues/7687

As for userspace, so far I only did a quick modetest
blob decoder (mainly to verify that it looks correct):
https://gitlab.freedesktop.org/vsyrjala/libdrm/-/commits/plane_size_hints

Sameer & co. have done a real mutter implementation:
https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/3165

This final version follows the original idea of having
just a list of sizes in a blob, rather than the mode
complex "2D bitmap" approach I also proposed later.
I think that's fair, the bitmap was probably overly
complicated.

The only difference to the previous version of this approach
is that the documentation now states that the list is sorted
in order of preference, and thus userspace should pick the
first suitable size from the list. This should match the
aforementioned mutter implementation.

Cc: Simon Ser <contact@emersion.fr>
Cc: Jonas Ådahl <jadahl@redhat.com>
Cc: Daniel Stone <daniel@fooishbar.org>
Cc: Sameer Lattannavar <sameer.lattannavar@intel.com>
Cc: Sebastian Wick <sebastian.wick@redhat.com>
Cc: Harry Wentland <harry.wentland@amd.com>
Cc: Pekka Paalanen <pekka.paalanen@collabora.com>

Ville Syrjälä (2):
  drm: Introduce plane SIZE_HINTS property
  drm/i915: Add SIZE_HINTS property for cursors

 drivers/gpu/drm/drm_mode_config.c           |  7 +++
 drivers/gpu/drm/drm_plane.c                 | 52 +++++++++++++++++++++
 drivers/gpu/drm/i915/display/intel_cursor.c | 24 ++++++++++
 include/drm/drm_mode_config.h               |  5 ++
 include/drm/drm_plane.h                     |  4 ++
 include/uapi/drm/drm_mode.h                 | 11 +++++
 6 files changed, 103 insertions(+)

-- 
2.43.0


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

end of thread, other threads:[~2024-03-15 17:04 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-27 19:35 [PATCH v2 0/2] drm: Add plane SIZE_HINTS property Ville Syrjala
2024-02-27 19:35 ` [PATCH v2 1/2] drm: Introduce " Ville Syrjala
2024-02-27 20:32   ` Sebastian Wick
2024-02-28 10:12   ` Simon Ser
2024-03-15 17:03     ` Ville Syrjälä
2024-02-28 10:28   ` Daniel Stone
2024-02-27 19:35 ` [PATCH v2 2/2] drm/i915: Add SIZE_HINTS property for cursors Ville Syrjala
2024-02-27 20:12 ` ✗ Fi.CI.SPARSE: warning for drm: Add plane SIZE_HINTS property (rev5) Patchwork
2024-02-27 20:24 ` ✓ Fi.CI.BAT: success " Patchwork
2024-02-28 12:01 ` ✓ Fi.CI.IGT: " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox