On Tue, Apr 07, 2015 at 11:37:43AM +0530, Kishon Vijay Abraham I wrote:
> On Thursday 02 April 2015 07:58 AM, Brian Norris wrote:
> >On Tue, Mar 31, 2015 at 11:31:40AM +0530, Kishon Vijay Abraham I wrote:
> >>On Saturday 28 March 2015 05:58 AM, Brian Norris wrote:
> >>>On Thu, Mar 26, 2015 at 03:29:44AM +0530, Kishon Vijay Abraham I wrote:
> >>>>On Thursday 19 March 2015 06:53 AM, Brian Norris wrote:
> >>>>>+static struct phy *brcm_sata_phy_xlate(struct device *dev,
> >>>>>+ struct of_phandle_args *args)
> >>>>>+{
> >>>>>+ struct brcm_sata_phy *priv = dev_get_drvdata(dev);
> >>>>>+ int i = args->args[0];
> >>>>>+
> >>>>>+ if (i >= MAX_PORTS || !priv->phys[i].phy) {
> >>>>>+ dev_err(dev, "invalid phy: %d\n", i);
> >>>>>+ return ERR_PTR(-ENODEV);
> >>>>>+ }
> >>>>>+
> >>>>>+ return priv->phys[i].phy;
> >>>>>+}
> >>>>
> >>>>this xlate is not required at all if the controller device tree node has
> >>>>phandle to the phy node (sub node) instead of the phy provider device tree
> >>>>node.
> >>>
> >>>That doesn't match any convention I see in existing SATA phy bindings,
> >>>nor do I see how the existing of_phy_simple_xlate() would support this,
> >>>unless I instantiate a device for each port's PHY. If I adjust the
> >>>device tree as you suggest, and use of_phy_simple_xlate() instead of
> >>>this, of_phy_get() can't find the PHY provider, because the provider is
> >>>registered to the parent, not the subnode.
> >>
> >>The phy core should still be able to get the PHY provider.
> >>See this in of_phy_provider_lookup
> >> for_each_child_of_node(phy_provider->dev->of_node, child)
> >> if (child == node)
> >> return phy_provider;
> >
> >That just searches for children of the node. It doesn't walk parent
> >nodes.
>
> okay.. in your phy_create pass the np of the PHYs (sub-node pointer
> to phy provider).
Ah, I see. I completely passed over the 2nd parameter to phy_create()...
Thanks for the tip.
> >>Can you post your device tree node here?
> >
> >You mean patch 5?
> >
> >https://lkml.org/lkml/2015/3/18/937
> >
> >>>
> >>>Can you elaborate on your suggestion?
>
> Change the dt node to something like below..
[snip]
Yes, that worked. Thanks.
OK, I'll fix this up and send out v2 shortly.
Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
