Re: [PATCH 1/1] drm/amd/powerplay: Fix DCEFCLK related compilation error for arcturus

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

 



[AMD Public Use]


No problem.

Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

From: Das, Nirmoy <Nirmoy.Das@xxxxxxx>
Sent: Tuesday, June 23, 2020 12:04 PM
To: Nirmoy Das <nirmoy.aiemd@xxxxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>
Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Das, Nirmoy <Nirmoy.Das@xxxxxxx>
Subject: Re: [PATCH 1/1] drm/amd/powerplay: Fix DCEFCLK related compilation error for arcturus
 

On 6/23/20 5:59 PM, Nirmoy Das wrote:
> arcturus doesn't support DCEFCLK
>
> Fixes: c67c791cd87d (drm/amd/powerplay: return current DCEFCLK on sysfs read)


Hi Alex,


Can you please squash this with c67c791cd87d (drm/amd/powerplay: return
current DCEFCLK on sysfs read)

I was bit too confident that arcturus changes would just work.


Regards,

Nirmoy


>
> Signed-off-by: Nirmoy Das <nirmoy.das@xxxxxxx>
> ---
>   drivers/gpu/drm/amd/powerplay/arcturus_ppt.c | 3 ---
>   1 file changed, 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
> index daeae14fd380..d93f8a43a96f 100644
> --- a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
> +++ b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c
> @@ -962,9 +962,6 @@ static int arcturus_get_smu_metrics_data(struct smu_context *smu,
>        case METRICS_CURR_FCLK:
>                *value = metrics->CurrClock[PPCLK_FCLK];
>                break;
> -     case METRICS_CURR_DCEFCLK:
> -             *value = metrics->CurrClock[PPCLK_DCEFCLK];
> -             break;
>        case METRICS_AVERAGE_GFXCLK:
>                *value = metrics->AverageGfxclkFrequency;
>                break;
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

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

  Powered by Linux