On Wed, Aug 28, 2019 at 4:32 AM Jean Delvare <jdelv...@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 <jdelv...@suse.de> > Cc: Alex Deucher <alexander.deuc...@amd.com> > Cc: "Christian König" <christian.koe...@amd.com> > Cc: "David (ChunMing) Zhou" <david1.z...@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@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