search

darkxst / ha-operating-system

HAOS for iHost
Apache License 2.0
36 stars 4 forks source link

CPU temperature #5

Closed riddik14 closed 1 month ago

riddik14 commented 7 months ago

Hi but command

cat /sys/class/thermal/thermal_zone*/temp

Not work. alternatives?

darkxst commented 7 months ago

Thermal Sensors, along with dynamic frequency scaling and PVTM are not yet implemented for rv1126/rv1109 SoC's in the mainline kernel. It will require porting some driver code across from Rockchip kernel in addition to the usual devicetree nodes.

I will look at adding these in a future release.

riddik14 commented 7 months ago

Ok. then this issue could become a pull request?

darkxst commented 7 months ago

Yes sure, if you would like to work on this I will happily merge any pull requests.

Thermels should be fairly straight forward, you can find a similar patch series that was for rv1108 thermal, that atleast will give you an idea of which parts need to be picked up from Rockchip kernel. Actual code will differ.

OPP is also straight forward, may not need any driver changes just devicetree nodes PVTM I've not really looked at, but probably leave this for last!

riddik14 commented 7 months ago

I thank you for your effort but I don't know how to fix the drivers or the kernel.

fabianoarruda commented 4 months ago

+1 for this, I think would be a nice feature to have 🙌

github-actions[bot] commented 1 month ago

There hasn't been any activity on this issue recently. To keep our backlog manageable we have to clean old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant OS version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.