From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1032768AbeBNQis (ORCPT ); Wed, 14 Feb 2018 11:38:48 -0500 Received: from mail-wm0-f65.google.com ([74.125.82.65]:55088 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1032718AbeBNQio (ORCPT ); Wed, 14 Feb 2018 11:38:44 -0500 X-Google-Smtp-Source: AH8x225R7lmmLgGzm9YsuVUXereNRfeI8T2kSBRX3Izi/AIzjLH/lEzH8NZFpwudVYKzWUSMKbi/OA== Subject: Re: [PATCH v2] PCI: qcom: add missing supplies required for msm8996 To: Srinivas Kandagatla , Stanimir Varbanov , linux-pci@vger.kernel.org, bhelgaas@google.com Cc: linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, robh+dt@kernel.org, devicetree@vger.kernel.org References: <20180214120616.23428-1-srinivas.kandagatla@linaro.org> <79820f4c-5166-852f-1bc5-8cef2ff81a00@mm-sol.com> From: Stanimir Varbanov Message-ID: Date: Wed, 14 Feb 2018 18:38:39 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/14/2018 06:14 PM, Srinivas Kandagatla wrote: > > > On 14/02/18 16:00, Stanimir Varbanov wrote: >>>   .../devicetree/bindings/pci/qcom,pcie.txt          |  4 ++++ >>>   drivers/pci/dwc/pcie-qcom.c                        | 24 >>> ++++++++++++++++++++-- >>>   2 files changed, 26 insertions(+), 2 deletions(-) >>> >>> diff --git a/Documentation/devicetree/bindings/pci/qcom,pcie.txt >>> b/Documentation/devicetree/bindings/pci/qcom,pcie.txt >>> index 3c9d321b3d3b..1fd703bd73e0 100644 >>> --- a/Documentation/devicetree/bindings/pci/qcom,pcie.txt >>> +++ b/Documentation/devicetree/bindings/pci/qcom,pcie.txt >>> @@ -189,6 +189,10 @@ >>>       Value type: >>>       Definition: A phandle to the analog power supply for IC which >>> generates >>>               reference clock >>> +- vddpe-3v3-supply: >> shouldn't it be vddpe_3v3-supply to be aligned with the other supplies? >> > I noticed that, I had to do it intentionally > As dt properties should follow - instead of _ I don't agree with that. If the supply name is more than one word it should be separated by underscore ('git grep \-supply' in dts files). -- regards, Stan