Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for Snapdragon 8cx #1589

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

treibholz
Copy link

Probably also works for other Snapdragons, but I can only test it on my 8cx.

The code is not beautiful, but "works on my machine" :-) (Thinkpad X13s).

Based the output of sensors:

$ sensors
skin_temp_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.7°C

cluster0_thermal-virtual-0
Adapter: Virtual device
temp1:        +33.7°C

cpu2_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.6°C

cpu6_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.3°C

cpu0_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.6°C

cpu4_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.6°C

nvme-pci-20100
Adapter: PCI adapter
Composite:    +29.9°C  (low  = -20.1°C, high = +77.8°C)
                       (crit = +81.8°C)
Sensor 1:     +29.9°C  (low  = -273.1°C, high = +65261.8°C)

ath11k_hwmon-pci-60100
Adapter: PCI adapter
temp1:        +33.0°C

pm8280_1_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.6°C

qcom_battmgr_bat-virtual-0
Adapter: Virtual device
in0:           7.60 V
temp:         +30.3°C
power1:       -4.25 W

mem_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.1°C

cpu3_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.6°C

cpu7_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.3°C

cpu1_thermal-virtual-0
Adapter: Virtual device
temp1:        +32.3°C

cpu5_thermal-virtual-0
Adapter: Virtual device
temp1:        +33.0°C

gpu_thermal-virtual-0
Adapter: Virtual device
temp1:        +31.5°C

pm8280_2_thermal-virtual-0
Adapter: Virtual device
temp1:        +30.6°C

probably also works for other Snapdragons.
linux/LibSensors.c Outdated Show resolved Hide resolved
@BenBE BenBE linked an issue Jan 17, 2025 that may be closed by this pull request
@BenBE
Copy link
Member

BenBE commented Jan 17, 2025

Mind to squash these two commits?

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.

Missing Temperature on Snapdragon 8cx Gen 3
2 participants