Re: [PATCH 1/2] drm/radeon: disable semaphores for UVD V1

["Christian König" <deathsimple@vodafone.de>]

On 01.05.2015 18:24, Alex Deucher wrote:
> On Fri, May 1, 2015 at 12:13 PM, Christian König
> <deathsimple@vodafone.de> wrote:
>> On 01.05.2015 17:13, Alex Deucher wrote:
>>> On Fri, May 1, 2015 at 6:34 AM, Christian König <deathsimple@vodafone.de>
>>> wrote:
>>>> From: Christian König <christian.koenig@amd.com>
>>>>
>>>> Hardware doesn't seem to work correctly, just block userspace in this
>>>> case.
>>>>
>>>> Bugs: https://bugs.freedesktop.org/show_bug.cgi?id=85320
>>>>
>>>> Signed-off-by: Christian König <christian.koenig@amd.com>
>>>> CC: stable@vger.kernel.org
>>> Applied both to my stable tree.  I had to fix the missing defines on this
>>> patch.
>>
>> Ups, thanks for doing this. I've once more send out the wrong version of the
>> patch. But your changes look like they should work as well.
> I can swap it out if you have a preferred version.

No, keep it as it is. My version was just emitting the semaphore using 
the FW instead of programming the regs directly.

This is actually the documented way how to do it, but IIRC we had 
problems with this and so we are using the direct way ever since shortly 
after releasing it.

No need to switch it back without a good reason, that could just cause 
trouble.

Christian.

>
> Alex
>
>> Christian.
>>
>>> Alex
>>

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel