On Mon, Sep 18, 2017 at 03:47:25PM +0000, icenowy-h8G6r0blFSE@public.gmane.org wrote:

在 2017-09-18 16:30，Maxime Ripard 写道：

quoted

On Mon, Sep 18, 2017 at 03:36:43PM +0800, Icenowy Zheng wrote:

quoted

于 2017年9月18日 GMT+08:00 下午3:33:36, Maxime Ripard
[off-list ref] 写到:

quoted

On Thu, Sep 14, 2017 at 10:52:46PM +0800, Icenowy Zheng wrote:

quoted

Allwinner H3 features a thermal sensor like the one in A33, but has

its

quoted

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.

Update the binding document to cover H3.

Signed-off-by: Icenowy Zheng <redacted>
Reviewed-by: Chen-Yu Tsai <redacted>
---
Changes in v4:
- Add nvmem calibration data (not yet used by the driver)
Changes in v3:
- Clock name changes.
- Example node name changes.
- Add interupts (not yet used by the driver).

 .../devicetree/bindings/mfd/sun4i-gpadc.txt        | 30

++++++++++++++++++++--

quoted

 1 file changed, 28 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/mfd/sun4i-gpadc.txt

b/Documentation/devicetree/bindings/mfd/sun4i-gpadc.txt

quoted

index badff3611a98..6c470d584bf9 100644

--- a/Documentation/devicetree/bindings/mfd/sun4i-gpadc.txt
+++ b/Documentation/devicetree/bindings/mfd/sun4i-gpadc.txt

@@ -4,12 +4,26 @@ The Allwinner SoCs all have an ADC that can also

act as a thermal sensor

quoted

 and sometimes as a touchscreen controller.

 Required properties:
-  - compatible: "allwinner,sun8i-a33-ths",
+  - compatible: must contain one of the following compatibles:
+		- "allwinner,sun8i-a33-ths"
+		- "allwinner,sun8i-h3-ths"
   - reg: mmio address range of the chip,
   - #thermal-sensor-cells: shall be 0,
   - #io-channel-cells: shall be 0,

-Example:
+Optional properties:
+  - nvmem-cells: A phandle to the calibration data provided by a

nvmem device.

quoted

+                 If unspecified default values shall be used.
+  - nvmem-cell-names: Should be "calibration-data"

I'd prefer to have which sensor it applies to here. It wouldn't change
anything for the H3, but it definitely does for example for the A83t
that has two sensors, one for each cluster, and one for the GPU, each
with calibration data.

What about cluster0-calibration?

I prefer sensor0-calibration to sensor3-calibration now.
(Theortically the new generation THS can support up to 4 sensors)

The mapping that explains what sensor0 means can change in the
future. It's better to be explicit here, and just say upfront what
it's about.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.