From: "rocky.hao" <rocky.hao@rock-chips.com>
To: "Heiko Stübner" <heiko@sntech.de>
Cc: rui.zhang@intel.com, edubezval@gmail.com, robh+dt@kernel.org,
mark.rutland@arm.com, catalin.marinas@arm.com,
will.deacon@arm.com, shawn.lin@rock-chips.com, cl@rock-chips.com,
william.wu@rock-chips.com, linux-pm@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org,
devicetree@vger.kernel.org, xxx@rock-chips.com,
jay.xu@rock-chips.com, wxt@rock-chips.com,
huangtao@rock-chips.com
Subject: Re: [PATCH 1/5] dt-bindings: rockchip-thermal: Support the RK3328 SoC compatible
Date: Thu, 27 Jul 2017 14:49:37 +0800 [thread overview]
Message-ID: <4c4a3974-e841-3d98-9850-131711c40c61@rock-chips.com> (raw)
In-Reply-To: <4621265.xPXIvBV6Fl@diego>
Hi Heiko,
Thank you for the quick reply. I will remove the gerrit Change-Ids in my
v2 patches.
Thanks again,
Rocky
在 2017/7/27 14:36, Heiko Stübner 写道:
> Hi Rocky,
>
> Am Dienstag, 25. Juli 2017, 17:09:44 CEST schrieb Rocky Hao:
>> attempt to new compatible for thermal founding on RK3328 SoC.
>>
>> Change-Id: Ie6f6d7b4e545891a0035851a923c0412f584140c
>> Signed-off-by: Rocky Hao <rocky.hao@rock-chips.com>
>
> generally looks good but you should not have gerrit Change-Ids in patches
> for mainline. This is true for all patches in this series.
>
>
> Heiko
>
>> ---
>> Documentation/devicetree/bindings/thermal/rockchip-thermal.txt | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/thermal/rockchip-thermal.txt
>> b/Documentation/devicetree/bindings/thermal/rockchip-thermal.txt index
>> 43003aec94bd..e3a6234fb1ac 100644
>> --- a/Documentation/devicetree/bindings/thermal/rockchip-thermal.txt
>> +++ b/Documentation/devicetree/bindings/thermal/rockchip-thermal.txt
>> @@ -4,6 +4,7 @@ Required properties:
>> - compatible : should be "rockchip,<name>-tsadc"
>> "rockchip,rk3228-tsadc": found on RK3228 SoCs
>> "rockchip,rk3288-tsadc": found on RK3288 SoCs
>> + "rockchip,rk3328-tsadc": found on RK3328 SoCs
>> "rockchip,rk3368-tsadc": found on RK3368 SoCs
>> "rockchip,rk3399-tsadc": found on RK3399 SoCs
>> - reg : physical base address of the controller and length of memory mapped
>
>
>
>
>
next prev parent reply other threads:[~2017-07-27 6:49 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-25 9:09 [PATCH 0/5] thermal: rockchip: add tsadc support in thermal driver and IPA thermal control for rk3328 in dts Rocky Hao
2017-07-25 9:09 ` [PATCH 1/5] dt-bindings: rockchip-thermal: Support the RK3328 SoC compatible Rocky Hao
2017-07-27 6:36 ` Heiko Stübner
2017-07-27 6:49 ` rocky.hao [this message]
2017-07-25 9:09 ` [PATCH 2/5] thermal: rockchip: Support the RK3328 SOC in thermal driver Rocky Hao
2017-08-11 3:02 ` Zhang Rui
2017-08-11 4:03 ` Caesar Wang
2017-07-25 9:09 ` [PATCH 3/5] arm64: dts: rockchip: add tsadc node for rk3328 SoC Rocky Hao
2017-07-25 9:09 ` [PATCH 4/5] arm64: dts: rockchip: add thermal nodes " Rocky Hao
2017-07-25 9:16 ` [PATCH 5/5] arm64: dts: rockchip: Enable tsadc module on RK3328 eavluation board Rocky Hao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4c4a3974-e841-3d98-9850-131711c40c61@rock-chips.com \
--to=rocky.hao@rock-chips.com \
--cc=catalin.marinas@arm.com \
--cc=cl@rock-chips.com \
--cc=devicetree@vger.kernel.org \
--cc=edubezval@gmail.com \
--cc=heiko@sntech.de \
--cc=huangtao@rock-chips.com \
--cc=jay.xu@rock-chips.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mark.rutland@arm.com \
--cc=robh+dt@kernel.org \
--cc=rui.zhang@intel.com \
--cc=shawn.lin@rock-chips.com \
--cc=will.deacon@arm.com \
--cc=william.wu@rock-chips.com \
--cc=wxt@rock-chips.com \
--cc=xxx@rock-chips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®