[PATCH] drm/amd/amdgpu: disable hwmon power1_cap* for gfx 11.0.3 on vf mode

[Alex Deucher]

On Tue, Aug 26, 2025 at 6:18 AM Yang Wang <kevinyang.wang at amd.com> wrote:
>
> the PPSMC_MSG_GetPptLimit msg is not valid for gfx 11.0.3 on vf mode,
> so skiped to create power1_cap* hwmon sysfs node.
>
> Signed-off-by: Yang Wang <kevinyang.wang at amd.com>
> Reviewed-by: Asad Kamal <asad.kamal at amd.com>

Acked-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/pm/amdgpu_pm.c | 18 ++++++++++--------
>  1 file changed, 10 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/amdgpu_pm.c b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> index d5cc4d7f84e2..c9a986cf3aef 100644
> --- a/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> +++ b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> @@ -3593,14 +3593,16 @@ static umode_t hwmon_attributes_visible(struct kobject *kobj,
>                 effective_mode &= ~S_IWUSR;
>
>         /* not implemented yet for APUs other than GC 10.3.1 (vangogh) and 9.4.3 */
> -       if (((adev->family == AMDGPU_FAMILY_SI) ||
> -            ((adev->flags & AMD_IS_APU) && (gc_ver != IP_VERSION(10, 3, 1)) &&
> -             (gc_ver != IP_VERSION(9, 4, 3) && gc_ver != IP_VERSION(9, 4, 4)))) &&
> -           (attr == &sensor_dev_attr_power1_cap_max.dev_attr.attr ||
> -            attr == &sensor_dev_attr_power1_cap_min.dev_attr.attr ||
> -            attr == &sensor_dev_attr_power1_cap.dev_attr.attr ||
> -            attr == &sensor_dev_attr_power1_cap_default.dev_attr.attr))
> -               return 0;
> +       if (attr == &sensor_dev_attr_power1_cap_max.dev_attr.attr ||
> +           attr == &sensor_dev_attr_power1_cap_min.dev_attr.attr ||
> +           attr == &sensor_dev_attr_power1_cap.dev_attr.attr ||
> +           attr == &sensor_dev_attr_power1_cap_default.dev_attr.attr) {
> +               if ((adev->family == AMDGPU_FAMILY_SI) ||
> +                   ((adev->flags & AMD_IS_APU) && (gc_ver != IP_VERSION(10, 3, 1)) &&
> +                    (gc_ver != IP_VERSION(9, 4, 3) && gc_ver != IP_VERSION(9, 4, 4))) ||
> +                   (amdgpu_sriov_vf(adev) && gc_ver == IP_VERSION(11, 0, 3)))
> +                       return 0;
> +       }
>
>         /* not implemented yet for APUs having < GC 9.3.0 (Renoir) */
>         if (((adev->family == AMDGPU_FAMILY_SI) ||
> --
> 2.34.1
>