Hi Ville, On Mon, Jun 13, 2022 at 11:03:09PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Eliminate unnecessary includes from drm_crtc.h to avoid > pointless rebuilds of the entire universe when touching > some random header. > > I didn't really feel like splitting this up per-driver since > that would have ended up being metric ton of one liners. > I'm thinking the conflicts (if any) should be trivial enough > to deal with even with bigger patches. Thanks for doing this! I would have been fine with a single bigger commit, as this is tedious manually jobs. It really does not make much sense if there is added 1 or 4 includes files when reviewing. And the bots will tell if build is broken somewhere. If we relax a little in the patch granularity, the chances we will see more similar cleanups are higher. I looked at a few - for the rest I trust the robots. So with the reported builderrros fixed and my alphabetic order comment addressed, then the full series are: Acked-by: Sam Ravnborg <sam@xxxxxxxxxxxx> Sam
