From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roger Quadros Subject: Re: [PATCH] USB: ehci-omap: Select USB_PHY Date: Thu, 11 Apr 2013 13:51:16 +0300 Message-ID: <516695A4.2050804@ti.com> References: <5166856C.3080906@ti.com> <20130411100435.GB14394@arwen.pp.htv.fi> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130411100435.GB14394@arwen.pp.htv.fi> Sender: linux-kernel-owner@vger.kernel.org To: balbi@ti.com Cc: Greg KH , Alan Stern , Linux USB Mailing List , Kernel development list , "linux-omap@vger.kernel.org" List-Id: linux-omap@vger.kernel.org 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. cheers, -roger