Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753121AbdFUCMb (ORCPT ); Tue, 20 Jun 2017 22:12:31 -0400 Received: from szxga01-in.huawei.com ([45.249.212.187]:8768 "EHLO szxga01-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752917AbdFUCM3 (ORCPT ); Tue, 20 Jun 2017 22:12:29 -0400 Subject: Re: [PATCH 1/3] dt-bindings: Document the hi3660 thermal sensor bindings To: Wei Xu , , , , , , CC: , , , , , References: <1497930035-60894-1-git-send-email-kevin.wangtao@hisilicon.com> <5948F89F.3070005@hisilicon.com> From: "Wangtao (Kevin, Kirin)" Message-ID: <9cccdaad-df22-0e4e-3c29-69d80e04832c@hisilicon.com> Date: Wed, 21 Jun 2017 10:10:08 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 MIME-Version: 1.0 In-Reply-To: <5948F89F.3070005@hisilicon.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.121.88.70] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020204.5949D5FD.00F5,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2014-11-16 11:51:01, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: 5b9b41d536207525aeb98dd6b11e8b0e Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1399 Lines: 50 e( 2017/6/20 18:27, Wei Xu ei: > Hi Tao, > > On 2017/6/20 4:40, Tao Wang wrote: >> This adds documentation of device tree bindings for the >> thermal sensor controller of hi3660 SoC. >> >> Signed-off-by: Tao Wang >> --- >> .../devicetree/bindings/thermal/hi3660-thermal.txt | 17 +++++++++++++++++ >> 1 file changed, 17 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/thermal/hi3660-thermal.txt >> >> diff --git a/Documentation/devicetree/bindings/thermal/hi3660-thermal.txt b/Documentation/devicetree/bindings/thermal/hi3660-thermal.txt >> new file mode 100644 >> index 0000000..c034670 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/thermal/hi3660-thermal.txt >> @@ -0,0 +1,17 @@ >> +* Temperature Sensor on hisilicon hi3660 SoC >> + >> +** Required properties : >> + >> +- compatible: "hisilicon,thermal-hi3660". > > It is better to put the SoC name before the function string. ok, I will fix it, thank you. > > BR, > Wei > >> +- reg: physical base address of thermal sensor and length of memory mapped >> + region. >> +- #thermal-sensor-cells: Should be 1. See ./thermal.txt for a description. >> + >> +Example : >> + >> + tsensor: tsensor { >> + compatible = "hisilicon,thermal-hi3660"; >> + reg = <0x0 0xfff30000 0x0 0x1000>; >> + #thermal-sensor-cells = <1>; >> + status = "ok"; >> + }; >> > > > . >