On Mon, Aug 22, 2016 at 12:38 PM, Rob Clark <robdclark@xxxxxxxxx> wrote: >> That said, _note_ that some applications are built with -C89 -pedantic >> [1] which means that using stdint.h may or may not work as expected. >> So we'll want a __STDC_VESION__ check + #error in case of pre-C99 ? >> If the affected programs are proprietary ones we should be safe, >> otherwise we want to update them ~alongside the transition. > > naw, at least for msm_drm.h, just don't build libdrm_freedreno w/ > -C89.. problem solved! Yeah, I think sprinkling an #ifdef __kernel___ #include <linux/types.h> #else #include <stdtypes.h> #endif at the opt of all drm uapi headers should be good enough. Or at least those which opt to choose stdints. Since our userspace is very limited, and our headers will never leak to general applications we can just require c99, at least for driver headers. For kms/general drm uapi that might not be the best idea. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
