Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752930AbaD1EIu (ORCPT ); Mon, 28 Apr 2014 00:08:50 -0400 Received: from mail-qa0-f54.google.com ([209.85.216.54]:57558 "EHLO mail-qa0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751872AbaD1EIs (ORCPT ); Mon, 28 Apr 2014 00:08:48 -0400 MIME-Version: 1.0 In-Reply-To: References: <1398424873-20257-1-git-send-email-gautam.vivek@samsung.com> Date: Mon, 28 Apr 2014 09:38:47 +0530 X-Google-Sender-Auth: j5Te8Y92IQqgwe6ygDvfOBAaBos Message-ID: Subject: Re: [PATCH v8 2/2] usb: ehci-exynos: Change to use phy provided by the generic phy framework From: Vivek Gautam To: Alan Stern Cc: Linux USB Mailing List , "linux-samsung-soc@vger.kernel.org" , "devicetree@vger.kernel.org" , linux-doc@vger.kernel.org, "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Greg KH , Felipe Balbi , Kukjin Kim , Kamil Debski , Jingoo Han 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, On Fri, Apr 25, 2014 at 8:11 PM, Alan Stern wrote: > On Fri, 25 Apr 2014, Vivek Gautam wrote: > >> From: Kamil Debski >> >> Add the phy provider, supplied by new Exynos-usb2phy using >> Generic phy framework. >> Keeping the support for older USB phy intact right now, in order >> to prevent any functionality break in absence of relevant >> device tree side change for ehci-exynos. >> Once we move to new phy in the device nodes for ehci, we can >> remove the support for older phys. >> >> Signed-off-by: Kamil Debski >> [gautam.vivek@samsung.com: Addressed review comments from mailing list] >> [gautam.vivek@samsung.com: Kept the code for old usb-phy, and just >> added support for new exynos5-usb2phy in generic phy framework] >> [gautam.vivek@samsung.com: Edited the commit message] >> Signed-off-by: Vivek Gautam > >> +static int exynos_ehci_phyg_off(struct phy *phy[]) >> +{ >> + int i; >> + int ret = 0; >> + >> + for (i = 0; ret == 0 && i < PHY_NUMBER; i++) >> + if (phy[i]) >> + ret = phy_power_off(phy[i]); >> + >> + return ret; >> +} > > Same comment as in the OHCI driver about ret. Ok, will change this. > >> @@ -175,6 +269,7 @@ skip_phy: >> fail_add_hcd: >> if (exynos_ehci->phy) >> usb_phy_shutdown(exynos_ehci->phy); >> + exynos_ehci_phyg_off(exynos_ehci->phy_g); >> fail_io: >> clk_disable_unprepare(exynos_ehci->clk); >> fail_clk: >> @@ -195,6 +290,8 @@ static int exynos_ehci_remove(struct platform_device *pdev) >> if (exynos_ehci->phy) >> usb_phy_shutdown(exynos_ehci->phy); >> >> + exynos_ehci_phyg_off(exynos_ehci->phy_g); >> + > > In both these places, you need to test exynos_ehci->phyg before calling > exynos_ehci_phyg_off(). > > Maybe it would help to add exynos_ehci_phy_enable/disable routines, > like in the OHCI driver. Will move these statements as a part of enable()/disable() routines, put necessary check there. -- Best Regards Vivek Gautam Samsung R&D Institute, Bangalore India -- 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/