[PATCH] libdrm/amdgpu: Fixed drm.h include.

[Michel Dänzer]

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 at amd.com>


> 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