On Tuesday 14 April 2015 13:19:34 Greg Kroah-Hartman wrote: > On Mon, Apr 13, 2015 at 03:10:46PM -0700, Arun Ramamurthy wrote: > > Getting phys by index instead of phy names so that we do > > not have to create a naming scheme when multiple phys > > are present > > > > Signed-off-by: Arun Ramamurthy <arun.ramamurthy@xxxxxxxxxxxx> > > Reviewed-by: Ray Jui <rjui@xxxxxxxxxxxx> > > Reviewed-by: Scott Branden <sbranden@xxxxxxxxxxxx> > > --- > > drivers/usb/host/Kconfig | 1 + > > drivers/usb/host/ehci-platform.c | 70 ++++++++++++++-------------------------- > > 2 files changed, 26 insertions(+), 45 deletions(-) > > > > diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig > > index 5ad60e4..563f22d 100644 > > --- a/drivers/usb/host/Kconfig > > +++ b/drivers/usb/host/Kconfig > > @@ -284,6 +284,7 @@ config USB_EHCI_ATH79 > > > > config USB_EHCI_HCD_PLATFORM > > tristate "Generic EHCI driver for a platform device" > > + select GENERIC_PHY > > Configs should never select if at all possible. > This is true, but all other drivers do the same for GENERIC_PHY at the moment. If this one gets changed, we should probably apply the same solution to all current users and fix them consistently. We can do one of these two: a) make sure that the framework has 'static inline' stubs that let you build all drivers using it when the framework itself is disabled. b) change the drivers using it to 'depends on', and make GENERIC_PHY itself a hidden option without a Kconfig prompt. Either solution is probably simple enough that it can be done as part of this series. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
