Re: [PATCH 2/2] g3dvl: disable iDCT for now

[Bryan Cain <bryancain3-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

On 07/27/2011 02:42 AM, Ben Skeggs wrote:
> On Wed, 2011-07-20 at 13:48 -0500, Bryan Cain wrote:
>> ---
>>  src/vl_hwmc.c |    4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
> The obvious question I have for this is, why?
>
> Ben

At least at the time I wrote these patches (in April), the iDCT stage
didn't work correctly with nv50.  If it's fixed now, this patch can be
ignored.

Bryan


>> diff --git a/src/vl_hwmc.c b/src/vl_hwmc.c
>> index 6e9e0c7..8e24786 100644
>> --- a/src/vl_hwmc.c
>> +++ b/src/vl_hwmc.c
>> @@ -55,7 +55,7 @@ static XF86MCSurfaceInfoRec yv12_mpeg2_surface =
>>  	2048,
>>  	2048,
>>  	2048,
>> -	XVMC_IDCT | XVMC_MOCOMP | XVMC_MPEG_2,
>> +	/*XVMC_IDCT |*/ XVMC_MOCOMP | XVMC_MPEG_2,
>>  	XVMC_SUBPICTURE_INDEPENDENT_SCALING | XVMC_BACKEND_SUBPICTURE,
>>  	&subpicture_list
>>  };
>> @@ -69,7 +69,7 @@ static XF86MCSurfaceInfoRec uyvy_mpeg2_surface =
>>  	2048,
>>  	2048,
>>  	2048,
>> -	XVMC_IDCT | XVMC_MOCOMP | XVMC_MPEG_2,
>> +	/*XVMC_IDCT |*/ XVMC_MOCOMP | XVMC_MPEG_2,
>>  	XVMC_SUBPICTURE_INDEPENDENT_SCALING | XVMC_BACKEND_SUBPICTURE,
>>  	&subpicture_list
>>  };
>
>