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

[Emil Velikov]

On 02/05/14 23:14, Marek Olšák wrote:
> Radeonsi requires Glamor and Glamor requires GBM and EGL DRM. Without
> the two, there is no way to get acceleration.
> 
Completely forgot that radeonsi uses glamor. Sorry for the noise.

-Emil

> 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