From: Arnd Bergmann <arnd@arndb.de>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Arun Ramamurthy <arun.ramamurthy@broadcom.com>,
        Kishon Vijay Abraham I <kishon@ti.com>,
        Tony Prisk <linux@prisktech.co.nz>,
        Alan Stern <stern@rowland.harvard.edu>, Felipe Balbi <balbi@ti.com>,
        Mathias Nyman <mathias.nyman@linux.intel.com>,
        Paul Bolle <pebolle@tiscali.nl>,
        Thomas Pugliese <thomas.pugliese@gmail.com>,
        Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
        David Mosberger <davidm@egauge.net>,
        Peter Griffin <peter.griffin@linaro.org>,
        Gregory CLEMENT <gregory.clement@free-electrons.com>,
        Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
        Kevin Hao <haokexin@gmail.com>, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-usb@vger.kernel.org,
        Dmitry Torokhov <dtor@google.com>, Anatol Pomazau <anatol@google.com>,
        Jonathan Richardson <jonathar@broadcom.com>,
        Scott Branden <sbranden@broadcom.com>, Ray Jui <rjui@broadcom.com>,
        bcm-kernel-feedback-list@broadcom.com
Subject: Re: [PATCHv2 2/3] usb: ehci-platform: Use devm_of_phy_get_by_index
Date: Tue, 14 Apr 2015 13:33:08 +0200
Message-ID: <6483716.157BQAaRqF@wuerfel>
User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; )
In-Reply-To: <20150414111934.GA27836@kroah.com>
References: <1428963047-23666-1-git-send-email-arun.ramamurthy@broadcom.com> <1428963047-23666-3-git-send-email-arun.ramamurthy@broadcom.com> <20150414111934.GA27836@kroah.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1865
Lines: 47

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@broadcom.com>
> > Reviewed-by: Ray Jui <rjui@broadcom.com>
> > Reviewed-by: Scott Branden <sbranden@broadcom.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.
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-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/