Re: [PATCH 0/4] musb fixes for v4.9-rc cycle

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

 



* Ladislav Michl <ladis@xxxxxxxxxxxxxx> [161108 06:39]:
> Hi Tony,
> 
> On Mon, Nov 07, 2016 at 02:50:16PM -0700, Tony Lindgren wrote:
> > Hi all,
> > 
> > Here are musb fixes for the issues that I've been able to track down.
> > Not sure if these will help with the problem Ladis was seeing as I'm
> > not able to reproduce that one it seems.
> 
> I still need 87326e858448c40e32f142c0b8dcc59d7b27c641 (usb: musb: Remove
> extra PM runtime calls from 2430 glue layer) reverted, but I'm happy with
> that solution so far.

Weird, I have not figured out what happens there..

The only thing I'm still wondering about is if phy-twl4030-usb.c
is losing ID pin interrupts occasionally but have not been
able to verify that. Maybe try the following additional patch
and see if it makes any difference?

Regards,

Tony

8< --------------
--- a/drivers/phy/phy-twl4030-usb.c
+++ b/drivers/phy/phy-twl4030-usb.c
@@ -304,6 +304,17 @@ static enum musb_vbus_id_status
 				status &= ~BIT(7);
 			else
 				twl->vbus_supplied = true;
+
+			/* Recheck status again for ID pin */
+			if (!(status & BIT(2))) {
+				msleep(200);
+				status = twl4030_readb(twl,
+						       TWL_MODULE_PM_MASTER,
+						       STS_HW_CONDITIONS);
+				if (status < 0)
+					dev_err(twl->dev, "USB link status err %d\n",
+						status);
+			}
 		}
 
 		if (status & BIT(2))
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux