Differential D19145

Expose AXP803 related sensors values via sysctl
ClosedPublic
Actions

Authored by ganbold on Feb 11 2019, 12:57 PM.

Details

Reviewers

Commits

rS344003: Add sensors support for AXP803/AXP813. Sensor values such as

Summary

Add sensors support for AXP803/AXP813. Sensor values such as
battery charging, charge state, voltage, charging current, discharging current,
battery capacity etc. can be obtained via sysctl.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ganbold created this revision.Feb 11 2019, 12:57 PM

Herald added subscribers: andrew, imp. · View Herald TranscriptFeb 11 2019, 12:57 PM

Harbormaster completed remote builds in B22430: Diff 53773.Feb 11 2019, 12:57 PM

manu added inline comments.Feb 11 2019, 1:44 PM

sys/arm/allwinner/axp81x.c
649 ↗	(On Diff #53773)	has_battery and has_fuel_gauge is always true. Don't think those variable are necessary.
677 ↗	(On Diff #53773)	Does AXP813 and AXP803 differs in battery management? I remember that there is a few subtle different between them but I can't remember what. If not just adding one config would be enough.