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
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 "".