[Letux-kernel] [PATCH v2] phy-twl4030-usb: initialize charging-related stuff via pm_runtime
Tony Lindgren
tony at atomide.com
Thu Aug 25 00:42:44 CEST 2016
* Andreas Kemnade <andreas at kemnade.info> [160824 15:30]:
> twl4030_phy_power_on() initializes some bits which are required for
> charging. As they are not set in twl4030_usb_runtime_resume()
> a call to pm_runtime_get_sync() is not sufficient to enable charging.
>
> This patch moves the initialization to twl4030_usb_runtime_resume()
> so everything needed for charging is initialized upon
> pm_runtime_get_sync().
>
> That also gives improved possibilities to debug problems in that area
> because the relevant parts can be checked separately. Charging can be
> enabled without having the musb subsystem active.
>
> As a side effect this hides some bugs in musb which causes
> unbalanced calls to phy_power_off()/phy_power_on() so that
> phy->power_count becomes -1.
>
> The result is that e.g. the GTA04 phone (dm3730 + twl4030) works
> finally as a usb gadget again and charging is working.
Thanks for updating it:
Acked-by: Tony Lindgren <tony at atomide.com>
More information about the Letux-kernel
mailing list