From: Sam Ravnborg <sam@ravnborg.org>
To: dri-devel@lists.freedesktop.org
Cc: Maxime Ripard <maxime.ripard@bootlin.com>,
Sean Paul <sean@poorly.run>, David Airlie <airlied@linux.ie>
Subject: [PATCH v1 0/6] drm: make headers self-contained
Date: Sun, 19 May 2019 16:20:30 +0200 [thread overview]
Message-ID: <20190519142036.22861-1-sam@ravnborg.org> (raw)
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@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next reply other threads:[~2019-05-19 14:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-19 14:20 Sam Ravnborg [this message]
2019-05-19 14:20 ` [PATCH v1 01/06] drm: make drm_auth.h self contained Sam Ravnborg
2019-05-19 14:20 ` [PATCH v1 02/06] drm: make drm_legacy.h self-contained Sam Ravnborg
2019-05-19 14:20 ` [PATCH v1 03/06] drm: make drm_crtc_internal.h self-contained Sam Ravnborg
2019-05-19 14:20 ` [PATCH v1 04/06] drm: make drm_internal.h self-contained Sam Ravnborg
2019-05-19 14:20 ` [PATCH v1 05/06] drm: make drm_legacy.h self-contained Sam Ravnborg
2019-05-19 14:20 ` [PATCH v1 06/06] drm: make drm_trace.h self-contained Sam Ravnborg
2019-05-20 18:45 ` [PATCH v1 0/6] drm: make headers self-contained Daniel Vetter
2019-05-20 19:13 ` Sam Ravnborg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190519142036.22861-1-sam@ravnborg.org \
--to=sam@ravnborg.org \
--cc=airlied@linux.ie \
--cc=dri-devel@lists.freedesktop.org \
--cc=maxime.ripard@bootlin.com \
--cc=sean@poorly.run \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.