[Mesa-dev] [PATCH 11/16] st/nine: Ignore D3DUSAGE_QUERY_POSTPIXELSHADER_BLENDING when D3DUSAGE_RENDERTARGET is specified

Ilia Mirkin imirkin at alum.mit.edu
Fri Apr 24 14:34:06 PDT 2015


On Fri, Apr 24, 2015 at 5:32 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> The diff does not agree with the commit description. Also please limit
> commit descriptions to 72 chars.

er, of course I meant commit *subjects*. Although the each line of the
commit description should also be wrapped at 72. (Because they're
indented by a tab when shown with 'git show' and so on.)

>
> On Fri, Apr 24, 2015 at 4:09 PM, Axel Davy <axel.davy at ens.fr> wrote:
>> From: Xavier Bouchoux <xavierb at gmail.com>
>>
>> This behaviour matches windows drivers.
>>
>> Reviewed-by: Axel Davy <axel.davy at ens.fr>
>> Signed-off-by: Xavier Bouchoux <xavierb at gmail.com>
>> ---
>>  src/gallium/state_trackers/nine/adapter9.c | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/gallium/state_trackers/nine/adapter9.c b/src/gallium/state_trackers/nine/adapter9.c
>> index 1d75155..9d6d659 100644
>> --- a/src/gallium/state_trackers/nine/adapter9.c
>> +++ b/src/gallium/state_trackers/nine/adapter9.c
>> @@ -311,7 +311,8 @@ NineAdapter9_CheckDeviceFormat( struct NineAdapter9 *This,
>>      if (CheckFormat == D3DFMT_ATOC && RType == D3DRTYPE_SURFACE)
>>          return D3D_OK;
>>
>> -    if (Usage & D3DUSAGE_QUERY_POSTPIXELSHADER_BLENDING)
>> +    if ((Usage & D3DUSAGE_QUERY_POSTPIXELSHADER_BLENDING) &&
>> +        (Usage & D3DUSAGE_RENDERTARGET))
>>          bind |= PIPE_BIND_BLENDABLE;
>>
>>      if (Usage & D3DUSAGE_DMAP) {
>> --
>> 2.1.0
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list