Add CPU temp sensor path selection option#8012
Conversation
|
Note for myself: read through this PR later and see if it could be applied to make dashboard temp sensing more accurate. |
|
Applied both suggestions, renamed the setting from SOFTWARE_DIETPI_CPU_TEMP_PATH to CONFIG_CPU_TEMP_PATH Updated dietpi-config accordingly to use CONFIG_CPU_TEMP_PATH Tested again with new changes and still works. |
d444016 to
5beefa7
Compare
|
I moved the menu to performance options and simplified it by using the sensor paths as menu index. Also added the coretemp API. But those paths can be long, let's see whether it looks fine, else we could implement some mechanism to only show the last X characters in the menu, prefixed with The display options are meant to be about video output and camera features. The benefit about performance options is that it already shows the temperature above the menu. So one can see the effect immediately, and if spottet false temperature there the right option is right below. |
Implements sysfs node selection for CPU temp MichaIng#7870 - Add SOFTWARE_DIETPI_CPU_TEMP_PATH option in dietpi.txt - Add sensor selection in dietpi-config Display Options - Update dietpi-globals to use custom path
5beefa7 to
38cd64c
Compare
And simplify menu/code by using the path itself as index
38cd64c to
5cc4229
Compare
MichaIng
left a comment
There was a problem hiding this comment.
Works great. On narrow screens it might look weird, but otherwise the max menu width of 120 characters has more than enough space.
Closes #7870
Adds option to manually select CPU temperature sensor path.
Changes:
dietpi.txt: newSOFTWARE_DIETPI_CPU_TEMP_PATHsettingdietpi-globals: check for user-defined path before auto-detectiondietpi-config: menu option under Display Options to select sensor from available sysfs nodesUsers can pick from detected sensors, enter a custom path, or keep auto-detect.