On 2018-06-07 08:38 AM, Leo Liu wrote:
>
>
> On 06/07/2018 03:10 AM, Christian König wrote:
>> Am 06.06.2018 um 20:59 schrieb James Zhu:
>>> Vega20 UVD Firmware has a new version naming convention:
>>> Â Â [31, 30] for encode interface major
>>> Â Â [29, 24] for encode interface minor
>>> Â Â [15, 8] for firmware revision
>>> Â Â [7, 0] for hardware family id
>>> Inside kernel log UVD firmware Version: 1.1.2 (denote
>>> major.minor.revision)
>>>
>>> Signed-off-by: James Zhu <James.Zhu at amd.com>
>>> ---
>>> Â drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 19 +++++++++++++++----
>>> Â 1 file changed, 15 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
>>> index bcf68f8..575aff1b 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
>>> @@ -208,10 +208,21 @@ int amdgpu_uvd_sw_init(struct amdgpu_device
>>> *adev)
>>> Â Â Â Â Â Â hdr = (const struct common_firmware_header
>>> *)adev->uvd.fw->data;
>>> Â Â Â Â Â family_id = le32_to_cpu(hdr->ucode_version) & 0xff;
>>> -Â Â Â version_major = (le32_to_cpu(hdr->ucode_version) >> 24) & 0xff;
>>> -Â Â Â version_minor = (le32_to_cpu(hdr->ucode_version) >> 8) & 0xff;
>>> -Â Â Â DRM_INFO("Found UVD firmware Version: %hu.%hu Family ID: %hu\n",
>>> -Â Â Â Â Â Â Â version_major, version_minor, family_id);
>>> +
>>> +Â Â Â if (adev->asic_type != CHIP_VEGA20) {
>>
>> I think it would be better to use "< CHIP_VEGA20" here.
> Agreed. hopefully the convention will be kept consistently.
>
> >>Inside kernel log UVD firmware Version: 1.1.2 (denote
> major.minor.revision)
>
> Now the major and minor become 1.1. if you keep looking the code after
> the changed part, the 1.1 will cause problem for number of handles,
> since that decides how big of bo size for FW runtime.
So Can I say after CHIP_VEGA20, all asics are using
AMDGPU_MAX_UVD_HANDLES? James
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
@@ -209,7 +209,7 @@ int amdgpu_uvd_sw_init(struct amdgpu_device *adev)
...
-Â Â Â Â Â Â if ((version_major > 0x01) ||
+Â Â Â Â Â Â if (adev->asic_type >= CHIP_VEGA20 || (version_major > 0x01) ||
           ((version_major == 0x01) && (version_minor >= 0x50)))
               adev->uvd.max_handles = AMDGPU_MAX_UVD_HANDLES;
>
> Regards,
> Leo
>
>
>>
>> Apart from that looks good to me, but Leo should have the last word.
>>
>> Thanks,
>> Christian.
>>
>>> +Â Â Â Â Â Â Â version_major = (le32_to_cpu(hdr->ucode_version) >> 24) &
>>> 0xff;
>>> +Â Â Â Â Â Â Â version_minor = (le32_to_cpu(hdr->ucode_version) >> 8) & 0xff;
>>> +Â Â Â Â Â Â Â DRM_INFO("Found UVD firmware Version: %hu.%hu Family ID:
>>> %hu\n",
>>> +Â Â Â Â Â Â Â Â Â Â Â version_major, version_minor, family_id);
>>> +Â Â Â } else {
>>> +Â Â Â Â Â Â Â unsigned int fw_rev;
>>> +
>>> +Â Â Â Â Â Â Â fw_rev = (le32_to_cpu(hdr->ucode_version) >> 8) & 0xff;
>>> +Â Â Â Â Â Â Â version_minor = (le32_to_cpu(hdr->ucode_version) >> 24) &
>>> 0x3f;
>>> +Â Â Â Â Â Â Â version_major = (le32_to_cpu(hdr->ucode_version) >> 30) & 0x3;
>>> +Â Â Â Â Â Â Â DRM_INFO("Found UVD firmware Version: %hu.%hu.%hu Family
>>> ID: %hu\n",
>>> +Â Â Â Â Â Â Â Â Â Â Â version_major, version_minor, fw_rev, family_id);
>>> +Â Â Â }
>>> Â Â Â Â Â Â /*
>>> Â Â Â Â Â Â * Limit the number of UVD handles depending on microcode major
>>
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
