That would work, but that's kind of an annoying requirement. I would prefer the header to be self-sufficient.

Thanks,

James

On Mon, Dec 7, 2020 at 2:47 AM Simon Ser <contact@emersion.fr> wrote:

On Monday, December 7th, 2020 at 11:44 AM, Pekka Paalanen <ppaalanen@gmail.com> wrote:

> But then, the code in the header should be literally
>
> #ifndef DRM_FOURCC_STANDALONE
> #include "drm.h"
> #endif
>
> without an #else branch.

As long as there is a #include "drm_basic_types.h" right before
(drm_fourcc.h needs __u32 and __u64), I believe this can work too
indeed.