Hi Thomas, On Wed, Jul 20, 2022 at 10:30:51AM +0200, Thomas Zimmermann wrote: > The plane helpers are included by dozens of files without any need. Only > a hand full of source files need anything from drm_plane_helper.h. > > Untangle everything and tidy up the code a bit. The patches were built > on x64-64, aarch64 and arm without issues. Build tested the final result here on a few more archs without any issues. I like the removal of the include statements and the simpler header file. I did not really get what is gained by pushing out drm_plane_funcs to the drivers but trust this is in the end a better solution. Series is: Reviewed-by: Sam Ravnborg <sam@xxxxxxxxxxxx> Sam