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)
Thu, Mar 12, 10:33 PM
Unknown Object (File)
Thu, Mar 12, 5:28 PM
Unknown Object (File)
Thu, Mar 12, 3:30 PM
Unknown Object (File)
Thu, Mar 12, 11:24 AM
Unknown Object (File)
Thu, Mar 12, 8:32 AM
Unknown Object (File)
Wed, Mar 11, 1:20 PM
Unknown Object (File)
Wed, Mar 11, 12:44 PM
Unknown Object (File)
Wed, Mar 4, 1:39 PM
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...