Tested-by: Evan Quan <evan.quan at amd.com> > -----Original Message----- > From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf Of > Huang Rui > Sent: Monday, December 19, 2016 3:16 PM > To: amd-gfx at lists.freedesktop.org > Cc: Huang, Ray <Ray.Huang at amd.com> > Subject: [PATCH] drm/amd/powerplay: fix request smc_sk firmware case > > This patch fixes firmware request error on polaris protection mode. > Because we need load smc_sk instead of smc under security protection mode. > > Signed-off-by: Huang Rui <ray.huang at amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c > index 9ada56c..3235630 100755 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c > @@ -713,6 +713,7 @@ static int amdgpu_cgs_rel_firmware(struct cgs_device > *cgs_device, enum cgs_ucode > CGS_FUNC_ADEV; > if ((CGS_UCODE_ID_SMU == type) || (CGS_UCODE_ID_SMU_SK == > type)) { > release_firmware(adev->pm.fw); > + adev->pm.fw = NULL; > return 0; > } > /* cannot release other firmware because they are not created by > cgs */ @@ -808,6 +809,9 @@ static int amdgpu_cgs_get_firmware_info(struct > cgs_device *cgs_device, > const uint8_t *src; > const struct smc_firmware_header_v1_0 *hdr; > > + if (CGS_UCODE_ID_SMU_SK == type) > + amdgpu_cgs_rel_firmware(cgs_device, > CGS_UCODE_ID_SMU); > + > if (!adev->pm.fw) { > switch (adev->asic_type) { > case CHIP_TOPAZ: > -- > 2.7.4 > > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx
