Re: [PATCH] drm/amd/pm: Fix incorrect comment about Vangogh power cap support

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

 



Applied.  Thanks!

On Thu, Mar 30, 2023 at 11:32 AM Guilherme G. Piccoli
<gpiccoli@xxxxxxxxxx> wrote:
>
> The comment mentions that power1 cap attributes are not supported on
> Vangogh, but the opposite is indeed valid: for APUs, only Vangogh is
> supported. While at it, also fixed the Renoir comment below (thanks
> Melissa for noticing that!).
>
> Cc: Lijo Lazar <lijo.lazar@xxxxxxx>
> Cc: Melissa Wen <mwen@xxxxxxxxxx>
> Signed-off-by: Guilherme G. Piccoli <gpiccoli@xxxxxxxxxx>
> ---
>
>
> Hi folks, this is a very trivial one, just to fix the comments - since I needed
> to re-read the code/comments to figure out, I'm trying to improve it here to
> avoid others to get confused as well heh
>
> Lemme know in case I made some mistake (even after re-re-reading the code).
> Cheers,
>
>
> Guilherme
>
>
>  drivers/gpu/drm/amd/pm/amdgpu_pm.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/amdgpu_pm.c b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> index d75a67cfe523..6460906c354c 100644
> --- a/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> +++ b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> @@ -3351,7 +3351,7 @@ static umode_t hwmon_attributes_visible(struct kobject *kobj,
>               attr == &sensor_dev_attr_pwm1_enable.dev_attr.attr)) /* can't manage state */
>                 effective_mode &= ~S_IWUSR;
>
> -       /* not implemented yet for GC 10.3.1 APUs */
> +       /* In the case of APUs, this is only implemented on Vangogh */
>         if (((adev->family == AMDGPU_FAMILY_SI) ||
>              ((adev->flags & AMD_IS_APU) && (gc_ver != IP_VERSION(10, 3, 1)))) &&
>             (attr == &sensor_dev_attr_power1_cap_max.dev_attr.attr ||
> @@ -3360,7 +3360,7 @@ static umode_t hwmon_attributes_visible(struct kobject *kobj,
>              attr == &sensor_dev_attr_power1_cap_default.dev_attr.attr))
>                 return 0;
>
> -       /* not implemented yet for APUs having <= GC 9.3.0 */
> +       /* not implemented yet for APUs having < GC 9.3.0 (Renoir) */
>         if (((adev->family == AMDGPU_FAMILY_SI) ||
>              ((adev->flags & AMD_IS_APU) && (gc_ver < IP_VERSION(9, 3, 0)))) &&
>             (attr == &sensor_dev_attr_power1_average.dev_attr.attr))
> --
> 2.40.0
>




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

  Powered by Linux