[PATCH] radeon: enable glamor automatically by default (v4)

Michel Dänzer michel at daenzer.net
Mon Sep 16 08:54:41 PDT 2013


On Mon, 2013-09-16 at 11:31 -0400, Alex Deucher wrote:
> It's required for newer asics, so enable it by
> default.
> 
> v2: update the autoconf help.  Require --disable-glamor
> to build without glamor support.
> v3: default to yes rather than auto
> v4: fix help text
> 
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
>  configure.ac | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index d1cf115..f25d5e8 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -100,10 +100,10 @@ CPPFLAGS="$CPPFLAGS $XORG_CFLAGS"
>  
>  AC_MSG_CHECKING([whether to include GLAMOR support])
>  AC_ARG_ENABLE(glamor,
> -	      AS_HELP_STRING([--enable-glamor],
> -			     [Enable glamor, a new GL-based acceleration [default=no]]),
> +	      AS_HELP_STRING([--disable-glamor],
> +			     [Disable glamor, a new GL-based acceleration [default=enabled]]),
>  	      [GLAMOR="$enableval"],
> -	      [GLAMOR=no])
> +	      [GLAMOR=yes])
>  AC_MSG_RESULT([$GLAMOR])
>  AM_CONDITIONAL(GLAMOR, test x$GLAMOR != xno)
>  if test "x$GLAMOR" != "xno"; then

The 'automatically' in the shortlog might be superfluous now, but either
way

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer           |                   http://www.amd.com
Libre software enthusiast         |          Debian, X and DRI developer



More information about the xorg-driver-ati mailing list