. [PATCH] drm/amd/amdgpu: Fix ring initialization for GFX9
Andres Rodriguez
andresx7 at gmail.com
Mon Jun 5 20:07:45 UTC 2017
On 2017-06-05 03:48 PM, Xie, AlexBin wrote:
> Hi Andres,
>
> I think the original patch was written by you. Would you comment? Is it a bug or intentional?
>
> Thank you.
>
> Alex Bin Xie
>
>
> Message: 1
> Date: Mon, 5 Jun 2017 10:36:59 -0400
> From: Tom St Denis <tom.stdenis at amd.com>
> To: amd-gfx at lists.freedesktop.org
> Subject: Re: amd-gfx Digest, Vol 13, Issue 29
> Message-ID: <58773ac0-93a7-e494-d447-88e93ded999c at amd.com>
> Content-Type: text/plain; charset=utf-8; format=flowed
>
> On 05/06/17 10:24 AM, Xie, AlexBin wrote:
>> Hi, Tom,
>>
>> You have found a bug.
>>
>> Your patch looks fine for me.
>>
>> Have you confirmed the deleted part is older version? Perhaps search email list or git history to confirm?
>
> It looks like the edits to the older GFX files (7/8) simply changed that
> block of code whereas the gfx9 version they pasted in the fixed block
> moving the old block down.
Yeah there was a problem with the rebase here. Tom/Alex's fix on the
mailing list is the intended solution.
Regards,
Andres
>
> Tom
>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>
More information about the amd-gfx
mailing list