DietPi-Banner | Implement sysfs node selection for CPU temp

[N7-BADA]

Creating a bug report/issue

• I have searched the existing open and closed issues

Required Information

• DietPi version | G_DIETPI_VERSION_CORE=9 G_DIETPI_VERSION_SUB=20 G_DIETPI_VERSION_RC=1 G_GITBRANCH='master' G_GITOWNER='MichaIng'
• Distro version | Trixie
• Kernel version | inux T50-N4 6.12.57+deb13-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.12.57-1 (2025-11-05) x86_64 GNU/Linux
• SBC model | Native PC (x86_64)

Additional Information (if applicable)

• Software title | dietpi-banner

Steps to reproduce

just SSH into system

- CPU temp : 26 °C / 78 °F : Who put me in the freezer!

Expected behaviour

Show correct CPU temp

Actual behaviour

Showing wrong CPU Temp

Extra details

the following does show correct temp

root@T50-N4:~# cat /sys/class/hwmon/hwmon1/temp1_input | awk '{printf "CPU Temp: %3.1f C\n", $1/1000}'

CPU Temp: 36.0 C

[MichaIng]

And I guess /sys/class/hwmon/hwmon0/temp1_input shows the 26 °C? There is not really a solution I can offer. If we give /sys/class/hwmon/hwmon1/temp1_input a higher priority than /sys/class/hwmon/hwmon0/temp1_input, it will break output on a lot of other systems, and native PC is not individual hardware ID like for SBCs, so we cannot make a special case either.

It is a pain that this thermal/hwmon API is used so inconsistently, instead of just being consistent in /sys/class/hwmon/hwmon0/temp1_input and /sys/class/thermal/thermal_zone0/temp being reserved for CPU in any case. Intel CPUs make use of a dedicated /sys/devices/platform/coretemp.[0-9]/hwmon/hwmon[0-9]/temp[1-9]_input API which is for CPU temperatures only, but this does not exist for other CPU vendors, it seems.

[N7-BADA]

And I guess /sys/class/hwmon/hwmon0/temp1_input shows the 26 °C? There is not really a solution I can offer. If we give /sys/class/hwmon/hwmon1/temp1_input a higher priority than /sys/class/hwmon/hwmon0/temp1_input, it will break output on a lot of other systems, and native PC is not individual hardware ID like for SBCs, so we cannot make a special case either.

It is a pain that this thermal/hwmon API is used so inconsistently, instead of just being consistent in /sys/class/hwmon/hwmon0/temp1_input and /sys/class/thermal/thermal_zone0/temp being reserved for CPU in any case. Intel CPUs make use of a dedicated /sys/devices/platform/coretemp.[0-9]/hwmon/hwmon[0-9]/temp[1-9]_input API which is for CPU temperatures only, but this does not exist for other CPU vendors, it seems.

Its an amd ryzen 7 3700x indeed.
Isn't it possible to create a manu so that the user can select the correct sensor to use if its not showing the correct one by default?

Like in the dietpi-banner command when selecting the CPU Temp then another menu pops up to either let the user write their own sensor path or have al selection of a a few default ones with their current respective temp next to it. ? Not sure if I explained correctly so hope you understand what i mean 😅

[MichaIng]

Isn't it possible to create a manu so that the user can select the correct sensor

Yeah, I think we have that feature request somewhere already. Though cannot find it on GitHub, let's turn this one into it.