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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux