[PATCH v1 0/6] drm: make headers self-contained

Sam Ravnborg sam at ravnborg.org
Sun May 19 14:20:30 UTC 2019


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(-)



More information about the dri-devel mailing list