[PATCH] drm: drm_basic_types.h, DRM_FOURCC_STANDALONE

[James Park]

I'm not sure what your suggestion is. Move which #ifdef block where?

I don't think everyone is of the opinion that adding drm.h and pulling in
unnecessary kernel structures is fine. If I'm not mistaken, the reason
people are making me jump through hoops in the first place is to avoid that.

I appreciate the feedback though.

- James

On Tue, Feb 2, 2021 at 9:28 AM Emil Velikov <emil.l.velikov at gmail.com>
wrote:

> Hi James,
>
> On Tue, 2 Feb 2021 at 08:27, James Park <james.park at lagfreegames.com>
> wrote:
> >
> > Hello,
> >
> > Is there something I can do to help move this patch along?
> >
> > Thanks,
> > James Park
> >
> > On Thu, Dec 10, 2020 at 1:13 AM James Park <jpark37 at lagfreegames.com>
> wrote:
> >>
> >> Create drm_basic_types.h to define types previously defined by drm.h.
> >>
> >> Use DRM_FOURCC_STANDALONE to include drm_fourcc.h without drm.h.
> >>
> >> This will allow Mesa to port code to Windows more easily.
> >>
> >> Signed-off-by: James Park <jpark37 at lagfreegames.com>
> >> Acked-by: Simon Ser <contact at emersion.fr>
> >> Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.com>
> >> ---
> >>  include/uapi/drm/drm.h             | 12 ++-------
> >>  include/uapi/drm/drm_basic_types.h | 52
> ++++++++++++++++++++++++++++++++++++++
> >>  include/uapi/drm/drm_fourcc.h      |  4 +++
> >>  3 files changed, 58 insertions(+), 10 deletions(-)
> >>  create mode 100644 include/uapi/drm/drm_basic_types.h
> >>
> Have you considered the possibility of having the ifdef block inlined
> within drm_fourcc.h?
>
> Sure some users might need to add an drm.h include in their code. At
> the same time they also need to explicitly define
> DRM_FOURCC_STANDALONE, so that is fine.
> We had all sorts of issues with these headers in the past, so adding
> another one might end up repeating some of those yet again.
>
> Thanks
> Emil
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20210202/d30a2ba6/attachment-0001.htm>