On Fri, 13 Nov 2015 20:38:51 +0000 Emil Velikov <emil.l.velikov@xxxxxxxxx> wrote: > On 13 November 2015 at 19:31, Gabriel Laskar <gabriel@xxxxxxxxxxxx> wrote: > > On Fri, 13 Nov 2015 20:24:30 +0100 > > Gabriel Laskar <gabriel@xxxxxxxxxxxx> wrote: > > > > Sorry for the noise, I had just saw the first version. > > > Heh I was wondering what's happening. I guess I should have been more > helpful and given you a link, rather than just throwing you a name. > Sorry about that. No worries, I will check a little more next time. > From a very quick look I believe that the only part that is not > covered by Mikko is virtgpu_drm.h as it landed recently. Other than > that - with the explicit include of drm.h from the respective headers, > and the stdint to __foo type conversion things should actually work. I will resend that patch. There is still some issues on the headers, like the inclusion of drm.h. AFAIK, we should include "drm.h", in order to minimize the changes between linux/libdrm when importing, as the folders are not always the same (on Archlinux for example, drm.h is installed in /usr/include/libdrm/drm.h). This is not always the case in some of the headers: * drm_sarea.h * exynos_drm.h * i810_drm.h * i915_drm.h * mga_drm.h * msm_drm.h * nouveau_drm.h * omap_drm.h * r128_drm.h * savage_drm.h * tegra_drm.h * via_drm.h * vmwgfx_drm.h (with #ifndef __KERNEL__ guards around it) * there is still some headers that are including stddef.h for no visible reasons (msm_drm.h and qxl_drm.h). * drm_fourcc.h and drm_mode.h are still using linux/types.h I can send patches for them too, I just need to know if this is useful, and not just nitpicking for you. -- Gabriel Laskar _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
