Re: [PATCH 8/9] usb: phy: rcar-gen2: register using usb_add_phy_dev()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/20/2013 09:38 PM, Ulrich Hecht wrote:
Allows binding of PCI USB host controllers to this phy.

Signed-off-by: Ulrich Hecht <ulrich.hecht@xxxxxxxxx>
---
  drivers/usb/phy/phy-rcar-gen2-usb.c | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/usb/phy/phy-rcar-gen2-usb.c b/drivers/usb/phy/phy-rcar-gen2-usb.c
index a99a695..25fdd91 100644
--- a/drivers/usb/phy/phy-rcar-gen2-usb.c
+++ b/drivers/usb/phy/phy-rcar-gen2-usb.c
@@ -212,8 +212,9 @@ static int rcar_gen2_usb_phy_probe(struct platform_device *pdev)
  	priv->phy.init = rcar_gen2_usb_phy_init;
  	priv->phy.shutdown = rcar_gen2_usb_phy_shutdown;
  	priv->phy.set_suspend = rcar_gen2_usb_phy_set_suspend;
+	priv->phy.type = USB_PHY_TYPE_USB2;

The above line is not really needed.
I think we should switch to usb_bind_phy and bind USB phy to the renesas_usbhs device instead.
In this case usbhs device probe will be deferred unless usb phy is available.


-	retval = usb_add_phy(&priv->phy, USB_PHY_TYPE_USB2);
+	retval = usb_add_phy_dev(&priv->phy);
  	if (retval < 0) {
  		dev_err(dev, "Failed to add USB phy\n");
  		return retval;


Thanks,
Val.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux