[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAL_JsqJV5TA2ADo35tqGXC61NWK9PDqw8Dq5m5fYHg59yaoEUA@mail.gmail.com>
Date: Thu, 25 May 2017 07:38:38 -0500
From: Rob Herring <robh@...nel.org>
To: songxiaowei <songxiaowei@...ilicon.com>
Cc: "Wangzhou (B)" <wangzhou1@...ilicon.com>,
Gabriele Paoloni <gabriele.paoloni@...wei.com>,
"bhelgaas@...gle.com" <bhelgaas@...gle.com>,
"mark.rutland@....com" <mark.rutland@....com>,
"xuwei (O)" <xuwei5@...ilicon.com>,
"catalin.marinas@....com" <catalin.marinas@....com>,
"will.deacon@....com" <will.deacon@....com>,
"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"Chenfeng (puck)" <puck.chen@...ilicon.com>,
"guodong.xu@...aro.org" <guodong.xu@...aro.org>,
Wangbinghui <wangbinghui@...ilicon.com>,
Suzhuangluan <suzhuangluan@...ilicon.com>
Subject: Re: 答复: [Patch v3 1/3] arm64: dts: hi3660: add pcie node
On Tue, May 23, 2017 at 8:39 PM, songxiaowei <songxiaowei@...ilicon.com> wrote:
> -----邮件原件-----
> 发件人: Rob Herring [mailto:robh@...nel.org]
> 发送时间: 2017年5月23日 22:17
> 收件人: songxiaowei
> 抄送: Wangzhou (B); Gabriele Paoloni; bhelgaas@...gle.com; mark.rutland@....com; xuwei (O); catalin.marinas@....com; will.deacon@....com; linux-pci@...r.kernel.org; devicetree@...r.kernel.org; linux-kernel@...r.kernel.org; Chenfeng (puck); guodong.xu@...aro.org; Wangbinghui; Suzhuangluan
> 主题: Re: [Patch v3 1/3] arm64: dts: hi3660: add pcie node
>
> On Thu, May 18, 2017 at 03:49:46PM +0800, Song Xiaowei wrote:
>> Add PCIe node for hi3660, and add binding documentation.
>>
>> Cc: Guodong Xu <guodong.xu@...aro.org>
>> Signed-off-by: Song Xiaowei <songxiaowei@...ilicon.com>
>> ---
>> .../devicetree/bindings/pci/hisilicon-pcie.txt | 52 ++++++++++++++++++++++
>> arch/arm64/boot/dts/hisilicon/hi3660.dtsi | 31 +++++++++++++
[...]
>> + "config": PCIe configuration space registers.
>> +- reset-gpio: perst assert/deassert gpio
>
> reset-gpios
> [songxiaowei] rest-gpio is used to signal 'pcie perst': high level refers to deassert
> and low level refers to assert. So, I think ' reset-gpio: creates perst assert/deassert signal '
> would be a better choice.
What I was saying is the name should be reset-gpios, not reset-gpio.
Rob
Powered by blists - more mailing lists