[Mesa-dev] [PATCH 1/9] configure.ac: radeonsi requires EGL_DRM and GBM

[Marek Olšák]

Radeonsi requires Glamor and Glamor requires GBM and EGL DRM. Without
the two, there is no way to get acceleration.

Marek

On Fri, May 2, 2014 at 11:31 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> On 26/04/14 14:27, Marek Olšák wrote:
>> From: Marek Olšák <marek.olsak at amd.com>
>>
>> ---
>>  configure.ac | 14 ++++++++++++++
>>  1 file changed, 14 insertions(+)
>>
>> diff --git a/configure.ac b/configure.ac
>> index c71fa26..b2ea8ca 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -1738,6 +1738,19 @@ gallium_require_drm_loader() {
>>      fi
>>  }
>>
>> +require_egl_drm() {
>> +    case "$with_egl_platforms" in
>> +        *drm*)
>> +            ;;
>> +         *)
>> +            AC_MSG_ERROR([--with-egl-platforms=drm is required to build the $1 driver.])
>> +            ;;
>> +    esac
>> +    if test "x$enable_gbm" != xyes; then
>> +            AC_MSG_ERROR([--enable-gbm is required to build the $1 driver.])
>> +    fi
>> +}
>> +
> Hi Marek,
>
> I'm not sure if the patch does exactly what you expect it to.
>
> AFAICS, based on the above you require the drm egl platform with radeonsi,
> although it will break the build for people that do not build gbm/egl.
>
> Do I take it that radeonsi is available/usable _only_ when gbm + drm egl is
> build ? Otherwise just wrap the above in a
>
> if test "x$enable_egl" = xyes; then
> ....
> fi
>
> Cheers,
> -Emil