Reviewed-by: Evan Quan <evan.q...@amd.com>

> -----Original Message-----
> From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of Alex
> Deucher
> Sent: 2018年11月29日 2:52
> To: amd-gfx@lists.freedesktop.org
> Cc: Deucher, Alexander <alexander.deuc...@amd.com>;
> sta...@vger.kernel.org
> Subject: [PATCH] drm/amdgpu: don't expose fan attributes on APUs
> 
> They don't have a fan controller.
> 
> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
> Cc: sta...@vger.kernel.org
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> index 0de8650c5d6e..1f61ed95727c 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> @@ -1644,6 +1644,19 @@ static umode_t hwmon_attributes_visible(struct
> kobject *kobj,
>           attr == &sensor_dev_attr_fan1_enable.dev_attr.attr))
>               return 0;
> 
> +     /* Skip fan attributes on APU */
> +     if ((adev->flags & AMD_IS_APU) &&
> +         (attr == &sensor_dev_attr_pwm1.dev_attr.attr ||
> +          attr == &sensor_dev_attr_pwm1_enable.dev_attr.attr ||
> +          attr == &sensor_dev_attr_pwm1_max.dev_attr.attr ||
> +          attr == &sensor_dev_attr_pwm1_min.dev_attr.attr ||
> +          attr == &sensor_dev_attr_fan1_input.dev_attr.attr ||
> +          attr == &sensor_dev_attr_fan1_min.dev_attr.attr ||
> +          attr == &sensor_dev_attr_fan1_max.dev_attr.attr ||
> +          attr == &sensor_dev_attr_fan1_target.dev_attr.attr ||
> +          attr == &sensor_dev_attr_fan1_enable.dev_attr.attr))
> +             return 0;
> +
>       /* Skip limit attributes if DPM is not enabled */
>       if (!adev->pm.dpm_enabled &&
>           (attr == &sensor_dev_attr_temp1_crit.dev_attr.attr ||
> --
> 2.13.6
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to