Re: [PATCH v3 1/2] USB: PHY: Make PHY driver selection possible by controller drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 14, 2013 at 12:51:41PM +0300, Roger Quadros wrote:
> Convert PHY Drivers from menuconfig to menu so that the PHY drivers
> can be explicitely selected by the controller drivers.
> 
> USB_PHY is no longer a user visible option. It is upto to the PHY
> drivers to select it if needed. This patch does so for the existing
> PHY drivers that use the USB_PHY library.
> 
> Doing so moves the USB_PHY and PHY driver selection problem from the
> end user to the PHY and controller driver developer.
> 
> e.g.
> 
> Earlier, a controller driver (e.g. EHCI_OMAP) that needs to select
> a PHY driver (e.g. NOP_PHY) couldn't do so because the PHY driver
> depended on USB_PHY. Making the controller driver depend on USB_PHY
> has a negative effect i.e. it becomes invisible to the user till
> USB_PHY is enabled. Most end users will not familiar with this.
> 
> With this patch, the end user just needs to select the controller driver
> needed for his/her platform without worrying about which PHY driver to
> select.
> 
> Also update USB_EHCI_MSM, USB_LPC32XX and USB_OMAP to not depend
> on USB_PHY any more. They can safely select the necessary PHY drivers.
> 
> CC: Pavankumar Kondeti <pkondeti@xxxxxxxxxxxxxx>
> CC: Roland Stigge <stigge@xxxxxxxxx>
> Signed-off-by: Roger Quadros <rogerq@xxxxxx>
> [USB_LPC32XX part] Acked-by: Roland Stigge <stigge@xxxxxxxxx>
> 
> Signed-off-by: Roger Quadros <rogerq@xxxxxx>

There is one problem (?) with this: phy-core.ko will never be a module.
If that's not a problem for anyone, I guess this is safe to apply. I'll
wait a little longer until merge window closes to give people some extra
time to shout.

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux