Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp975433yba; Thu, 16 May 2019 12:00:31 -0700 (PDT) X-Google-Smtp-Source: APXvYqyKcSJHvwrLzX/8WaWklv8hg917SjjbQKQO0KOwm1OcunPtksWCoNGX/Rb0NHEPhysT9QuI X-Received: by 2002:a17:902:b584:: with SMTP id a4mr15303631pls.333.1558033231153; Thu, 16 May 2019 12:00:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1558033231; cv=none; d=google.com; s=arc-20160816; b=YbHP00q4hVNdHMCSBtfWTMPNagJAhllyPK5T6vaIWqri99r86EYdrU/YsySmnnWbHq zAXJ9wRev+3oM6Exg84J8Vk0vDWeuqZ85f4b1BMsKXs1KJtAVzaR+Pp8oHztwtxCH3Xq US2TML+b0Rn+BMFl8AANq919wYt2/q9MLeF3VZRLMj27yKoEvAoicH+GRldzV2WfY5hm vXSC+VXTh7N3FywK4AzepDet8JX2vrCuQ87rTF7tdrd4zzKi44dH3JJ7yTk50awaLhjk SvmSV+3bsRGM1w4W8E7dwMjekdMb1v3st6/wLs6wM8DiyS/kQH5BZ5vb6I2j1WlCDPOE 7kWA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=oYizfKO0iUCkCx87QTv9y+9/yguDDJWHGpNmrjjyMQE=; b=a4jF4B87Q4CHUeIPJrEFHhWm5+p3ctnNUaZQX8GUA1St+R01qntrmQPCnc5RavBBAw 8u5ekXprtozhawZo7OCCNnhLQXQY7g3MPT66YoEkOywCeUPBdYjgL6oQW69MQ3GaJ+jZ keGx3Xa3/LUkNHWm8g610e8pleO74ys4gnAY8aOBAMjNJ3QfBL3l3S+XqjnJ/tpKXmTe 37F8EDhYlamdK/wUa0e+GVSTrWLmxQWzjWXlzqKbng88BZ/Roryvqxbn0hDfcbb81y1S 9Psz5+6DxjZxJwriI8Ztk91YjEi6PGc5xdAQOg3mPEzghPiLc2o0Nu+iGWz9BMH7JY/T 2Sig== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=T1a1UQAW; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j1si5518056pgq.173.2019.05.16.12.00.14; Thu, 16 May 2019 12:00:31 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=T1a1UQAW; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726880AbfEPSOL (ORCPT + 99 others); Thu, 16 May 2019 14:14:11 -0400 Received: from mail-it1-f196.google.com ([209.85.166.196]:38344 "EHLO mail-it1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726314AbfEPSOL (ORCPT ); Thu, 16 May 2019 14:14:11 -0400 Received: by mail-it1-f196.google.com with SMTP id i63so7830387ita.3; Thu, 16 May 2019 11:14:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=oYizfKO0iUCkCx87QTv9y+9/yguDDJWHGpNmrjjyMQE=; b=T1a1UQAWEN1zwXTAxJ0W6YsgQvMdJVGrZGrDTxkDkBdftsG7bFuxqMMuyRYIbwpp3N zpUtrb0fyJY34h5ZAqJ5AKxT5uAswqiB8LO5sX7EJZTZgfYEuGvfO/pb8QwJVTKmVOqr MpYvM0qdYL46t4TGTV4SiIP/n20fxoie6h4lu9vHtVeHW6WMLT7uWNa4D7JCmGJdqinU Pff5NHk3/urnJe9KHzJ6+b6680Gwf/AWHqMdWWb+a/6DEl/LFjfmKJv/qzjGlsL+oluN QFRErVrPnqIzZrVDXUDAYofBl17/UhgtMEOjPMZ4uAeyPqGuIKtX1oJjyRayHP/2d5U4 lc4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=oYizfKO0iUCkCx87QTv9y+9/yguDDJWHGpNmrjjyMQE=; b=aI3BJkR77k41yerUhldC952Lgy/oKTQNv9pFBpp6ZHJXfWHgNppFPPKR63/OgKNvji WOEectIMFLrFFPfkZPpWVc3p+bvNXB7fLNa6FV3/3ZNjO2yw7UW26KV6kxV36U+KWg5U bmlCZOBl7xzdvEVv0lkmUZuT3kxYRUh8ba+ICWZYza8TjF4oCYJbDNVs0w/48Zf6Ie6u xeGx5x3cFESJ2dKxx1GXmfQdsCCB/RMzYNultjszjl/Ux9VUYapFRZ/H6nvcJuLi3IJp J0HnF6EkFzgNjjLUGaEgscd7vcywjSnxUJNUFno03ZVo6wOqQvSShYDvTKkOccxRog08 TLeg== X-Gm-Message-State: APjAAAXXykYoKxaINn+ab7wg/iSsFv4vFk9F6yssAfPJjgiq8rV5gEe0 Ue3/o0J0gJc52+tHt/XwASXbTEd3TW+lgHNXpyQ= X-Received: by 2002:a05:660c:105:: with SMTP id w5mr1049148itj.37.1558030450144; Thu, 16 May 2019 11:14:10 -0700 (PDT) MIME-Version: 1.0 References: <20190512082614.9045-1-tiny.windzz@gmail.com> <20190512082614.9045-4-tiny.windzz@gmail.com> <20190512134152.yrletgtiglxncyo4@flea> In-Reply-To: <20190512134152.yrletgtiglxncyo4@flea> From: Frank Lee Date: Fri, 17 May 2019 02:13:58 +0800 Message-ID: Subject: Re: [PATCH 3/3] dt-bindings: thermal: add binding document for h6 thermal controller To: Maxime Ripard Cc: rui.zhang@intel.com, Eduardo Valentin , Daniel Lezcano , robh+dt@kernel.org, Mark Rutland , Chen-Yu Tsai , catalin.marinas@arm.com, will.deacon@arm.com, David Miller , Mauro Carvalho Chehab , Greg Kroah-Hartman , Jonathan.Cameron@huawei.com, Nicolas Ferre , paulmck@linux.ibm.com, Andy Gross , olof@lixom.net, bjorn.andersson@linaro.org, Jagan Teki , marc.w.gonzalez@free.fr, stefan.wahren@i2se.com, enric.balletbo@collabora.com, Linux PM , devicetree@vger.kernel.org, Linux ARM , Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, May 12, 2019 at 9:41 PM Maxime Ripard wrote: > > Hi, > > On Sun, May 12, 2019 at 04:26:14AM -0400, Yangtao Li wrote: > > This patch adds binding document for allwinner h6 thermal controller. > > > > Signed-off-by: Yangtao Li > > --- > > .../bindings/thermal/sun50i-thermal.txt | 32 +++++++++++++++++++ > > 1 file changed, 32 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/thermal/sun50i-thermal.txt > > We're starting to convert to YAML for binding descriptions that will > allow to validate that all DT are properly using the binding. It would > be great if you could use it as well. What have been changed to this now? > > > diff --git a/Documentation/devicetree/bindings/thermal/sun50i-thermal.txt b/Documentation/devicetree/bindings/thermal/sun50i-thermal.txt > > new file mode 100644 > > index 000000000000..67eda7794262 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/thermal/sun50i-thermal.txt > > @@ -0,0 +1,32 @@ > > +Binding for Thermal Sensor of Allwinner SOC. > > + > > +This describes the device tree binding for the Allwinner thermal controller > > +which measures the on-SoC temperatures. > > + > > +Required properties: > > +- compatible: > > + - "allwinner,sun50i-h6-ths" : For H6 > > +- reg: Address range of the thermal controller > > +- clocks, clock-names: Clocks needed for the thermal controller. > > + The required clocks for h6 are: "bus". > > If there's a single clock, then we don't need clock-names Yeah, but, IIRC, H3 have two clk. So I'd like to keep it. > > > +- resets, reset-names: Reference to the reset controller controlling > > + the thermal controller. > > Ditto. Done. Thx, Yangtao > > > +- nvmem-cells: A phandle to the calibration data provided by a nvmem device. If > > + unspecified default values shall be used. > > +- nvmem-cell-names: Should be "calib" > > I thought you said that nvmem support was optional in the > driver. Maybe we could make it optional in the DT too? > > Thanks! > Maxime > > -- > Maxime Ripard, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com