Skip to content

Fix reporting CPU temperatures when thermal zones are used - #2136

Open
PourOutYourPowerAndLove wants to merge 1 commit into
flightlessmango:masterfrom
PourOutYourPowerAndLove:cpu-again
Open

PourOutYourPowerAndLove wants to merge 1 commit into
flightlessmango:masterfrom
PourOutYourPowerAndLove:cpu-again

Conversation

@PourOutYourPowerAndLove

Copy link
Copy Markdown

The regex pattern match was wrong and wouldn't pick up on the sysfs directories because they are named like cpu_0_0_0_thermal_2 and so on

ez fix

learn git and stop using the web interface scrub
@17314642

17314642 commented Sep 6, 2026 •

Copy link
Copy Markdown
Contributor

This breaks cpu temperature on rockchip boards, and possibly some others

Instead of removing old regex, consider checking with both regexes

Image below is from a year ago, back when I was adding cpu temperature support, and you can see that in this case, cpu temperature sensor is called cpu_thermal, while newer rockchip boards have cpu0_thermal and so on
646878290-cd5df63a-9def-421b-b5c6-1f79c43aa00e

@17314642

17314642 commented Sep 6, 2026 •

Copy link
Copy Markdown
Contributor
646878839-f7bb992f-fde8-473a-bb0d-8c30e55ccb31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants