mate-desktop / mate-sensors-applet

Display readings from hardware sensors in your MATE panel
http://www.mate-desktop.org
GNU General Public License v2.0
16 stars 13 forks source link

sensor names with yoga c630 using the thermal to hwmon connector #116

Open bm16ton opened 4 years ago

bm16ton commented 4 years ago

Expected behaviour

different names for each sensor

Actual behaviour

temp1 is listed for all 20 sensors under label and sensor

Steps to reproduce the behaviour

install on any yoga c630 arm64 laptop (and possibly anybox with thermal drivers connected to hwmon via THERMAL_HWMON kernel config option)

MATE general version

1.24

Package version

1.24.0+1

Linux Distribution

ubuntu 20.04

So mate applets currently displays temp1 for label and sensor. I know when THERMAL_HWMON is used/needed it put all temps in the /sys/class/hwmon/hwmon*/temp1_input, I tried looking at your code to see if it was using the folder name etc, but it was above my head. sensors -A
shows correct names IE # I used the -A to remove adapters to make it more clear; cpu6_thermal-virtual-0 temp1: +39.7°C (crit = +100.0°C)

I could see sum of the code was borrowed from the sensors app, but I couldnt figure out which code showed which or where to replace. If a command line option or a build option to switch the naming could happen thatd be great. If not if anyone can help me switch the code to show the correct name, Id be greatly appreciative! The only sensor with a diff name is in0 for the battery and its placement in the list changes sporadically making it impossible to figure out how the list is ordered. Thank you