Page MenuHomeFreeBSD
Differential D5298

AXP209 Power Management Unit temperature sensor support
ClosedPublic
Actions

Authored by jmcneill_invisible.ca on Feb 16 2016, 11:05 AM.
Tags
Referenced Files
Unknown Object (File)
Sat, Jul 11, 4:25 PM
Unknown Object (File)
Sat, Jul 11, 7:30 AM
Unknown Object (File)
Tue, Jul 7, 8:49 AM
Unknown Object (File)
Sat, Jul 4, 5:56 AM
Unknown Object (File)
Fri, Jul 3, 11:52 PM
Unknown Object (File)
Fri, Jul 3, 3:38 AM
Unknown Object (File)
Thu, Jul 2, 11:44 PM
Unknown Object (File)
Thu, Jul 2, 8:50 AM
View All Files
Subscribers
andrew
imp

Details

Reviewers
None
Group Reviewers
ARM
Commits
rS295658: Add a sysctl to read the internal temperature of the AXP209 Power
Summary

This change exposes the AXP209 Power Management "Internal temperature" sensor via sysctl:

dev.axp209_pmu.0.temp: 26.1C

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jmcneill_invisible.ca updated this revision to Diff 13356.Feb 16 2016, 11:05 AM
jmcneill_invisible.ca retitled this revision from to AXP209 Power Management Unit temperature sensor support.
jmcneill_invisible.ca updated this object.
jmcneill_invisible.ca edited the test plan for this revision. (Show Details)
jmcneill_invisible.ca added a reviewer: ARM.
jmcneill_invisible.ca set the repository for this revision to rS FreeBSD src repository - subversion.
jmcneill_invisible.ca added a project: ARM.
Herald added subscribers: andrew, imp. · View Herald TranscriptFeb 16 2016, 11:05 AM
andrew added a comment.Feb 16 2016, 11:10 AM

You might need to rebase on my recent change

sys/arm/allwinner/axp209.c
198 ↗(On Diff #13356)

You don't seem to be using AXP209_TEMP in the sysctl handler.

jmcneill_invisible.ca added inline comments.Feb 16 2016, 11:13 AM
sys/arm/allwinner/axp209.c
198 ↗(On Diff #13356)

I figured we'd eventually add other sensors (voltage etc). Should I pass 0 for now?

andrew added inline comments.Feb 16 2016, 11:22 AM
sys/arm/allwinner/axp209.c
198 ↗(On Diff #13356)

I would check the value in axp209_sysctl and return ENOENT if it's invalid.

jmcneill_invisible.ca updated this revision to Diff 13359.Feb 16 2016, 11:31 AM

Added test for sensor type in axp209_sensor, cleanup whitespace.

Closed by commit rS295658: Add a sysctl to read the internal temperature of the AXP209 Power (authored by andrew). · Explain WhyFeb 16 2016, 11:52 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
arm/
allwinner/
41 lines

Diff 13364

View Options

head/sys/arm/allwinner/axp209.c

Loading...