From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vivek Gautam Subject: Re: [PATCH v3 3/4] dt-bindings: phy: Add support for QMP phy Date: Thu, 29 Dec 2016 10:35:11 +0530 Message-ID: References: <1482253431-23160-1-git-send-email-vivek.gautam@codeaurora.org> <1482253431-23160-4-git-send-email-vivek.gautam@codeaurora.org> <20161228230412.GC17126@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <20161228230412.GC17126-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Boyd Cc: robh+dt , kishon , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Mark Rutland , Srinivas Kandagatla , linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On Thu, Dec 29, 2016 at 4:34 AM, Stephen Boyd wrote: > On 12/20, Vivek Gautam wrote: >> + >> +Example: >> + pcie_phy: phy@34000 { >> + compatible = "qcom,msm8996-qmp-pcie-phy"; >> + reg = <0x034000 0x48f>, >> + <0x035000 0x5bf>, >> + <0x036000 0x5bf>, >> + <0x037000 0x5bf>; >> + /* tx, rx, pcs */ >> + lane-offsets = <0x0 0x200 0x400>; >> + #phy-cells = <1>; >> + >> + clocks = <&gcc GCC_PCIE_PHY_AUX_CLK>, >> + <&gcc GCC_PCIE_PHY_CFG_AHB_CLK>, >> + <&gcc GCC_PCIE_CLKREF_CLK>, >> + <&gcc GCC_PCIE_0_PIPE_CLK>, >> + <&gcc GCC_PCIE_1_PIPE_CLK>, >> + <&gcc GCC_PCIE_2_PIPE_CLK>; >> + clock-names = "aux", "cfg_ahb", "ref", >> + "pipe0", "pipe1", "pipe2"; > > Can we add a #clock-cells = <0> or <1> here given that this is a > clk provider? We may want to express the clk circular dependency > between this phy node and GCC via the clocks property at some > point instead of doing it implicitly via strings in C code. Sure, will add #clock-cells = <1>. Although phys like USB and PIPE currently have just the pipe_clk being controlled by gcc, the UFS phy has tx/rx symbol clocks that are controlled by gcc but are generated by phy the same way as pipe_clk. So, i guess #clock-cells = <1 > makes sense. Thanks Vivek -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html