While removing use of drmP.h from files in drm/* I noticed that I had to add the same include files due to build errors in the header files. It is better to let the header files include what is necessary and let the users pull in only the additional headers files required. So I went ahead and made the header files self-conatined. (I did not check if this made any includes redundant in some files, I do not have tooling in place to do so). These patches are preparation for removing use of drmP.h in all the files in drm/* I have the patchset ready - one patch per file. It is a lot of small patches (59) and maybe it is better to squash them all into one big patch. Any opinions on this? I will await feedback on this patchset before sending the patch(es) to remove drmP.h in drm/* Sam Sam Ravnborg (6): drm: make drm_auth.h self contained drm: make drm_legacy.h self-contained drm: make drm_crtc_internal.h self-contained drm: make drm_internal.h self-contained drm: make drm_legacy.h self-contained drm: make drm_trace.h self-contained drivers/gpu/drm/drm_crtc_internal.h | 24 ++++++++++++++++++++---- drivers/gpu/drm/drm_internal.h | 10 +++++++++- drivers/gpu/drm/drm_legacy.h | 4 ++++ drivers/gpu/drm/drm_trace.h | 2 ++ include/drm/drm_auth.h | 7 +++++++ include/drm/drm_legacy.h | 2 ++ 6 files changed, 44 insertions(+), 5 deletions(-) _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel