* [PATCH] libdrm/amdgpu: Fixed drm.h include.
@ 2015-07-16 20:19 akulichalexander
2015-07-17 9:13 ` Michel Dänzer
0 siblings, 1 reply; 9+ messages in thread
From: akulichalexander @ 2015-07-16 20:19 UTC (permalink / raw)
To: dri-devel
From: Alexandr Akulich <akulichalexander@gmail.com>
The include type changed from system to own.
Signed-off-by: Alexandr Akulich <akulichalexander@gmail.com>
---
include/drm/amdgpu_drm.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/amdgpu_drm.h b/include/drm/amdgpu_drm.h
index b6fce90..25e9b0a 100644
--- a/include/drm/amdgpu_drm.h
+++ b/include/drm/amdgpu_drm.h
@@ -32,7 +32,7 @@
#ifndef __AMDGPU_DRM_H__
#define __AMDGPU_DRM_H__
-#include <drm/drm.h>
+#include "drm.h"
#define DRM_AMDGPU_GEM_CREATE 0x00
#define DRM_AMDGPU_GEM_MMAP 0x01
--
2.4.4
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] libdrm/amdgpu: Fixed drm.h include.
2015-07-16 20:19 [PATCH] libdrm/amdgpu: Fixed drm.h include akulichalexander
@ 2015-07-17 9:13 ` Michel Dänzer
2015-07-17 13:33 ` Alexandr Akulich
0 siblings, 1 reply; 9+ messages in thread
From: Michel Dänzer @ 2015-07-17 9:13 UTC (permalink / raw)
To: akulichalexander; +Cc: dri-devel
On 17.07.2015 05:19, akulichalexander@gmail.com wrote:
> From: Alexandr Akulich <akulichalexander@gmail.com>
>
> The include type changed from system to own.
>
> Signed-off-by: Alexandr Akulich <akulichalexander@gmail.com>
> ---
> include/drm/amdgpu_drm.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/drm/amdgpu_drm.h b/include/drm/amdgpu_drm.h
> index b6fce90..25e9b0a 100644
> --- a/include/drm/amdgpu_drm.h
> +++ b/include/drm/amdgpu_drm.h
> @@ -32,7 +32,7 @@
> #ifndef __AMDGPU_DRM_H__
> #define __AMDGPU_DRM_H__
>
> -#include <drm/drm.h>
> +#include "drm.h"
>
> #define DRM_AMDGPU_GEM_CREATE 0x00
> #define DRM_AMDGPU_GEM_MMAP 0x01
>
It should be
#include <drm.h>
<drm/drm.h> is wrong because that's the path of the kernel header, which
not every distro ships.
"drm.h" is wrong because amdgpu_drm.h is used by other projects.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] libdrm/amdgpu: Fixed drm.h include.
2015-07-17 9:13 ` Michel Dänzer
@ 2015-07-17 13:33 ` Alexandr Akulich
2015-07-22 7:19 ` Michel Dänzer
0 siblings, 1 reply; 9+ messages in thread
From: Alexandr Akulich @ 2015-07-17 13:33 UTC (permalink / raw)
To: Michel Dänzer; +Cc: dri-devel
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)
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.
This fixes libdrm build with amdgpu support on bare system for me.
On Fri, Jul 17, 2015 at 2:13 PM, Michel Dänzer <michel@daenzer.net> wrote:
> On 17.07.2015 05:19, akulichalexander@gmail.com wrote:
>> From: Alexandr Akulich <akulichalexander@gmail.com>
>>
>> The include type changed from system to own.
>>
>> Signed-off-by: Alexandr Akulich <akulichalexander@gmail.com>
>> ---
>> include/drm/amdgpu_drm.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/drm/amdgpu_drm.h b/include/drm/amdgpu_drm.h
>> index b6fce90..25e9b0a 100644
>> --- a/include/drm/amdgpu_drm.h
>> +++ b/include/drm/amdgpu_drm.h
>> @@ -32,7 +32,7 @@
>> #ifndef __AMDGPU_DRM_H__
>> #define __AMDGPU_DRM_H__
>>
>> -#include <drm/drm.h>
>> +#include "drm.h"
>>
>> #define DRM_AMDGPU_GEM_CREATE 0x00
>> #define DRM_AMDGPU_GEM_MMAP 0x01
>>
>
> It should be
>
> #include <drm.h>
>
> <drm/drm.h> is wrong because that's the path of the kernel header, which
> not every distro ships.
>
> "drm.h" is wrong because amdgpu_drm.h is used by other projects.
>
>
> --
> Earthling Michel Dänzer | http://www.amd.com
> Libre software enthusiast | Mesa and X developer
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] libdrm/amdgpu: Fixed drm.h include.
2015-07-17 13:33 ` Alexandr Akulich
@ 2015-07-22 7:19 ` Michel Dänzer
2015-07-22 7:33 ` Alexandr Akulich
0 siblings, 1 reply; 9+ messages in thread
From: Michel Dänzer @ 2015-07-22 7:19 UTC (permalink / raw)
To: Alexandr Akulich; +Cc: dri-devel
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 "".
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] libdrm/amdgpu: Fixed drm.h include.
2015-07-22 7:19 ` Michel Dänzer
@ 2015-07-22 7:33 ` Alexandr Akulich
2015-07-22 7:35 ` Alexandr Akulich
2015-07-22 8:20 ` Michel Dänzer
0 siblings, 2 replies; 9+ messages in thread
From: Alexandr Akulich @ 2015-07-22 7:33 UTC (permalink / raw)
To: Michel Dänzer; +Cc: dri-devel
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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] libdrm/amdgpu: Fixed drm.h include.
2015-07-22 7:33 ` Alexandr Akulich
@ 2015-07-22 7:35 ` Alexandr Akulich
2015-07-22 8:20 ` Michel Dänzer
1 sibling, 0 replies; 9+ messages in thread
From: Alexandr Akulich @ 2015-07-22 7:35 UTC (permalink / raw)
To: Michel Dänzer; +Cc: dri-devel
>> 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?
I mean "apply the patch".
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] libdrm/amdgpu: Fixed drm.h include.
2015-07-22 7:33 ` Alexandr Akulich
2015-07-22 7:35 ` Alexandr Akulich
@ 2015-07-22 8:20 ` Michel Dänzer
2015-07-29 2:35 ` Alexandr Akulich
1 sibling, 1 reply; 9+ messages in thread
From: Michel Dänzer @ 2015-07-22 8:20 UTC (permalink / raw)
To: Alexandr Akulich; +Cc: dri-devel
On 22.07.2015 16:33, Alexandr Akulich wrote:
> 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?
I assume your patch is against the amdgpu branch of
http://cgit.freedesktop.org/~agd5f/drm/, right? Only Alex Deucher has
write access to that.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] libdrm/amdgpu: Fixed drm.h include.
2015-07-22 8:20 ` Michel Dänzer
@ 2015-07-29 2:35 ` Alexandr Akulich
2015-07-29 20:59 ` Alex Deucher
0 siblings, 1 reply; 9+ messages in thread
From: Alexandr Akulich @ 2015-07-29 2:35 UTC (permalink / raw)
To: Michel Dänzer; +Cc: dri-devel
On Wed, Jul 22, 2015 at 1:20 PM, Michel Dänzer <michel@daenzer.net> wrote:
> On 22.07.2015 16:33, Alexandr Akulich wrote:
>> 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?
>
> I assume your patch is against the amdgpu branch of
> http://cgit.freedesktop.org/~agd5f/drm/, right? Only Alex Deucher has
> write access to that.
Well, what I can do?
Alex, can you apply the patch, please?
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] libdrm/amdgpu: Fixed drm.h include.
2015-07-29 2:35 ` Alexandr Akulich
@ 2015-07-29 20:59 ` Alex Deucher
0 siblings, 0 replies; 9+ messages in thread
From: Alex Deucher @ 2015-07-29 20:59 UTC (permalink / raw)
To: Alexandr Akulich; +Cc: Michel Dänzer, Maling list - DRI developers
On Tue, Jul 28, 2015 at 10:35 PM, Alexandr Akulich
<akulichalexander@gmail.com> wrote:
> On Wed, Jul 22, 2015 at 1:20 PM, Michel Dänzer <michel@daenzer.net> wrote:
>> On 22.07.2015 16:33, Alexandr Akulich wrote:
>>> 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?
>>
>> I assume your patch is against the amdgpu branch of
>> http://cgit.freedesktop.org/~agd5f/drm/, right? Only Alex Deucher has
>> write access to that.
>
> Well, what I can do?
> Alex, can you apply the patch, please?
Applied. thanks!
Alex
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-07-29 20:59 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-16 20:19 [PATCH] libdrm/amdgpu: Fixed drm.h include akulichalexander
2015-07-17 9:13 ` Michel Dänzer
2015-07-17 13:33 ` Alexandr Akulich
2015-07-22 7:19 ` Michel Dänzer
2015-07-22 7:33 ` Alexandr Akulich
2015-07-22 7:35 ` Alexandr Akulich
2015-07-22 8:20 ` Michel Dänzer
2015-07-29 2:35 ` Alexandr Akulich
2015-07-29 20:59 ` Alex Deucher
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).