mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-06 02:50:43 +00:00
usb: chipidea: Grab the (legacy) USB PHY by phandle first
According to the chipidea driver bindings, the USB PHY is specified via the "phys" phandle node. However, this only takes effect for USB PHYs that use the common PHY framework. For legacy USB PHYs, a simple lookup based on the USB PHY type is done instead. This does not play out well when more than one USB PHY is registered, since the first registered PHY matching the type will always be returned regardless of what the driver was bound to. Fix this by looking up the PHY based on the "phys" phandle node. Although generic PHYs are rather matched by their "phys-name" and not the "phys" phandle directly, there is no helper for similar lookup on legacy PHYs and it's probably not worth the effort to add it. When no legacy USB PHY is found by phandle, fallback to grabbing any registered USB2 PHY. This ensures backward compatibility if some users were actually relying on this mechanism. Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Signed-off-by: Peter Chen <peter.chen@nxp.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
5f0632c467
commit
68ef236274
1 changed files with 8 additions and 1 deletions
|
@ -954,8 +954,15 @@ static int ci_hdrc_probe(struct platform_device *pdev)
|
|||
} else if (ci->platdata->usb_phy) {
|
||||
ci->usb_phy = ci->platdata->usb_phy;
|
||||
} else {
|
||||
ci->usb_phy = devm_usb_get_phy_by_phandle(dev->parent, "phys",
|
||||
0);
|
||||
ci->phy = devm_phy_get(dev->parent, "usb-phy");
|
||||
ci->usb_phy = devm_usb_get_phy(dev->parent, USB_PHY_TYPE_USB2);
|
||||
|
||||
/* Fallback to grabbing any registered USB2 PHY */
|
||||
if (IS_ERR(ci->usb_phy) &&
|
||||
PTR_ERR(ci->usb_phy) != -EPROBE_DEFER)
|
||||
ci->usb_phy = devm_usb_get_phy(dev->parent,
|
||||
USB_PHY_TYPE_USB2);
|
||||
|
||||
/* if both generic PHY and USB PHY layers aren't enabled */
|
||||
if (PTR_ERR(ci->phy) == -ENOSYS &&
|
||||
|
|
Loading…
Reference in a new issue