LGTM Reviewed-by: Plamena Manolova <[email protected]>

On Thu, Feb 2, 2017 at 2:32 AM, Emil Velikov <[email protected]>
wrote:

> From: Emil Velikov <[email protected]>
>
> They are versions of the respective libdrm package. They are _not_ the
> version of libdrm[.so] required for driver X.
>
> Doing the latter will lead to combinatoric explosion and in all fairness
> things will likely be broken most of the time.
>
> To make things even more confusing the kernel UAPI is provided by libdrm
> itself.
>
> Cc: Vinson Lee <[email protected]>
> Cc: Kenneth Graunke <[email protected]>
> Signed-off-by: Emil Velikov <[email protected]>
> ---
> Ken, you/Chad have things spot on. Yet semes like other people struggle
> deeply with these.
> ---
>  configure.ac | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index 92339b4a3f..e8e154b412 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -67,6 +67,13 @@ OPENCL_VERSION=1
>  AC_SUBST([OPENCL_VERSION])
>
>  dnl Versions for external dependencies
> +dnl
> +dnl The LIBDRM instances reference either the generic libdrm or the
> specific
> +dnl specific libdrm-$hw.
> +dnl They are _not_ "the min. libdrm required for dri/gallium/other driver
> $foo."
> +dnl
> +dnl Note that the "basic" ABI for $hw -> $hw_drm.h is provided by libdrm
> itself.
> +dnl
>  LIBDRM_REQUIRED=2.4.75
>  LIBDRM_RADEON_REQUIRED=2.4.56
>  LIBDRM_AMDGPU_REQUIRED=2.4.63
> --
> 2.11.0
>
> _______________________________________________
> mesa-dev mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to