[PATCH] drm/amdgpu/pm: Don't show pp_power_profile_mode for YC and later APUs

[Alex Deucher]

Right.

Alex

On Sun, Oct 31, 2021 at 11:41 PM Lazar, Lijo <Lijo.Lazar at amd.com> wrote:
>
> There are two subsystems - powerplay and swsmu. The subsystem implementation details are hidden from amdgpu_pm funcs. I thought Alex is talking about that.
>
> Thanks,
> Lijo
> ________________________________
> From: Limonciello, Mario <Mario.Limonciello at amd.com>
> Sent: Monday, November 1, 2021 8:16:27 AM
> To: Lazar, Lijo <Lijo.Lazar at amd.com>; Alex Deucher <alexdeucher at gmail.com>
> Cc: amd-gfx at lists.freedesktop.org <amd-gfx at lists.freedesktop.org>; Deucher, Alexander <Alexander.Deucher at amd.com>
> Subject: RE: [PATCH] drm/amdgpu/pm: Don't show pp_power_profile_mode for YC and later APUs
>
>
> [Public]
>
>
>
> smu_get_power_profile_mode actually calls smu->ppt_funcs->get_power_profile_mode
>
>
>
> So shouldn’t it be sufficient to do this?
>
>
>
>         } else if (DEVICE_ATTR_IS(pp_power_profile_mode)) {
>
>                 if (!adev->smu.ppt_funcs->get_power_profile_mode)
>
>                         *states = ATTR_STATE_UNSUPPORTED;
>
>         }
>
>
>
>
>
> From: Lazar, Lijo <Lijo.Lazar at amd.com>
> Sent: Saturday, October 30, 2021 10:06
> To: Alex Deucher <alexdeucher at gmail.com>
> Cc: Limonciello, Mario <Mario.Limonciello at amd.com>; amd-gfx at lists.freedesktop.org; Deucher, Alexander <Alexander.Deucher at amd.com>
> Subject: Re: [PATCH] drm/amdgpu/pm: Don't show pp_power_profile_mode for YC and later APUs
>
>
>
> [Public]
>
>
>
> For swsmu, we could add a helper macro like smu_is_supported(x) that checks if ppt func is not NULL.
>
>
>
> Thanks,
> Lijo