Michel you need more coffee :) And yes, there are two header files. amdgpu_drm.h is the one synced from the kernel, amdgpu.h is the one local to libdrm. Regards, Christian. Am 15.05.2017 um 04:39 schrieb Xie, AlexBin: > > Hi Michel, > > > This patch was from Nicolai. > > > Though there are two amdgpu.h, one header is in kernel and the other > header is in libdrm. But their content are irrelevant. They are not > synchronized. > > > I think include/drm/README is not applicable to this case. > > > Thanks, > > Alex Bin Xie > > > ------------------------------------------------------------------------ > *From:* Michel Dänzer <michel at daenzer.net> > *Sent:* Sunday, May 14, 2017 10:09 PM > *To:* Xie, AlexBin; Haehnle, Nicolai > *Cc:* amd-gfx at lists.freedesktop.org > *Subject:* Re: [PATCH libdrm] amdgpu: add missing extern "C" headers > On 15/05/17 10:10 AM, Xie, AlexBin wrote: > > amdgpu/amdgpu.h | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > This header is synchronized with the kernel, see include/drm/README. > > > BTW, don't send patches for review as HTML. Ideally use git send-email. > > > -- > Earthling Michel Dänzer | http://www.amd.com > <http://www.amd.com> > Libre software enthusiast | Mesa and X developer > > > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx -------------- next part -------------- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20170515/4fa4a0ae/attachment.html>
