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)
Fri, May 29, 2:38 PM
Unknown Object (File)
Sat, May 23, 3:32 PM
Unknown Object (File)
Sat, May 23, 1:53 PM
Unknown Object (File)
Fri, May 22, 12:52 AM
Unknown Object (File)
Thu, May 21, 2:24 AM
Unknown Object (File)
Fri, May 15, 8:20 AM
Unknown Object (File)
Apr 27 2026, 8:06 PM
Unknown Object (File)
Apr 23 2026, 11:37 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 Passed
Unit
No Test Coverage
Build Status
Buildable 2523
Build 2540: arc lint + arc unit

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

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

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

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
sys/
arm/
allwinner/
38 lines

Diff 13356

View Options

sys/arm/allwinner/axp209.c

Loading...