Re: [PATCH 1/3] phy: core: Add devm_of_phy_get_by_index to phy-core

[Kishon Vijay Abraham I]

On Friday 27 March 2015 04:37 AM, Dmitry Torokhov wrote:
> On Wed, Mar 25, 2015 at 05:04:32PM -0700, Arun Ramamurthy wrote:
> > On 15-03-25 03:03 PM, Kishon Vijay Abraham I wrote:
> > > Hi,
> > >
> > > On Saturday 21 March 2015 02:59 AM, Arun Ramamurthy wrote:
> > > > On 15-03-20 02:26 PM, Dmitry Torokhov wrote:
> > > > > Hi Arun,
> > > > >
> > > > > On Fri, Mar 20, 2015 at 02:07:08PM -0700, Arun Ramamurthy wrote:
> > > > > > Adding devm_of_phy_get_by_index to get phys by supplying an index
> > > > > > and not a phy name when multiple phys are declared
> > >
> > > I think a bit more explanation on why get_by_index is needed here.
> > Thanks Kison. Can you be more specific? I am unsure of what more I
> > can explain here.
>
> We just need to mention that some generic drivers, such as ehci, may use
> multiple phys, and for such drivers referencing phy(s) by name(s) does
> not make sense. Instead of inventing elaborate naming schemes and
> producing custom code to iterate over names, such drivers are better of
> using nameless phy bindings and using this newly introduced API to
> iterate through them.

+1

-Kishon
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/