[PATCH] radeon: enable glamor automatically by default

[Marek Olšák]

I think it's better to fail and force a user to use --disable-glamor
(or install glamor) than silently succeed without any acceleration for
SI.

Marek

On Mon, Sep 16, 2013 at 4:00 PM, Michel Dänzer <michel at daenzer.net> wrote:
> On Mon, 2013-09-16 at 09:42 -0400, Alex Deucher wrote:
>> It's required for newer asics, so enable it by
>> default.
>>
>> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
>> ---
>>  configure.ac | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index d1cf115..7cd30d2 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -101,9 +101,9 @@ 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]]),
>> +                          [Enable glamor, a new GL-based acceleration [default=auto]]),
>>             [GLAMOR="$enableval"],
>> -           [GLAMOR=no])
>> +           [GLAMOR=auto])
>>  AC_MSG_RESULT([$GLAMOR])
>>  AM_CONDITIONAL(GLAMOR, test x$GLAMOR != xno)
>>  if test "x$GLAMOR" != "xno"; then
>
> I think this will cause configure to fail without --disable-glamor when
> glamor isn't available? Probably need more changes similar to what's
> done for udev support.
>
>
> --
> Earthling Michel Dänzer           |                   http://www.amd.com
> Libre software enthusiast         |          Debian, X and DRI developer
>
> _______________________________________________
> xorg-driver-ati mailing list
> xorg-driver-ati at lists.x.org
> http://lists.x.org/mailman/listinfo/xorg-driver-ati