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 [imir...@alum.mit.edu]
Received: Samstag, 24 Aug. 2013, 16:02
To: Martin Peres [martin.pe...@labri.fr]
CC: Koenig, Christian [christian.koe...@amd.com]; Maarten Lankhorst 
[maarten.lankho...@ubuntu.com]; mesa-dev@lists.freedesktop.org 
[mesa-dev@lists.freedesktop.org]
Subject: Re: [PATCH] nouveau/video: avoid overwriting base codec init with 
template

On Sat, Aug 24, 2013 at 8:36 AM, Martin Peres <martin.pe...@labri.fr> 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 <martin.pe...@labri.fr>
>>> Signed-off-by: Ilia Mirkin <imir...@alum.mit.edu>
>>
>>
>> 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 <christian.koe...@amd.com>

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
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to