On 17.07.2015 22:33, Alexandr Akulich wrote: > As I see, this is not a kernel header, but a local (belongs to libdrm) one. > (Otherwise, I would like you to point me on such file at, say, > https://github.com/torvalds/linux/tree/master/include/drm) https://github.com/torvalds/linux/blob/master/include/uapi/drm/amdgpu_drm.h > At files drm_sarea.h, mga_drm.h, qxl_drm.h, radeon_drm.h and via_drm.h we have > > #include "drm.h" > > At the same time, at i915_drm.h and tegra_drm.h. we have > > #include <drm.h> > > I'm pretty sure that the first case is correct, because drm.h is > always would be local file (in the same directory), related to > amdgpu_drm.h, and that is why local include "drm.h" would be always > valid. Makes sense to me, the patch is Reviewed-by: Michel Dänzer <michel.daenzer@xxxxxxx> > This fixes libdrm build with amdgpu support on bare system for me. FWIW though, that's because you're dropping the drm/ prefix, not because you're changing from <> to "". -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
