Commit a3a47548 authored by Paul Kocialkowski's avatar Paul Kocialkowski Committed by Greg Kroah-Hartman
Browse files

usb: chipidea: Refactor USB PHY selection and keep a single PHY 



Refactor the code in charge of looking up the USB PHY when no platdata
is provided. Attempt to get a generic USB PHY first, then look for a
legacy USB PHY through device-tree and finally get any registered PHY
with the correct type.

This way, only a single USB PHY is obtained and the flow is easier to
understand and follow.

All error pointers (except for EPROBE_DEFER) are considered as PHY
not found.

Signed-off-by: default avatarPaul Kocialkowski <paul.kocialkowski@bootlin.com>
Signed-off-by: default avatarPeter Chen <peter.chen@nxp.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 68ef2362

drivers/usb/chipidea/core.c

+32 −17
Original line number Diff line number Diff line
@@ -954,33 +954,48 @@ 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);

		/* Look for a generic PHY first */

		ci->phy = devm_phy_get(dev->parent, "usb-phy");



		/* 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 &&

				PTR_ERR(ci->usb_phy) == -ENXIO) {

			ret = -ENXIO;

		if (PTR_ERR(ci->phy) == -EPROBE_DEFER) {

			ret = -EPROBE_DEFER;

			goto ulpi_exit;

		} else if (IS_ERR(ci->phy)) {

			ci->phy = NULL;

		}



		if (IS_ERR(ci->phy) && IS_ERR(ci->usb_phy)) {

		/* Look for a legacy USB PHY from device-tree next */

		if (!ci->phy) {

			ci->usb_phy = devm_usb_get_phy_by_phandle(dev->parent,

								  "phys", 0);



			if (PTR_ERR(ci->usb_phy) == -EPROBE_DEFER) {

				ret = -EPROBE_DEFER;

				goto ulpi_exit;

			} else if (IS_ERR(ci->usb_phy)) {

				ci->usb_phy = NULL;

			}

		}



		if (IS_ERR(ci->phy))

			ci->phy = NULL;

		else if (IS_ERR(ci->usb_phy))

		/* Look for any registered legacy USB PHY as last resort */

		if (!ci->phy && !ci->usb_phy) {

			ci->usb_phy = devm_usb_get_phy(dev->parent,

						       USB_PHY_TYPE_USB2);



			if (PTR_ERR(ci->usb_phy) == -EPROBE_DEFER) {

				ret = -EPROBE_DEFER;

				goto ulpi_exit;

			} else if (IS_ERR(ci->usb_phy)) {

				ci->usb_phy = NULL;

			}

		}



		/* No USB PHY was found in the end */

		if (!ci->phy && !ci->usb_phy) {

			ret = -ENXIO;

			goto ulpi_exit;

		}

	}



	ret = ci_usb_phy_init(ci);

	if (ret) {

CRA Git | Maintained and supported by SUSTech CRA and CCSE