Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753295AbdDENk1 (ORCPT ); Wed, 5 Apr 2017 09:40:27 -0400 Received: from fllnx209.ext.ti.com ([198.47.19.16]:54407 "EHLO fllnx209.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754531AbdDENkM (ORCPT ); Wed, 5 Apr 2017 09:40:12 -0400 Subject: Re: [PATCH V7 0/4] phy: USB and PCIe phy drivers for Qcom chipsets To: Vivek Gautam , References: <1491395558-16902-1-git-send-email-vivek.gautam@codeaurora.org> CC: , , , , , , , , From: Kishon Vijay Abraham I Message-ID: Date: Wed, 5 Apr 2017 19:08:27 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <1491395558-16902-1-git-send-email-vivek.gautam@codeaurora.org> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4037 Lines: 92 Hi Vivek, On Wednesday 05 April 2017 06:02 PM, Vivek Gautam wrote: > This patch series adds couple of PHY drivers for Qualcomm chipsets. > a) qcom-qusb2 phy driver: that provides High Speed USB functionality. > b) qcom-qmp phy driver: that is a combo phy providing support for > USB3, PCIe, UFS and few other controllers. > > The patches are based on next branch of linux-phy tree. > > These patches have been tested on Dragon board db820c hardware with > required set of dt patches and the patches to get rpm up on msm8996. > Couple of other patches [1, 2] fixing DMA config for XHCI are also > pulled in for testing. > A branch based on torvald's master is available in github [3]. I get a bunch of checkpatch errors/warnings when I run checkpatch with --strict option. Those look simple enough to be fixed. Can you respin your series fixing those? Thanks Kishon > > Changes since v6: > - Rebased on phy/next and *not* including phy grouping series[4]. > - qusb2-phy: addressed Stephen's comment. > - Dropped pm8994_s2 corner regulator from QUSB2 phy bindings. > - qmp-phy: none on functionality side. > > Changes since v5: > - Addressed review comments from Bjorn: > - Removed instances of readl/wirtel_relaxed calls from the drivers. > Instead, using simple readl/writel. Inserting a readl after a writel > to ensure the write is through to the device. > - Replaced regulator handling with regulator_bulk_** apis. This helps > in cutting down a lot of regulator handling code. > - Fixed minor return statements. > > Changes since v4: > - Addressed comment to add child nodes for qmp phy driver. Each phy lane > now has a separate child node under the main qmp node. > - Modified the clock and reset initialization and enable methods. > Different phys - pcie, usb and later ufs, have varying number of clocks > and resets that are mandatory. So adding provision for clocks and reset > lists helps in requesting all mandatory resources for individual phys > and handle their failure cases accordingly. > > Changes since v3: > - Addressed review comments given by Rob and Stephen for qusb2 phy > and qmp phy bindings respectively. > - Addressed review comments given by Stephen and Bjorn for qmp phy driver. > > Changes since v2: > - Addressed review comments given by Rob and Stephen for bindings. > - Addressed the review comments given by Stephen for the qusb2 and qmp > phy drivers. > > Changes since v1: > - Moved device tree binding documentation to separate patches, as suggested > by Rob. > - Addressed review comment regarding qfprom accesses by qusb2 phy driver, > given by Rob. > - Addressed review comments from Kishon. > - Addressed review comments from Srinivas for QMP phy driver. > - Addressed kbuild warning. > > Please see individual patches for detailed changelogs. > > [1] https://patchwork.kernel.org/patch/9567767/ > [2] https://patchwork.kernel.org/patch/9567779/ > [3] https://github.com/vivekgautam1/linux/tree/linux-v4.11-rc5-qmp-phy-db820c > [4] https://lkml.org/lkml/2017/3/20/407 > > Vivek Gautam (4): > dt-bindings: phy: Add support for QUSB2 phy > phy: qcom-qusb2: New driver for QUSB2 PHY on Qcom chips > dt-bindings: phy: Add support for QMP phy > phy: qcom-qmp: new qmp phy driver for qcom-chipsets > > .../devicetree/bindings/phy/qcom-qmp-phy.txt | 106 ++ > .../devicetree/bindings/phy/qcom-qusb2-phy.txt | 43 + > drivers/phy/Kconfig | 18 + > drivers/phy/Makefile | 2 + > drivers/phy/phy-qcom-qmp.c | 1153 ++++++++++++++++++++ > drivers/phy/phy-qcom-qusb2.c | 491 +++++++++ > 6 files changed, 1813 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/qcom-qmp-phy.txt > create mode 100644 Documentation/devicetree/bindings/phy/qcom-qusb2-phy.txt > create mode 100644 drivers/phy/phy-qcom-qmp.c > create mode 100644 drivers/phy/phy-qcom-qusb2.c >