Re: [PATCH i-g-t 1/8] igt: Name drmModeCrtc variables 'drm_crtc'

["Ville Syrjälä" <ville.syrjala@linux.intel.com>]

On Mon, Nov 24, 2025 at 11:37:23AM +0200, Jani Nikula wrote:
> On Fri, 21 Nov 2025, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >
> > The current situation with pipes vs. crtcs is a giant mess.
> > In order to untangle things properly I think we need to
> > move away from the pipe stuff into a more pure crtc world.
> >
> > I suspect the best way to get there is by renaming igt_pipe_t
> > into igt_crtc_t and making everyone use it instead of the raw
> > index/pipe numbers. And presumable we'll want to call the
> > igt_crtc_t variables 'crtc', so let's clear out the namespace
> > for that by using 'drm_crtc' as the variable name for the
> > lower level drmModeCrtc stuff.
> 
> For clarity, I suppose also s/crtc_offset/crtc_index/g for the member in
> igt_pipe_t (igt_crtc_t) would be in order as follow-up.

Yeah, IIRC I have that in one of my wip cocci scripts as well.

-- 
Ville Syrjälä
Intel