From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754024AbdA0FQS (ORCPT ); Fri, 27 Jan 2017 00:16:18 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:47934 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753758AbdA0FQQ (ORCPT ); Fri, 27 Jan 2017 00:16:16 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org ED70760A06 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=vivek.gautam@codeaurora.org Subject: Re: [PATCH v4 3/4] dt-bindings: phy: Add support for QMP phy To: Stephen Boyd References: <1484045519-19030-1-git-send-email-vivek.gautam@codeaurora.org> <1484045519-19030-4-git-send-email-vivek.gautam@codeaurora.org> <587C88FE.2040900@ti.com> <50612693-5345-55da-8207-8c5e721fb68a@codeaurora.org> <20170118182223.GP10531@minitux> <20170119004028.GA4857@codeaurora.org> <5cee25c5-434b-6e73-301e-3942dedd16fa@codeaurora.org> <20170119214258.GD7829@codeaurora.org> <58871F6D.1090206@ti.com> <20170126234355.GF8801@codeaurora.org> Cc: Kishon Vijay Abraham I , Bjorn Andersson , robh+dt , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Mark Rutland , Srinivas Kandagatla , linux-arm-msm@vger.kernel.org From: Vivek Gautam Message-ID: Date: Fri, 27 Jan 2017 10:46:07 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20170126234355.GF8801@codeaurora.org> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/27/2017 05:13 AM, Stephen Boyd wrote: > On 01/24, Vivek Gautam wrote: >> Below is one binding that works for me. >> -------------------- >> phy@34000 { >> compatible = "qcom,msm8996-qmp-pcie-phy"; >> reg = <0x034000 0x488>; >> #clock-cells = <1>; >> #address-cells = <1>; >> #size-cells = <1>; >> ranges; >> >> clocks = <&gcc GCC_PCIE_PHY_AUX_CLK>, >> <&gcc GCC_PCIE_PHY_CFG_AHB_CLK>, >> <&gcc GCC_PCIE_CLKREF_CLK>; >> clock-names = "aux", "cfg_ahb", "ref"; >> >> vdda-phy-supply = <&pm8994_l28>; >> vdda-pll-supply = <&pm8994_l12>; >> >> resets = <&gcc GCC_PCIE_PHY_BCR>, >> <&gcc GCC_PCIE_PHY_COM_BCR>, >> <&gcc GCC_PCIE_PHY_COM_NOCSR_BCR>; >> reset-names = "phy", "common", "cfg"; >> >> pciephy_p0: port@0 { > The unit address '@0' should be replaced with something from the > reg properties. Sure, will take care of this. > > > Also 'port' and 'ports' are almost keywords in DT now with the > graph binding so we need to be careful when using them. From "./Documentation/devicetree/bindings/graph.txt" - "The device tree graph bindings described herein abstract more complex devices that can have multiple specifiable ports, each of which can be linked to one or more ports of other devices." So, this means we use 'port', 'ports' and 'endpoint' for devices whose one or more ports is connected to other device's one or more ports. I can use 'lane' for the node name here. > >> reg = <0x035000 0x130>, >> <0x035200 0x200>, >> <0x035400 0x1dc>; >> #phy-cells = <0>; >> >> clocks = <&gcc GCC_PCIE_0_PIPE_CLK>; >> clock-names = "pipe0"; >> resets = <&gcc GCC_PCIE_0_PHY_BCR>; >> reset-names = "lane0"; >> }; >> >> pciephy_p1: port@1 { >> reg = <0x036000 0x130>, >> <0x036200 0x200>, >> <0x036400 0x1dc>; >> #phy-cells = <0>; >> >> clocks = <&gcc GCC_PCIE_1_PIPE_CLK>; >> clock-names = "pipe1"; >> resets = <&gcc GCC_PCIE_1_PHY_BCR>; >> reset-names = "lane1"; >> }; >> >> pciephy_p2: port@2 { >> reg = <0x037000 0x130>, >> <0x037200 0x200>, >> <0x037400 0x1dc>; >> #phy-cells = <0>; >> >> clocks = <&gcc GCC_PCIE_2_PIPE_CLK>; >> clock-names = "pipe2"; >> resets = <&gcc GCC_PCIE_2_PHY_BCR>; >> reset-names = "lane2"; >> }; >> }; >> -------------------- >> >> let me know if this looks okay. >> >> > What's the plan for non-pcie qmp phy binding? In that case we > don't have ports, so it gets folded into one node? > The non-pcie qmp phys still have one lane, that provides tx/rx. I am of the opinion that we don't have two different ways to create phys in the driver, and keep one port/lane for such phys in dt. Regards Vivek -- The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project