On Mon, Sep 10, 2018 at 02:44:24PM -0500, Rob Herring wrote: > On Fri, Aug 31, 2018 at 10:48:54AM +0200, Maxime Ripard wrote: > > On Thu, Aug 30, 2018 at 05:45:02PM +0200, Philipp Rossak wrote: > > > Allwinner H3 features a thermal sensor like the one in A33, but has its > > > register re-arranged, the clock divider moved to CCU (originally the > > > clock divider is in ADC) and added a pair of bus clock and reset. > > > > > > Allwinner A83T features a thermal sensor similar to the H3, the ths clock, > > > the bus clock and the reset was removed from the CCU. The THS in A83T > > > has a clock that is directly connected and runs with 24 MHz. > > > > > > Update the binding document to cover H3 and A83T. > > > > > > Signed-off-by: Philipp Rossak > > > > You probably want to have a look at: > > https://www.spinics.net/lists/arm-kernel/msg670167.html > > Well, which is it? An ADC or thermal sensor? It's both actually. This IP used to be called GPADC, and had a thermal sensor + some ADC channels. The design evolved across several generations of SoCs to drop the ADC channels and be used only to have thermal sensors. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com