Differential D6378

Allwinner A83T thermal sensor support
ClosedPublic
Actions

Authored by jmcneill on May 15 2016, 8:19 PM.

Details

Reviewers

manu

Group Reviewers

ARM

Commits

rS299871: Add Allwinner A83T thermal sensor controller support.

Summary

This diff adds support for the thermal sensor found in the A83T SoC.

The A83T thermal sensor controller has three thermal sensors. Sensor 0 corresponds to CPU cluster 0, sensor 1 corresponds to CPU cluster 1, and sensor 2 corresponds to the GPU. The sensor values are exported via sysctl.

Calibration data is obtained from SRAM found in the Secure ID module.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jmcneill updated this revision to Diff 16353.May 15 2016, 8:19 PM

jmcneill retitled this revision from to Allwinner A83T thermal sensor support.

jmcneill updated this object.

jmcneill edited the test plan for this revision. (Show Details)

jmcneill added a reviewer: ARM.

jmcneill set the repository for this revision to rS FreeBSD src repository - subversion.

jmcneill added a project: ARM.

Herald added subscribers: andrew, imp. · View Herald TranscriptMay 15 2016, 8:19 PM

I've just added sid support in my tree and my driver is ~90% the same than yours so I'll merge them after.

manu accepted this revision.May 15 2016, 8:36 PM

manu added a reviewer: manu.

This revision is now accepted and ready to land.May 15 2016, 8:36 PM

manu added inline comments.May 15 2016, 8:59 PM

sys/arm/allwinner/aw_sid.c
133 ↗	(On Diff #16353)	It's probably better to use EARLY_DRIVER_MODULE with BUS_PASS_RESOURCE + BUS_PASS_ORDER_FIRST since some device will rely on it's presence.