From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dilip Kota Subject: Re: [PATCH v3 1/2] dt-bindings: PCI: intel: Add YAML schemas for the PCIe RC controller Date: Wed, 18 Sep 2019 14:48:57 +0800 Message-ID: References: <20190906091950.GJ2680@smile.fi.intel.com> <20190917183306.GA24684@bogus> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <20190917183306.GA24684@bogus> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Rob Herring , Andy Shevchenko Cc: Martin Blumenstingl , jingoohan1@gmail.com, gustavo.pimentel@synopsys.com, lorenzo.pieralisi@arm.com, linux-pci@vger.kernel.org, hch@infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, cheol.yong.kim@intel.com, chuanhua.lei@linux.intel.com, qi-ming.wu@intel.com List-Id: devicetree@vger.kernel.org Hi Rob, On 9/18/2019 2:33 AM, Rob Herring wrote: > On Fri, Sep 06, 2019 at 12:19:50PM +0300, Andy Shevchenko wrote: >> On Thu, Sep 05, 2019 at 10:31:29PM +0200, Martin Blumenstingl wrote: >>> On Wed, Sep 4, 2019 at 12:11 PM Dilip Kota wrote: >>>> + phy-names: >>>> + const: pciephy >>> the most popular choice in Documentation/devicetree/bindings/pci/ is "pcie-phy" >>> if Rob is happy with "pciephy" (which is already part of two other >>> bindings) then I'm happy with "pciephy" as well >> I'm not Rob, though I consider more practical to have a hyphenated variant. > *-names is kind of pointless when there is only one entry and 'foo' in > the values of 'foo-names' is redundant. Ok, i will change it to:     phy-name:         const: pcie Regards, Dilip > > Rob