<div dir="ltr"><div>The original code blocks in drm.h look identical to me. I see:</div><div><br></div><div>#include <linux/types.h><br>#include <asm/ioctl.h><br>typedef unsigned int drm_handle_t;<br></div><div><br></div>Good point about drm_basic_types.h. I'll change it to say "Not Linux" after waiting a bit for more feedback.<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 1:59 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 10:55 AM, James Park <<a href="mailto:james.park@lagfreegames.com" target="_blank">james.park@lagfreegames.com</a>> wrote:<br>
<br>
> I'd noticed the #if could be combined, but they weren't in drm,h when<br>
> they could have been, so I didn't want to depart from the existing<br>
> pattern.<br>
<br>
I see. The original code really needed the two branches and<br>
drm_basic_types.h doesn't. But let's see what others think.<br>
<br>
> I think "One of the BSDs" is more informative than "Not Linux" if<br>
> that statement is still true. Given the aversion to making drm.h<br>
> robust to Windows, I don't think we want to imply compatibility that<br>
> isn't there.<br>
<br>
Well, drm_basic_types.h would be included on Windows as well from<br>
drm_fourcc.h, right?<br>
</blockquote></div>