NACK/Cmnt[G]: [PATCH 0/2][SRU][OEM-5.10/G] AMDGPU: Support to load new MC firmware for Polaris12 32bit ASIC

[Stefan Bader]

On 21.06.21 07:14, Koba Ko wrote:
> BugLink: https://bugs.launchpad.net/bugs/1932993
> 
> [Impact]
> AMD introduce a new firmware and has been SRUed in this public bug.
> https://bugs.launchpad.net/ubuntu/focal/+source/linux-firmware/+bug/1929147
> 
> For the kernel(oem-5.10/groovy), also need other patches to load this firmware.
> 
> [Fix]
> Support to load polaris12_32_mc.bin
> 
> [Test]
> 1. Check by modinfo and cnofirm polaris12_32_mc.bin is loaded by amdgpu.ko
> 
> #modinfo amdgpu | grep polaris12_32_mc.bin
> firmware: amdgpu/polaris12_32_mc.bin
> 
> 2. Equipped the rx540(PD5001) that would trigger issue and system would bootup successfully into desktop.
> *some rx540(PD5000) can't trigger the issue.
> 
> [Regression Potential]
> Medium, introduce a brand new firmware may cause regression.
> 
> Evan Quan (2):
>    drm/amd/pm: correct the checks for polaris kickers
>    drm/amdgpu: add new MC firmware for Polaris12 32bit ASIC
> 
>   drivers/gpu/drm/amd/amdgpu/amdgpu.h     | 39 +++++++++++++++++++++++++
>   drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 34 ++++-----------------
>   drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c   | 35 +++++++++-------------
>   3 files changed, 57 insertions(+), 51 deletions(-)
> 

There was never any reply as to what sense this makes for Groovy and given that 
it EOLs next week, this will no longer be doable either. So rejecting it officially.

-Stefan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20210716/f7f2dde4/attachment.sig>