<div dir="ltr">I'm not sure what your suggestion is. Move which #ifdef block where?<div><br></div><div>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.</div><div><br></div><div>I appreciate the feedback though.</div><div><br></div><div>- James</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 2, 2021 at 9:28 AM Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi James,<br>
<br>
On Tue, 2 Feb 2021 at 08:27, James Park <<a href="mailto:james.park@lagfreegames.com" target="_blank">james.park@lagfreegames.com</a>> wrote:<br>
><br>
> Hello,<br>
><br>
> Is there something I can do to help move this patch along?<br>
><br>
> Thanks,<br>
> James Park<br>
><br>
> On Thu, Dec 10, 2020 at 1:13 AM James Park <<a href="mailto:jpark37@lagfreegames.com" target="_blank">jpark37@lagfreegames.com</a>> wrote:<br>
>><br>
>> Create drm_basic_types.h to define types previously defined by drm.h.<br>
>><br>
>> Use DRM_FOURCC_STANDALONE to include drm_fourcc.h without drm.h.<br>
>><br>
>> This will allow Mesa to port code to Windows more easily.<br>
>><br>
>> Signed-off-by: James Park <<a href="mailto:jpark37@lagfreegames.com" target="_blank">jpark37@lagfreegames.com</a>><br>
>> Acked-by: Simon Ser <<a href="mailto:contact@emersion.fr" target="_blank">contact@emersion.fr</a>><br>
>> Reviewed-by: Pekka Paalanen <<a href="mailto:pekka.paalanen@collabora.com" target="_blank">pekka.paalanen@collabora.com</a>><br>
>> ---<br>
>>  include/uapi/drm/drm.h             | 12 ++-------<br>
>>  include/uapi/drm/drm_basic_types.h | 52 ++++++++++++++++++++++++++++++++++++++<br>
>>  include/uapi/drm/drm_fourcc.h      |  4 +++<br>
>>  3 files changed, 58 insertions(+), 10 deletions(-)<br>
>>  create mode 100644 include/uapi/drm/drm_basic_types.h<br>
>><br>
Have you considered the possibility of having the ifdef block inlined<br>
within drm_fourcc.h?<br>
<br>
Sure some users might need to add an drm.h include in their code. At<br>
the same time they also need to explicitly define<br>
DRM_FOURCC_STANDALONE, so that is fine.<br>
We had all sorts of issues with these headers in the past, so adding<br>
another one might end up repeating some of those yet again.<br>
<br>
Thanks<br>
Emil<br>
</blockquote></div>