[Mesa-dev] [Mesa-stable] [PATCH] radeon uvd add uvd fw version for amdgpu

[Jiang, Sonny]

Hi Emil,


I have added comments for each patch, and applied for branch 12.0.

Please find attached patches.


Thanks,

Sonny

________________________________
From: mesa-dev <mesa-dev-bounces at lists.freedesktop.org> on behalf of Christian König <deathsimple at vodafone.de>
Sent: Friday, July 1, 2016 8:07:51 AM
To: Emil Velikov
Cc: Jiang, Sonny; 12.0; mesa-dev at lists.freedesktop.org
Subject: Re: [Mesa-dev] [Mesa-stable] [PATCH] radeon uvd add uvd fw version for amdgpu

Am 01.07.2016 um 13:14 schrieb Emil Velikov:
> Hi all,
>
> On 29 June 2016 at 20:20, Christian König <deathsimple at vodafone.de> wrote:
>> Am 29.06.2016 um 18:35 schrieb Alex Deucher:
>>> On Wed, Jun 29, 2016 at 11:38 AM, Leo Liu <leo.liu at amd.com> wrote:
>>>> From: sonjiang <sonny.jiang at amd.com>
>>>>
>>>> Signed-off-by: sonjiang <sonny.jiang at amd.com>
>>>> Cc: "12.0" <mesa-stable at lists.freedesktop.org>
>>> For the series:
>>> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
>>
>> Reviewed-by: Christian König <christian.koenig at amd.com> as well.
>>
> Here we have three patches, suggesting a bug with absolutely no
> information what the issue is and/or why this approach is correct.
>
> I'm sorry to say this, but as is, this series is not landing in
> stable. Sonjiang, being the author of these please reply with a brief
> justification why we want those. Before doing so I would strongly
> recommend reading this [1] blog post.

Well to put a carrot on the front of your stick: I asked what the
firmware version patch is all about internally as well when I've seen
those patches. So it would have even made our internal review much
easier if Sonny added a commit message in the first place.

My fault to not requesting that his answer is put as a commit message on
the patches.

On the other hand this is for Polaris, we had time pressure to get it
out of the door and today is a public holiday in Canada. So you probably
won't get updated message before Monday.

Is that soon enough? Otherwise UVD will be broken on Polaris in the
stable branch.

Regards,
Christian.

>
> Thanks
> Emil
>
> [1] http://who-t.blogspot.co.uk/2009/12/on-commit-messages.html

_______________________________________________
mesa-dev mailing list
mesa-dev at lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160704/d5e71a8b/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-radeon-uvd-add-uvd-fw-version-for-amdgpu.patch
Type: text/x-patch
Size: 2984 bytes
Desc: 0001-radeon-uvd-add-uvd-fw-version-for-amdgpu.patch
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160704/d5e71a8b/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-radeon-uvd-seperate-uvd-context-buffer-from-DPB.patch
Type: text/x-patch
Size: 7404 bytes
Desc: 0002-radeon-uvd-seperate-uvd-context-buffer-from-DPB.patch
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160704/d5e71a8b/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-radeon-uvd-fix-a-h265-context-size-bug.patch
Type: text/x-patch
Size: 1080 bytes
Desc: 0003-radeon-uvd-fix-a-h265-context-size-bug.patch
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160704/d5e71a8b/attachment-0005.bin>