On Thu, Dec 09, 2021 at 03:14:52PM +0200, Jani Nikula wrote: > On Fri, 03 Dec 2021, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > These plane/pipe->crtc mapping arrays are rather pointless. > > Get rid of them and just iterate the lists instead. > > Nice follow-up to my cleanups. On the series, > > Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > > intel_crtc_first() might have been nice for the prefix, but does it > sound too awkward? *shrug* Yeah, sounds a bit wonky to my ear. I'd probably have of those "wtf does that do?" moments if I saw a intel_crtc_first() call in the code. -- Ville Syrjälä Intel