[Mesa-dev] [PATCH 5/5] configure: Add the new "vc5" driver to the list, requiring a simulator.

[Emil Velikov]

On 19 September 2017 at 19:06, Eric Anholt <eric at anholt.net> wrote:
> My intent is to develop the vc5 driver in-tree for some time to build the
> CL generation and shader compiler code, and keep out-of-tree patches for
> talking to an actual kernel driver until the kernel driver can be
> stabilized on the hardware.
>
> v2: Define a HAVE_BROADCOM_DRIVERS, like HAVE_INTEL or HAVE_AMD.
> ---
>  configure.ac    | 15 ++++++++++++++-
>  src/Makefile.am |  2 +-
>  2 files changed, 15 insertions(+), 2 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 605c9b4e992f..5c5aa984fc78 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1267,7 +1267,7 @@ GALLIUM_DRIVERS_DEFAULT="r300,r600,svga,swrast"
>  AC_ARG_WITH([gallium-drivers],
>      [AS_HELP_STRING([--with-gallium-drivers@<:@=DIRS...@:>@],
>          [comma delimited Gallium drivers list, e.g.
> -        "i915,nouveau,r300,r600,radeonsi,freedreno,pl111,svga,swrast,swr,vc4,virgl,etnaviv,imx"
> +        "i915,nouveau,r300,r600,radeonsi,freedreno,pl111,svga,swrast,swr,vc4,vc5,virgl,etnaviv,imx"
>          @<:@default=r300,r600,svga,swrast@:>@])],
>      [with_gallium_drivers="$withval"],
>      [with_gallium_drivers="$GALLIUM_DRIVERS_DEFAULT"])
> @@ -2579,6 +2579,14 @@ if test -n "$with_gallium_drivers"; then
>                                 DEFINES="$DEFINES -DUSE_VC4_SIMULATOR"],
>                                [USE_VC4_SIMULATOR=no])
>              ;;
> +        xvc5)
> +            HAVE_GALLIUM_VC5=yes
> +
> +            PKG_CHECK_MODULES([VC5_SIMULATOR], [v3dv3],
> +                              [USE_VC5_SIMULATOR=yes;
> +                               DEFINES="$DEFINES -DUSE_VC5_SIMULATOR"],
> +                              [AC_MSG_ERROR([vc5 requires the simulator])])
> +            ;;
Sounds reasonable.

Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

-Emil