From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vivek Gautam Subject: Re: [PATCH V7 0/4] phy: USB and PCIe phy drivers for Qcom chipsets Date: Wed, 5 Apr 2017 20:54:20 +0530 Message-ID: References: <1491395558-16902-1-git-send-email-vivek.gautam@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Kishon Vijay Abraham I Cc: robh+dt , "linux-arm-kernel@lists.infradead.org" , linux-arm-msm@vger.kernel.org, "linux-kernel@vger.kernel.org" , Linux USB Mailing List , "devicetree@vger.kernel.org" , Mark Rutland , Stephen Boyd , Bjorn Andersson , Srinivas Kandagatla List-Id: linux-arm-msm@vger.kernel.org Hi Kishon, On Wed, Apr 5, 2017 at 7:08 PM, Kishon Vijay Abraham I wrote: > 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? Sure, I am on it. Thanks for pointing out. Regards Vivek > > 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 >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755453AbdDEPYq (ORCPT ); Wed, 5 Apr 2017 11:24:46 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:58138 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755077AbdDEPYX (ORCPT ); Wed, 5 Apr 2017 11:24:23 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org A5EB860708 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: References: <1491395558-16902-1-git-send-email-vivek.gautam@codeaurora.org> From: Vivek Gautam Date: Wed, 5 Apr 2017 20:54:20 +0530 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH V7 0/4] phy: USB and PCIe phy drivers for Qcom chipsets To: Kishon Vijay Abraham I Cc: "robh+dt" , "linux-arm-kernel@lists.infradead.org" , linux-arm-msm@vger.kernel.org, "linux-kernel@vger.kernel.org" , Linux USB Mailing List , "devicetree@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 Kishon, On Wed, Apr 5, 2017 at 7:08 PM, Kishon Vijay Abraham I wrote: > 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? Sure, I am on it. Thanks for pointing out. Regards Vivek > > 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 >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project From mboxrd@z Thu Jan 1 00:00:00 1970 From: vivek.gautam@codeaurora.org (Vivek Gautam) Date: Wed, 5 Apr 2017 20:54:20 +0530 Subject: [PATCH V7 0/4] phy: USB and PCIe phy drivers for Qcom chipsets In-Reply-To: References: <1491395558-16902-1-git-send-email-vivek.gautam@codeaurora.org> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Kishon, On Wed, Apr 5, 2017 at 7:08 PM, Kishon Vijay Abraham I wrote: > 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? Sure, I am on it. Thanks for pointing out. Regards Vivek > > 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 >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project