From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753193AbdCTI1l (ORCPT ); Mon, 20 Mar 2017 04:27:41 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:42814 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752894AbdCTI0g (ORCPT ); Mon, 20 Mar 2017 04:26:36 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 9A6B5607F0 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 MIME-Version: 1.0 In-Reply-To: <20170315203456.pxroh7nir2iwll3o@rob-hp-laptop> References: <1489050441-3240-1-git-send-email-vivek.gautam@codeaurora.org> <1489050441-3240-4-git-send-email-vivek.gautam@codeaurora.org> <20170315203456.pxroh7nir2iwll3o@rob-hp-laptop> From: Vivek Gautam Date: Mon, 20 Mar 2017 13:56:33 +0530 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v5 3/4] dt-bindings: phy: Add support for QMP phy To: Rob Herring Cc: kishon , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Mark Rutland , Stephen Boyd , Bjorn Andersson , Srinivas Kandagatla Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, On Thu, Mar 16, 2017 at 2:04 AM, Rob Herring wrote: > On Thu, Mar 09, 2017 at 02:37:20PM +0530, Vivek Gautam wrote: >> Qualcomm chipsets have QMP phy controller that provides >> support to a number of controller, viz. PCIe, UFS, and USB. >> Adding dt binding information for the same. >> >> Signed-off-by: Vivek Gautam >> Cc: Rob Herring >> --- >> >> Hi Rob, >> I have removed your Acked-by tag because of the change in bindings. >> Please consider adding your Ack again if you are fine with these >> updated bindings. >> >> Changes since v4: >> - Added bindings for child nodes. Each phy lane is represented by child >> node with its own register space (for tx, rx and pcs blocks), and clocks >> and resets for power control facility. >> - Removed register space and lane offsets for tx, rx and pcs blocks from >> qmp phy node. >> - #phy-cells is now part of each child node and thus must be 0. >> - Added information on list of mandatory clocks and resets for each phy. >> >> Changes since v3: >> - Added #clock-cells = <1>, indicating that phy is a clock provider. >> >> Changes since v2: >> - Removed binding for "ref_clk_src" since we don't request this >> clock in the driver. > > Humm, sounds suspicious. You should describe clocks connected to the h/w > block, not what the driver uses (currently). The "ref_clk_src" clock is not really going to the phy block. After cross-checking the hardware documents, we decided to remove this. On the SoC, this is an external clock (outside of the clock controller) that comes to a clock pad that feeds few clock tx blocks. These blocks supply the reference clock to each IP blocks. Output of these tx blocks is what we refer to as "ref" clock in our bindings. > >> - Addressed s/ref_clk/ref. Don't need to add '_clk' suffix to clock names. >> - Using 'phy' for the node name. >> >> Changes since v1: >> - New patch, forked out of the original driver patch: >> "phy: qcom-qmp: new qmp phy driver for qcom-chipsets" >> - Added 'Acked-by' from Rob. >> - Updated bindings to include mem resource as a list of >> offset - length pair for serdes block and for each lane. >> - Added a new binding for 'lane-offsets' that contains offsets >> to tx, rx and pcs blocks from each lane base address. >> >> .../devicetree/bindings/phy/qcom-qmp-phy.txt | 106 +++++++++++++++++++++ >> 1 file changed, 106 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/qcom-qmp-phy.txt > > Acked-by: Rob Herring Thanks for reviewing. Best Regards Vivek -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project