[PATCH] drm/amd/amdgpu: hide voltage and power sensors on SI and KV parts

Alex Deucher alexdeucher at gmail.com
Wed Aug 28 20:22:33 UTC 2019


On Wed, Aug 28, 2019 at 4:32 AM Jean Delvare <jdelvare at suse.de> wrote:
>
> The driver does not support these sensors yet and there is no point in
> creating sysfs attributes which will always return an error.
>
> Signed-off-by: Jean Delvare <jdelvare at suse.de>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Cc: "Christian K├Ânig" <christian.koenig at amd.com>
> Cc: "David (ChunMing) Zhou" <David1.Zhou at amd.com>
> ---
> This works for me however I couldn't seen any other place in the driver
> where adev->family is tested directly like that, so I'm not sure if
> this is the right way to perform the test?

That's fine.  Applied.  Thanks!

Alex

>
>  drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c |   10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
>
> --- linux-5.2.orig/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c       2019-08-28 09:38:11.245376539 +0200
> +++ linux-5.2/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c    2019-08-28 09:58:59.896314948 +0200
> @@ -2150,7 +2150,9 @@ static umode_t hwmon_attributes_visible(
>                         effective_mode &= ~S_IWUSR;
>         }
>
> -       if ((adev->flags & AMD_IS_APU) &&
> +       if (((adev->flags & AMD_IS_APU) ||
> +            adev->family == AMDGPU_FAMILY_SI ||        /* not implemented yet */
> +            adev->family == AMDGPU_FAMILY_KV) &&       /* not implemented yet */
>             (attr == &sensor_dev_attr_power1_average.dev_attr.attr ||
>              attr == &sensor_dev_attr_power1_cap_max.dev_attr.attr ||
>              attr == &sensor_dev_attr_power1_cap_min.dev_attr.attr||
> @@ -2174,6 +2176,12 @@ static umode_t hwmon_attributes_visible(
>                         return 0;
>         }
>
> +       if ((adev->family == AMDGPU_FAMILY_SI ||        /* not implemented yet */
> +            adev->family == AMDGPU_FAMILY_KV) &&       /* not implemented yet */
> +           (attr == &sensor_dev_attr_in0_input.dev_attr.attr ||
> +            attr == &sensor_dev_attr_in0_label.dev_attr.attr))
> +               return 0;
> +
>         /* only APUs have vddnb */
>         if (!(adev->flags & AMD_IS_APU) &&
>             (attr == &sensor_dev_attr_in1_input.dev_attr.attr ||
>
>
> --
> Jean Delvare
> SUSE L3 Support
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list