On Tue, Aug 26, 2025 at 6:18 AM Yang Wang <kevinyang.w...@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.w...@amd.com>
> Reviewed-by: Asad Kamal <asad.ka...@amd.com>
Acked-by: Alex Deucher <alexander.deuc...@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
>
