* Andreas Kemnade <andreas@xxxxxxxxxxxx> [160805 08:35]: > I repeat the subject line of the patch: > [PATCH v2] musb: omap2430: do not assume balanced enable()/disable() > It is *not* fix charging. > > So you mean that the phy should magically know at which refcounter > it should power on and off if power on/off is not called in the > balanced way? No, I mean we need to figure out why things can be called in unbalanced way. With your patch I end up with unbalanced calls leaving the phy on after all the modules have been removed :) > Maybe the phy-twl4030 uses the phy layer wrong. > Now the relevant part of power on/off in phy-twl4030 is done via struct > phy_ops.power_off/power_on (*not* via pm_runtime). Maybe that is wrong > and more parts should be moved to the pm_runtime stuff (which is also > present). We should use phy power_off/power_on for the USB related parts. The parts needed by other components, like VBUS detection, should be handled by PM runtime. We should get phy-twl4030-usb and the charger driver working also when no musb modules are loaded. > Then the phy subsystem has its own power refcounter in struct > phy.power_count. It it handled via phy_power_off()/phy_power_on(). > And that is called from musb/omap2430.c > But that is another story. Yes that's what the USB driver is expected to do. But obviously there are issues remaining also in the phy-twl4030-usb.c driver. And it seems that we should have some OTG parts always enabled when VBUS is detected when twl4030-charger is configured? > > If there are MUSB specific PM runtime issues then let's fix > > those separately. > > > And that exactly tries my patch to do. For that task it does not > even use the PM runtime system. Again please read the subject line of > the patch. Maybe it unveils some other pm issues in musb > which should first be fixed in a complete patch series. Certainly that needs to be fixed, but let's do it in a way where things work for other test cases also. Care to describe how to to reproduce the issue you're seeing? It seems that you are seeing devices not being enmerated leading to the charger not working? Is this with built in MUSB and phy modules? Regrds, Tony -- 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
