[Letux-kernel] [PATCH v2] musb: omap2430: do not assume balanced enable()/disable()
tony at atomide.com
Thu Aug 4 17:01:26 CEST 2016
* H. Nikolaus Schaller <hns at goldelico.com> [160804 07:50]:
> > Am 04.08.2016 um 16:29 schrieb Tony Lindgren <tony at atomide.com>:
> > So I gave this patch a try but it now blocks all deeper SoC idle
> > states as the PHY stays active. I think the real fix is to make
> > sure the charger behaves independent of the USB PHY state.
> IMHO, plugin detection of the cable is a phy task and then it tells
> the charger to start. This part works.
> Charging did work up to kernel 4.3. It started to fail with 4.4-rc1
> without obvious changes to the charger but many patches to phy
> and musb. We had even backported the 4.7 charger driver
> to 4.3 and it failed as well.
> > So
> > probably this needs to be fixed in phy-twl4030-usb.c and
> > twl4030_charger.c instead. Now it sounds like we're also shutting
> > down the charger with the USB PHY.
> As a very deeply hidden side-effect the charger is shut down immediately
> after being started. Because phy-twl4030-usb.c does not do what it is expected
> and told to do.
> I have developed a workaround for the charger driver but I do not consider it
> as the solution.
> With that we have a workaround in the charger, but not a correct solution.
> That is what Andreas is trying to fix. The charger driver seems to be ok to
OK. So does the charger work with just phy-twl4030-usb and charger
modules loaded when cable is inserted?
More information about the Letux-kernel