Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757652AbbEVSHy (ORCPT ); Fri, 22 May 2015 14:07:54 -0400 Received: from mail-qk0-f177.google.com ([209.85.220.177]:34105 "EHLO mail-qk0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161055AbbEVSG7 (ORCPT ); Fri, 22 May 2015 14:06:59 -0400 MIME-Version: 1.0 In-Reply-To: <555F5885.70105@mm-sol.com> References: <1430743338-10441-1-git-send-email-svarbanov@mm-sol.com> <1430743338-10441-3-git-send-email-svarbanov@mm-sol.com> <554783CD.50708@ti.com> <55478F1A.3000401@mm-sol.com> <20150519224143.GS31666@google.com> <555C8743.6020708@ti.com> <555F5885.70105@mm-sol.com> From: Bjorn Helgaas Date: Fri, 22 May 2015 13:06:37 -0500 Message-ID: Subject: Re: [PATCH v2 2/5] phy: qcom: Add Qualcomm PCIe PHY To: Stanimir Varbanov Cc: Kishon Vijay Abraham I , Rob Herring , Kumar Gala , Mark Rutland , Grant Likely , Russell King , Arnd Bergmann , linux-arm-msm , "linux-kernel@vger.kernel.org" , linux-arm , "devicetree@vger.kernel.org" , "linux-pci@vger.kernel.org" , Mathieu Olivari , Srinivas Kandagatla , Jingoo Han , Pratyush Anand Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2842 Lines: 77 [+cc Jingoo, Pratyush] On Fri, May 22, 2015 at 11:25 AM, Stanimir Varbanov wrote: > On 05/20/2015 04:23 PM, Bjorn Helgaas wrote: >> On Wed, May 20, 2015 at 8:08 AM, Kishon Vijay Abraham I wrote: >>> Hi Bjorn, >>> >>> >>> On Wednesday 20 May 2015 04:11 AM, Bjorn Helgaas wrote: >>>> >>>> On Mon, May 04, 2015 at 06:24:10PM +0300, Stanimir Varbanov wrote: >>>>> >>>>> On 05/04/2015 05:35 PM, Kishon Vijay Abraham I wrote: >>>>>> >>>>>> Hi, >>>>>> >>>>>> On Monday 04 May 2015 06:12 PM, Stanimir Varbanov wrote: >>>>>>> >>>>>>> Add a PCIe PHY driver used by PCIe host controller driver >>>>>>> on Qualcomm SoCs like Snapdragon 805. >>>>>>> >>>>>>> Signed-off-by: Stanimir Varbanov >>>>>>> --- >>>>>>> drivers/phy/Kconfig | 9 ++ >>>>>>> drivers/phy/Makefile | 1 + >>>>>>> drivers/phy/phy-qcom-pcie.c | 291 >>>>>>> +++++++++++++++++++++++++++++++++++++++++++ >>>>>> >>>>>> >>>>>> Why do you need a new PHY driver for this? Why not use the existing QCOM >>>>>> PHY driver. I can see the registers used here in >>>>>> phy-qcom-ufs-qmp-14nm.h? >>>>> >>>>> >>>>> I agree that on first glance there are similarities, but I'm not sure >>>>> does the PHYs are the same IP blocks. Or at least they are different >>>>> revisions which have too many differences. So trying to combine them >>>>> will lead to more code than now. >>>>> >>>>> Either way I will try to understand how many the differences are. >>>> >>>> >>>> Ping, where are we with this? Should I wait for something else, or are >>>> you >>>> convinced there's enough difference to warrant a new PHY driver, Kishon? >>> >>> >>> I'd like to wait to see if Stanimir can use existing driver instead of >>> creating a new driver. >> >> OK, I'm waiting for a v3 with either tweaks to the QCOM PHY driver, or >> stronger justification as to why that's impossible. It's hard to >> quantify statements like "there are similarities, but there are too >> many differences," so you might have to actually attempt a patch for >> the QCOM code and we can see how ugly that would turn out to be. > > Kishon, > > I'm still checking the differences, sorry for the delay. > > Bjorn, is the pcie driver looks good enough to be merged? I haven't looked at that (I was waiting until I could deal with the whole series). But it sounds like it would be useful to merge patches 3-5 by themselves? I'll take a look at those. It would be good to get Arnd and Rob to take a look, especially at the DT parts. And Jingoo and Pratyush since you use the DesignWare core. Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/