[Mesa-dev] [PATCH v1 5/7] android: enable texture-float

[Emil Velikov]

On 16 June 2017 at 15:30, Rob Clark <robdclark at gmail.com> wrote:
> On Fri, Jun 16, 2017 at 9:49 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>> On 16 June 2017 at 14:25, Rob Clark <robdclark at gmail.com> wrote:
>>> On Fri, Jun 16, 2017 at 9:18 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>>>> On 15 June 2017 at 21:47, Robert Foss <robert.foss at collabora.com> wrote:
>>>>> From: Rob Herring <robh at kernel.org>
>>>>>
>>>>> This is required by freedreno at least for GLES3 support.
>>>>>
>>>>> See docs/patents.txt for information about turning this on for s/w
>>>>> renderers.
>>>>>
>>>> I'm inclined to agree with Tapani. We're not lawyers and with the
>>>> patent expiring in half a year, I think we can defer this toggle to
>>>> builders and their legal teams ;-)
>>>>
>>>
>>> Not really sure what msg from Tapani you are referring to.  But tbh,
>>> what we really want is a separate option for sw renderers.
>>>
>> This quote from the cover letter
>> "I'm not sure about patch to enable texture-float, should this be left
>> as a 'policy decision' for end distribution, it's not enabled by
>> default on desktop?"
>
> I think the only reason it isn't enabled by default is because
> (currently) the same option covers both hw drivers and sw drivers.
> Otoh, i965 ignores this option and always advertised texture-float.
>
> Anyways, if android has some way to conditionalize this based on some
> var set in board file, that might be an option.  Ideally I think we
> want to get to the point where we can build android with upstream
> mesa, without having to carry patches.
>
Fully support the zero patches topic. Yet since none of us is a lawyer
so it's understandable to be cautious ;-)
If people agree to enable it for non SW drivers, be that like i965 or
otherwise, let's have that consistent across Mesa.

Emil