On Wed, Jul 22, 2015 at 1:19 PM, Michel Dänzer michel@daenzer.net wrote:
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@amd.com
Thanks! I have no write access to this fd.o repository. Can you apply it please?
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 "".
I know :-). Compiler would "fallback" from <> to "", if it can't find a header in passed include directories.
-- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer