On Thu, Dec 09, 2021 at 03:14:52PM +0200, Jani Nikula wrote: > On Fri, 03 Dec 2021, Ville Syrjala <[email protected]> wrote: > > From: Ville Syrjälä <[email protected]> > > > > 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 <[email protected]> > > 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
