Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751380Ab2H3FQo (ORCPT ); Thu, 30 Aug 2012 01:16:44 -0400 Received: from hqemgate03.nvidia.com ([216.228.121.140]:8611 "EHLO hqemgate03.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751003Ab2H3FQm convert rfc822-to-8bit (ORCPT ); Thu, 30 Aug 2012 01:16:42 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Wed, 29 Aug 2012 22:10:20 -0700 From: Venu Byravarasu To: Stephen Warren CC: "ccross@android.com" , "olof@lixom.net" , "linux@arm.linux.org.uk" , "stern@rowland.harvard.edu" , "gregkh@linuxfoundation.org" , "balbi@ti.com" , "linux-kernel@vger.kernel.org" , "linux-tegra@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-usb@vger.kernel.org" Date: Thu, 30 Aug 2012 10:46:35 +0530 Subject: RE: [PATCH] usb: tegra: moving phy driver into drivers directory Thread-Topic: [PATCH] usb: tegra: moving phy driver into drivers directory Thread-Index: Ac2GC/3mxs+l0Rh7TRWx3TkhiZJcaAAYoj9Q Message-ID: References: <1346146338-4996-1-git-send-email-vbyravarasu@nvidia.com> <503CD0A0.7080300@wwwdotorg.org> <503E51C9.7000606@wwwdotorg.org> In-Reply-To: <503E51C9.7000606@wwwdotorg.org> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US MIME-Version: 1.0 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4120 Lines: 102 > -----Original Message----- > From: linux-tegra-owner@vger.kernel.org [mailto:linux-tegra- > owner@vger.kernel.org] On Behalf Of Stephen Warren > Sent: Wednesday, August 29, 2012 11:01 PM > To: Venu Byravarasu > Cc: ccross@android.com; olof@lixom.net; linux@arm.linux.org.uk; > stern@rowland.harvard.edu; gregkh@linuxfoundation.org; balbi@ti.com; > linux-kernel@vger.kernel.org; linux-tegra@vger.kernel.org; linux-arm- > kernel@lists.infradead.org; linux-usb@vger.kernel.org > Subject: Re: [PATCH] usb: tegra: moving phy driver into drivers directory > > On 08/28/12 22:17, Venu Byravarasu wrote: > > Stephen Warren wrote at Tuesday, August 28, 2012 7:37 PM: > >> On 08/28/2012 02:32 AM, Venu Byravarasu wrote: > >>> In order to keep up with the USB driver files organization, > >>> moving USB phy driver from mach-tegra to drivers/USB directory. > >>> > >>> Signed-off-by: Venu Byravarasu > >> > >>> diff --git a/arch/arm/mach-tegra/devices.c b/arch/arm/mach- > >> tegra/devices.c > >> > >>> -struct tegra_ulpi_config tegra_ehci2_ulpi_phy_config = { > >>> - .reset_gpio = -1, > >>> - .clk = "cdev2", > >>> -}; > >>> - > >>> struct tegra_ehci_platform_data tegra_ehci1_pdata = { > >>> .operating_mode = TEGRA_USB_OTG, > >>> .power_down_on_bus_suspend = 1, > >>> @@ -450,7 +444,7 @@ struct tegra_ehci_platform_data > tegra_ehci1_pdata > >> = { > >>> }; > >>> > >>> struct tegra_ehci_platform_data tegra_ehci2_pdata = { > >>> - .phy_config = &tegra_ehci2_ulpi_phy_config, > >>> + .phy_config = NULL, > >> > >> The PHY driver checks that field isn't NULL, and fails if it is: > >> > >>> struct tegra_usb_phy *tegra_usb_phy_open(struct device *dev, int > >> instance, > >>> void __iomem *regs, void *config, enum tegra_usb_phy_mode > >> phy_mode) > >>> { > >> ... > >>> phy->config = config; > >>> phy->mode = phy_mode; > >>> > >>> if (!phy->config) { > >>> if (phy_is_ulpi(phy)) { > >>> pr_err("%s: ulpi phy configuration missing", __func__); > >>> err = -EINVAL; > >>> goto err0; > >> > >> So, this change will completely break ULPI support, which currently > >> works fine. So, NAK. > > > > My initial plan was to add support for phy interfaces one by one. > > As part of that thought of UTMI only support at first and then add > > ULPI and HSIC in next patches. > > However as you were mentioning that it is not correct way, will > > push ULPI & UTMI support at once in next patches. > > But with the existing code, both ULPI and UTMI work. This patch breaks > something that already works. > > >> I also plan on deleting devices.[ch] in kernel 3.7, and moving the USB > >> platform data into board-dt-tegra20.c, since that's the only place it's > >> used right now. So, this patch would conflict with that rather badly. I > >> just posted the patches for that to the linux-tegra mailing list last > >> night. Do you have better proposals for that? Perhaps usb_phy.c should > >> set phy->config to &ulpi_default in a similar fashion to how it works > >> for UTMI; that would remove some of the coupling between the changes. > >> > >> BTW, in your response to Felipe, you said... > >> > >>> Thanks Felipe for your comments. > >>> Created a patch to separate out phy related stuff to phy.h with you as a > >> reviewer. > >>> Plz let me know your comments. > >> > >> ... where is that patch? > > > > Plz see https://lkml.org/lkml/2012/8/28/58 > > Doesn't that link point at the patch I replied to? Sorry, by mistake I sent the wrong one. Here is the correct one: https://lkml.org/lkml/2012/8/29/40 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-tegra" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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/