<div dir="ltr">LGTM Reviewed-by: Plamena Manolova <<a href="mailto:plamena.manolova@intel.com">plamena.manolova@intel.com</a>></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 2, 2017 at 2:32 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Emil Velikov <<a href="mailto:emil.velikov@collabora.com">emil.velikov@collabora.com</a>><br>
<br>
They are versions of the respective libdrm package. They are _not_ the<br>
version of libdrm[.so] required for driver X.<br>
<br>
Doing the latter will lead to combinatoric explosion and in all fairness<br>
things will likely be broken most of the time.<br>
<br>
To make things even more confusing the kernel UAPI is provided by libdrm<br>
itself.<br>
<br>
Cc: Vinson Lee <<a href="mailto:vlee@freedesktop.org">vlee@freedesktop.org</a>><br>
Cc: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
Signed-off-by: Emil Velikov <<a href="mailto:emil.velikov@collabora.com">emil.velikov@collabora.com</a>><br>
---<br>
Ken, you/Chad have things spot on. Yet semes like other people struggle<br>
deeply with these.<br>
---<br>
 <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> | 7 +++++++<br>
 1 file changed, 7 insertions(+)<br>
<br>
diff --git a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
index 92339b4a3f..e8e154b412 100644<br>
--- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
+++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
@@ -67,6 +67,13 @@ OPENCL_VERSION=1<br>
 AC_SUBST([OPENCL_VERSION])<br>
<br>
 dnl Versions for external dependencies<br>
+dnl<br>
+dnl The LIBDRM instances reference either the generic libdrm or the specific<br>
+dnl specific libdrm-$hw.<br>
+dnl They are _not_ "the min. libdrm required for dri/gallium/other driver $foo."<br>
+dnl<br>
+dnl Note that the "basic" ABI for $hw -> $hw_drm.h is provided by libdrm itself.<br>
+dnl<br>
 LIBDRM_REQUIRED=2.4.75<br>
 LIBDRM_RADEON_REQUIRED=2.4.56<br>
 LIBDRM_AMDGPU_REQUIRED=2.4.63<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.11.0<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div>