Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933918Ab3DKNSm (ORCPT ); Thu, 11 Apr 2013 09:18:42 -0400 Received: from bear.ext.ti.com ([192.94.94.41]:50120 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753419Ab3DKNSk (ORCPT ); Thu, 11 Apr 2013 09:18:40 -0400 Message-ID: <5166B829.1020304@ti.com> Date: Thu, 11 Apr 2013 16:18:33 +0300 From: Roger Quadros User-Agent: Mozilla/5.0 (X11; Linux i686; rv:16.0) Gecko/20121011 Thunderbird/16.0.1 MIME-Version: 1.0 To: CC: Greg KH , Alan Stern , Linux USB Mailing List , Kernel development list , "linux-omap@vger.kernel.org" Subject: Re: [PATCH] USB: ehci-omap: Select USB_PHY References: <5166856C.3080906@ti.com> <20130411100435.GB14394@arwen.pp.htv.fi> <516695A4.2050804@ti.com> <20130411105557.GE14394@arwen.pp.htv.fi> <5166AFD2.4070108@ti.com> In-Reply-To: <5166AFD2.4070108@ti.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2158 Lines: 64 On 04/11/2013 03:42 PM, Roger Quadros wrote: > On 04/11/2013 01:55 PM, Felipe Balbi wrote: >> Hi, >> >> On Thu, Apr 11, 2013 at 01:51:16PM +0300, Roger Quadros wrote: >>> On 04/11/2013 01:04 PM, Felipe Balbi wrote: >>>> Hi, >>>> >>>> On Thu, Apr 11, 2013 at 12:42:04PM +0300, Roger Quadros wrote: >>>>> Hi Greg, >>>>> >>>>> The following patch gets rid of Kbuild warnings when USB_EHCI_HCD_OMAP >>>>> is enabled. >>>>> >>>>> Patch is based on your usb-next branch and is needed for 3.10. >>>>> >>>>> From: Roger Quadros >>>>> Date: Thu, 11 Apr 2013 12:08:19 +0300 >>>>> Subject: [PATCH] USB: ehci-omap: Select USB_PHY >>>>> >>>>> As we need NOP_USB_XCEIV which depends on USB_PHY >>>>> we need to select USB_PHY as well. >>>>> >>>>> Gets rid of the below warnings when USB_EHCI_HCD_OMAP >>>>> is enabled. >>>>> >>>>> warning: (USB_EHCI_HCD_OMAP) selects NOP_USB_XCEIV which has unmet direct dependencies (USB_SUPPORT && USB_PHY) >>>>> warning: (USB_EHCI_HCD_OMAP) selects NOP_USB_XCEIV which has unmet direct dependencies (USB_SUPPORT && USB_PHY) >>>>> >>>>> Signed-off-by: Roger Quadros >>>> >>>> Ideally, however, we wouldn't select any PHY in particular as different >>>> boards might need a different PHY driver, even on OMAP ;-) >>>> >>> Right, but we need to select USB_PHY here as the driver uses >>> the USB_PHY APIs. >>> >>> The NOP_USB_XCEIV selection could be done by the board config. >> >> I would avoid 'select' completely and just update omap2plus_defconfig >> adding those two as modules. >> > > OK, makes sense. I will update the patch to remove "select NOP_USB_XCEIV". > One more issue to clarify. if USB_PHY is not enabled, then all phy_get() API's should return NULL and not -ENXIO as it does now. This way the drivers need not treat it as an error and all PHY ops can be NOPs. This will make it behave like other frameworks. e.g. clk. cheers, -roger. -- 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/