lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 14 Apr 2015 14:37:37 +0200 From: Greg Kroah-Hartman <gregkh@...uxfoundation.org> To: Arnd Bergmann <arnd@...db.de> Cc: Arun Ramamurthy <arun.ramamurthy@...adcom.com>, Kishon Vijay Abraham I <kishon@...com>, Tony Prisk <linux@...sktech.co.nz>, Alan Stern <stern@...land.harvard.edu>, Felipe Balbi <balbi@...com>, Mathias Nyman <mathias.nyman@...ux.intel.com>, Paul Bolle <pebolle@...cali.nl>, Thomas Pugliese <thomas.pugliese@...il.com>, Srinivas Kandagatla <srinivas.kandagatla@...aro.org>, David Mosberger <davidm@...uge.net>, Peter Griffin <peter.griffin@...aro.org>, Gregory CLEMENT <gregory.clement@...e-electrons.com>, Laurent Pinchart <laurent.pinchart@...asonboard.com>, Kevin Hao <haokexin@...il.com>, linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, linux-usb@...r.kernel.org, Dmitry Torokhov <dtor@...gle.com>, Anatol Pomazau <anatol@...gle.com>, Jonathan Richardson <jonathar@...adcom.com>, Scott Branden <sbranden@...adcom.com>, Ray Jui <rjui@...adcom.com>, bcm-kernel-feedback-list@...adcom.com Subject: Re: [PATCHv2 2/3] usb: ehci-platform: Use devm_of_phy_get_by_index On Tue, Apr 14, 2015 at 01:33:08PM +0200, Arnd Bergmann wrote: > 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@...adcom.com> > > > Reviewed-by: Ray Jui <rjui@...adcom.com> > > > Reviewed-by: Scott Branden <sbranden@...adcom.com> > > > --- > > > 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. Yes, please do that. > b) change the drivers using it to 'depends on', and make GENERIC_PHY > itself a hidden option without a Kconfig prompt. Then how could GENERIC_PHY ever get set? > Either solution is probably simple enough that it can be done as > part of this series. That's fine with me, but please no more selects. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists