RE: [PATCH] drm/amdgpu: simplify amdgpu_ucode_get_load_type()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Reviewed-by: Guchun Chen <guchun.chen@xxxxxxx>

Regards,
Guchun

-----Original Message-----
From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Alex Deucher
Sent: Tuesday, June 7, 2022 2:40 AM
To: Deucher, Alexander <Alexander.Deucher@xxxxxxx>
Cc: amd-gfx list <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH] drm/amdgpu: simplify amdgpu_ucode_get_load_type()

Ping?

Alex

On Tue, May 24, 2022 at 10:09 PM Alex Deucher <alexander.deucher@xxxxxxx> wrote:
>
> This is the same as the default case, so drop the extra logic.
>
> Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 20 --------------------
>  1 file changed, 20 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
> index ffa4c0d207db..c312577df596 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
> @@ -486,26 +486,6 @@ amdgpu_ucode_get_load_type(struct amdgpu_device *adev, int load_type)
>         case CHIP_POLARIS12:
>         case CHIP_VEGAM:
>                 return AMDGPU_FW_LOAD_SMU;
> -       case CHIP_VEGA10:
> -       case CHIP_RAVEN:
> -       case CHIP_VEGA12:
> -       case CHIP_VEGA20:
> -       case CHIP_ARCTURUS:
> -       case CHIP_RENOIR:
> -       case CHIP_NAVI10:
> -       case CHIP_NAVI14:
> -       case CHIP_NAVI12:
> -       case CHIP_SIENNA_CICHLID:
> -       case CHIP_NAVY_FLOUNDER:
> -       case CHIP_VANGOGH:
> -       case CHIP_DIMGREY_CAVEFISH:
> -       case CHIP_ALDEBARAN:
> -       case CHIP_BEIGE_GOBY:
> -       case CHIP_YELLOW_CARP:
> -               if (!load_type)
> -                       return AMDGPU_FW_LOAD_DIRECT;
> -               else
> -                       return AMDGPU_FW_LOAD_PSP;
>         case CHIP_CYAN_SKILLFISH:
>                 if (!(load_type &&
>                       adev->apu_flags & AMD_APU_IS_CYAN_SKILLFISH2))
> --
> 2.35.3
>




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux