Am Dienstag, 14. September 2010, 02:52:10 schrieb Dmitry Torokhov: > On Mon, Sep 13, 2010 at 09:20:23PM +0200, Oliver Neukum wrote: > > Am Montag, 13. September 2010, 19:10:47 schrieb Dmitry Torokhov: > > > > > I think this introduces significant change in behavior though - before > > > we did not do usb_autopm_put_interface() on successful open, basically > > > disabling autopm facilities, right? > > > > Right. Which makes no sense at all. You'd better remove anything related > > to runtime PM and not set supports_autosuspend for that. > > > > That not what I meant, I do not want to remove autopm, it's just it was > effectively disabled and if we fix it we might start getting some > regression reports ;) True. So currently we have - a deadlock - disabled runtime power management We need to fix the deadlock. We can fix it retaining a disabled runtime power management. Or we can fix it fixing the runtime power management at the same time. However this opens the door to regressions. So for now I really suggest removing it from the driver and reintroduce it properly for the next merge window. Regards Oliver -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
