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

Update to get correct CPU temp #2563

Merged
merged 1 commit into from
Feb 17, 2019
Merged

Update to get correct CPU temp #2563

merged 1 commit into from
Feb 17, 2019

Conversation

lwqcz
Copy link
Contributor

@lwqcz lwqcz commented Feb 17, 2019

Update to get correct CPU temperature reading on Pine64 with sunxi64 based kernel. As it gives temperature multiplied by 1000 and this value is integer instead of float. Just a minor change.

Commit list/description:

  • DietPi-CPUInfo | Minor change of func/dietpi-globals file to correct behaviour on Pine platform.

Update to get correct CPU temperature reading on Pine64 with sunxi64 based kernel. As it gives temperature multiplied by 1000 and this value is integer instead of float. Just a minor change.
@Fourdee
Copy link
Collaborator

Fourdee commented Feb 17, 2019

@lwqcz

Many thanks for the PR, looks good will merge 👍

Please email me your address for some free stickers! [email protected]

@Fourdee Fourdee added this to the v6.22 milestone Feb 17, 2019
@Fourdee Fourdee merged commit 843f403 into MichaIng:dev Feb 17, 2019
@OstrovCity
Copy link

Update to get correct CPU temperature on Core2duo.
in G_OBTAIN_CPU_TEMP() add string '/sys/class/hwmon/hwmon0/temp2_input' to array afp_temperature

local afp_temperature=(
	'/sys/class/thermal/thermal_zone0/temp'
	'/sys/devices/platform/sunxi-i2c.0/i2c-0/0-0034/temp1_input'
	'/sys/class/hwmon/hwmon0/device/temp_label'
	'/sys/class/hwmon/hwmon0/temp2_input'
)

@MichaIng
Copy link
Owner

@OstrovCity
Many thanks for this. temp0_input and/or temp1_input do not show correct data?

@OstrovCity
Copy link

there are no such files in /sys/class/hwmon/hwmon0:
~device
/power
~subsystem
name
temp2_crit
temp2_crit_alarm
temp2_input
temp2_label
temp2_max
temp3_crit
temp3_crit_alarm
temp3_input
temp3_label
temp3_max
uevent

@MichaIng
Copy link
Owner

MichaIng commented May 21, 2019

@OstrovCity
Okay perfect, will add it!

Done: 662ff28
Changelog: https://github.com/MichaIng/DietPi/blob/dev/CHANGELOG.txt

@MichaIng MichaIng mentioned this pull request Jun 24, 2019
@MichaIng MichaIng mentioned this pull request Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants