<html> <head> <meta content="text/html; charset=utf-8" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> <div class="moz-cite-prefix">On 24.11.2015 10:36, Thierry Reding wrote: </div> <blockquote cite="mid:20151124093623.GB6149@ulmo" type="cite"> <pre wrap="">On Tue, Nov 24, 2015 at 09:59:09AM +0100, Daniel Vetter wrote: </pre> <blockquote type="cite"> <pre wrap="">On Tue, Nov 24, 2015 at 04:55:19PM +0800, Jammy Zhou wrote: </pre> <blockquote type="cite"> <pre wrap="">This series enable the DKMS build of amdgpu driver. </pre> </blockquote> <pre wrap=""> For the curious: What's DKMS? </pre> </blockquote> <pre wrap=""> I believe in this context it's "Dynamic Kernel Module Support": <a class="moz-txt-link-freetext" href="https://wiki.debian.org/KernelDKMS">https://wiki.debian.org/KernelDKMS</a></pre> </blockquote> Yeah, correct. <blockquote cite="mid:20151124093623.GB6149@ulmo" type="cite"> <pre wrap=""> I'm somewhat surprised that one would have to do something special to the kernel build system to "enable" such a build. But perhaps this is completely unrelated.</pre> </blockquote> I'm a bit torn apart on this. On the one hand I'm not sure if we have drivers upstream explicitly supporting this as well? On the other hand it's just the Makefiles which need to be written in a way which makes them relocatable. E.g. no absolute path like drivers/gpu/drm/amd/... in them and that's a good idea anyway. Maybe just changing the commit message to "don't use absolute paths in the makefiles" would be sufficient? Regards, Christian. <blockquote cite="mid:20151124093623.GB6149@ulmo" type="cite"> <pre wrap=""> Thierry </pre> <fieldset class="mimeAttachmentHeader"></fieldset> <pre wrap="">_______________________________________________ dri-devel mailing list <a class="moz-txt-link-abbreviated" href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a> <a class="moz-txt-link-freetext" href="http://lists.freedesktop.org/mailman/listinfo/dri-devel">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a> </pre> </blockquote> </body> </html>