The offending patch didn't went into 9.2, so no harm done there.

Going to commit the fix as soon as I'm home again.

Christian.

Sent from my Android phone using TouchDown (www.nitrodesk.com)

-----Original Message-----
From: Ilia Mirkin [[email protected]]
Received: Samstag, 24 Aug. 2013, 16:02
To: Martin Peres [[email protected]]
CC: Koenig, Christian [[email protected]]; Maarten Lankhorst 
[[email protected]]; [email protected] 
[[email protected]]
Subject: Re: [PATCH] nouveau/video: avoid overwriting base codec init with 
template

On Sat, Aug 24, 2013 at 8:36 AM, Martin Peres <[email protected]> wrote:
> On 24/08/2013 11:38, Christian König wrote:
>>
>> Am 24.08.2013 06:50, schrieb Ilia Mirkin:
>>>
>>> Commit 53e20b8b introduced the use of a template to initialize some
>>> common fields. Move this copying of fields to before the common vp3
>>> fields are initialized.
>>>
>>> Reported-by: Martin Peres <[email protected]>
>>> Signed-off-by: Ilia Mirkin <[email protected]>
>>
>>
>> Sorry, that some fields are initialized earlier wasn't obviously to me.

It wasn't obvious to me either at first -- I saw the changes and they
looked fine :)

>>
>> Patch is: Reviewed-by: Christian König <[email protected]>

Thanks! I don't have commit access, so I'd appreciate it if you could
check it in. You can pull it from https://github.com/imirkin/mesa
bugfixes if you prefer to use git.

> This patch should also be pushed in 9.2 before its release.

I don't think that the "use the template" patches went into 9.2, so
this patch is also not appropriate for 9.2 (it wouldn't apply).

  -ilia

_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to