All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t 00/11] lib/kms: Carve up igt_display_require()
@ 2025-12-17 15:37 Ville Syrjala
  2025-12-17 15:37 ` [PATCH i-g-t 01/11] lib/kms: Extract igt_crtc_init() Ville Syrjala
                   ` (13 more replies)
  0 siblings, 14 replies; 31+ messages in thread
From: Ville Syrjala @ 2025-12-17 15:37 UTC (permalink / raw)
  To: igt-dev

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

Coccinelle really doesn't like igt_display_require(). Sometimes
it takes approximately forever to do any changes to igt_kms.c
because of this. Splitting the function up seems to help a bit,
or at least then some modifications can be done in finite time.

While at it try to clean up the plane->index stuff a bit.

Ville Syrjälä (11):
  lib/kms: Extract igt_crtc_init()
  lib/kms: Nuke 'n_planes' from igt_crtc_init()
  lib/kms: Nuke 'last_plane' from igt_crtc_init()
  lib/kms: Get rid of the 'p' plane index variable
  lib/kms: Extract igt_crtc_plane_init()
  lib/kms: Nuke pipe->plane_primary
  lib/kms: Nuke pipe->plane_cursor
  lib/kms: Use '{old,new}_primary' variable names in plane swapping
  lib/kms: Pimp the priamry plane swapping
  lib/kms: Extract plane_type_index()
  lib/kms: Streamline plane index handling further

 lib/igt_kms.c | 269 ++++++++++++++++++++++++++------------------------
 lib/igt_kms.h |   2 -
 2 files changed, 141 insertions(+), 130 deletions(-)

-- 
2.51.2


^ permalink raw reply	[flat|nested] 31+ messages in thread
* [PATCH i-g-t v2 00/11] lib/kms: Carve up igt_display_require()
@ 2025-12-19 20:33 Ville Syrjala
  2025-12-19 20:33 ` [PATCH i-g-t v2 04/11] lib/kms: Get rid of the 'p' plane index variable Ville Syrjala
  0 siblings, 1 reply; 31+ messages in thread
From: Ville Syrjala @ 2025-12-19 20:33 UTC (permalink / raw)
  To: igt-dev

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

Coccinelle really doesn't like igt_display_require(). Sometimes
it takes approximately forever to do any changes to igt_kms.c
because of this. Splitting the function up seems to help a bit,
or at least then some modifications can be done in finite time.

While at it try to clean up the plane->index stuff a bit.

v2: Fix plane index asserts
    Fix whitespace issues
    Fix typos in commit messages

Ville Syrjälä (11):
  lib/kms: Extract igt_crtc_init()
  lib/kms: Nuke 'n_planes' from igt_crtc_init()
  lib/kms: Nuke 'last_plane' from igt_crtc_init()
  lib/kms: Get rid of the 'p' plane index variable
  lib/kms: Extract igt_crtc_plane_init()
  lib/kms: Nuke pipe->plane_primary
  lib/kms: Nuke pipe->plane_cursor
  lib/kms: Use '{old,new}_primary' variable names in plane swapping
  lib/kms: Pimp the primary plane swapping
  lib/kms: Extract plane_type_index()
  lib/kms: Streamline plane index handling further

 lib/igt_kms.c | 270 ++++++++++++++++++++++++++------------------------
 lib/igt_kms.h |   2 -
 2 files changed, 142 insertions(+), 130 deletions(-)

-- 
2.51.2


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

end of thread, other threads:[~2025-12-19 20:33 UTC | newest]

Thread overview: 31+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-17 15:37 [PATCH i-g-t 00/11] lib/kms: Carve up igt_display_require() Ville Syrjala
2025-12-17 15:37 ` [PATCH i-g-t 01/11] lib/kms: Extract igt_crtc_init() Ville Syrjala
2025-12-17 19:04   ` Jani Nikula
2025-12-17 15:37 ` [PATCH i-g-t 02/11] lib/kms: Nuke 'n_planes' from igt_crtc_init() Ville Syrjala
2025-12-17 19:11   ` Jani Nikula
2025-12-17 15:37 ` [PATCH i-g-t 03/11] lib/kms: Nuke 'last_plane' " Ville Syrjala
2025-12-17 19:12   ` Jani Nikula
2025-12-17 15:37 ` [PATCH i-g-t 04/11] lib/kms: Get rid of the 'p' plane index variable Ville Syrjala
2025-12-17 19:17   ` Jani Nikula
2025-12-18 11:33   ` Jani Nikula
2025-12-18 15:47   ` [PATCH i-g-t v2 " Ville Syrjala
2025-12-19 10:55     ` Jani Nikula
2025-12-17 15:37 ` [PATCH i-g-t 05/11] lib/kms: Extract igt_crtc_plane_init() Ville Syrjala
2025-12-17 19:18   ` Jani Nikula
2025-12-18 15:48   ` [PATCH i-g-t v2 " Ville Syrjala
2025-12-17 15:37 ` [PATCH i-g-t 06/11] lib/kms: Nuke pipe->plane_primary Ville Syrjala
2025-12-18 11:37   ` Jani Nikula
2025-12-17 15:37 ` [PATCH i-g-t 07/11] lib/kms: Nuke pipe->plane_cursor Ville Syrjala
2025-12-18 11:39   ` Jani Nikula
2025-12-17 15:37 ` [PATCH i-g-t 08/11] lib/kms: Use '{old, new}_primary' variable names in plane swapping Ville Syrjala
2025-12-17 19:20   ` Jani Nikula
2025-12-17 15:37 ` [PATCH i-g-t 09/11] lib/kms: Pimp the priamry " Ville Syrjala
2025-12-19 12:00   ` Jani Nikula
2025-12-17 15:37 ` [PATCH i-g-t 10/11] lib/kms: Extract plane_type_index() Ville Syrjala
2025-12-19 12:08   ` Jani Nikula
2025-12-17 15:37 ` [PATCH i-g-t 11/11] lib/kms: Streamline plane index handling further Ville Syrjala
2025-12-19 12:10   ` Jani Nikula
2025-12-17 20:36 ` ✗ Xe.CI.BAT: failure for lib/kms: Carve up igt_display_require() Patchwork
2025-12-17 21:00 ` ✗ i915.CI.BAT: " Patchwork
2025-12-18 20:21 ` ✗ Xe.CI.Full: " Patchwork
  -- strict thread matches above, loose matches on Subject: below --
2025-12-19 20:33 [PATCH i-g-t v2 00/11] " Ville Syrjala
2025-12-19 20:33 ` [PATCH i-g-t v2 04/11] lib/kms: Get rid of the 'p' plane index variable Ville Syrjala

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.