On Tue, Jan 19, 2021 at 02:25:36PM +0800, Liang, Prike wrote:
> [AMD Official Use Only - Internal Distribution Only]
> 
> Thanks help clean up. Generally that seems fine but could we keep the green 
> sardine chip name to retrieve the GPU info FW when the IP discovery fail back 
> to legacy mode?

Do you want to only clean MODULE_FIRMWARE(gpu_info.bin)? That's fine for
me.

Thanks,
Ray

> 
> Anyway this patch is Reviewed-by: Prike Liang <prike.li...@amd.com>
> 
> Thanks,
> Prike
> > -----Original Message-----
> > From: Huang, Ray <ray.hu...@amd.com>
> > Sent: Tuesday, January 19, 2021 1:52 PM
> > To: amd-gfx@lists.freedesktop.org
> > Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Liang, Prike
> > <prike.li...@amd.com>; Huang, Ray <ray.hu...@amd.com>
> > Subject: [PATCH] drm/amdgpu: remove gpu info firmware of green sardine
> >
> > The ip discovery is supported on green sardine, it doesn't need gpu info
> > firmware anymore.
> >
> > Signed-off-by: Huang Rui <ray.hu...@amd.com>
> > ---
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 3 +--
> >  1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> > b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> > index 4d434803fb49..f1a426d8861d 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> > @@ -81,7 +81,6 @@ MODULE_FIRMWARE("amdgpu/navi10_gpu_info.bin");
> >  MODULE_FIRMWARE("amdgpu/navi14_gpu_info.bin");
> >  MODULE_FIRMWARE("amdgpu/navi12_gpu_info.bin");
> >  MODULE_FIRMWARE("amdgpu/vangogh_gpu_info.bin");
> > -MODULE_FIRMWARE("amdgpu/green_sardine_gpu_info.bin");
> >
> >  #define AMDGPU_RESUME_MS2000
> >
> > @@ -1825,7 +1824,7 @@ static int
> > amdgpu_device_parse_gpu_info_fw(struct amdgpu_device *adev)
> >  if (adev->apu_flags & AMD_APU_IS_RENOIR)
> >  chip_name = "renoir";
> >  else
> > -chip_name = "green_sardine";
> > +return 0;
> >  break;
> >  case CHIP_NAVI10:
> >  chip_name = "navi10";
> > --
> > 2.25.1
> 
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to