<div dir="ltr">Oh, I thought you meant:<div><br></div><div>#include "drm_basic_types.h"</div><div>#include "drm_fourcc.h"</div><div><br></div><div>Yours should work too. I don't have a preference vs. what I already have, so if no one says anything, I can switch over to yours.</div><div><br></div><div>Thanks,</div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 7, 2020 at 2:53 AM Simon Ser <<a href="mailto:contact@emersion.fr">contact@emersion.fr</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">On Monday, December 7th, 2020 at 11:49 AM, James Park <<a href="mailto:james.park@lagfreegames.com" target="_blank">james.park@lagfreegames.com</a>> wrote:<br>
<br>
> That would work, but that's kind of an annoying requirement. I would<br>
> prefer the header to be self-sufficient.<br>
<br>
I don't want to make it more confusing than before, but here Pekka (I<br>
think) suggests to replace this:<br>
<br>
diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h<br>
index 82f3278..5eb07a5 100644<br>
--- a/include/uapi/drm/drm_fourcc.h<br>
+++ b/include/uapi/drm/drm_fourcc.h<br>
@@ -24,7 +24,11 @@<br>
 #ifndef DRM_FOURCC_H<br>
 #define DRM_FOURCC_H<br>
<br>
+#ifdef DRM_FOURCC_STANDALONE<br>
+#include "drm_basic_types.h"<br>
+#else<br>
 #include "drm.h"<br>
+#endif<br>
<br>
 #if defined(__cplusplus)<br>
 extern "C" {<br>
<br>
With this:<br>
<br>
diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h<br>
index 82f3278..5eb07a5 100644<br>
--- a/include/uapi/drm/drm_fourcc.h<br>
+++ b/include/uapi/drm/drm_fourcc.h<br>
@@ -24,7 +24,11 @@<br>
 #ifndef DRM_FOURCC_H<br>
 #define DRM_FOURCC_H<br>
<br>
+#include "drm_basic_types.h"<br>
+<br>
+#ifndef DRM_FOURCC_STANDALONE<br>
 #include "drm.h"<br>
+#endif<br>
<br>
 #if defined(__cplusplus)<br>
 extern "C" {<br>
<br>
That wouldn't change whether the header is self-sufficient or not,<br>
would it?<br>
</blockquote></div>