On 07/23/2016 03:42 AM, Pavel Rojtberg wrote:
> resume still works here
>
> 2016-07-23 12:15 GMT+02:00 Pavel Rojtberg <rojtberg@xxxxxxxxx>:
>> If you blame the line, you will see that it was introduced to fix the
>> exact problem you are describing.
>> Maybe the used USB controller is relevant or something changed in USB
>> core meanwhile.
>> I will test whether resume works for me without that line as well and
>> report back.
>>
Yes, I saw the original commit. I included you on the email
directly so you would see the change.
I think whether the issue appears depends on USB power settings
in the system firmware. My desktop never reproduces the bug,
but my laptop almost always does.
On the machine where it never reproduced, the wireless adapter
_always_ reconnected on resume. Apparently, the HC lost enough
state that the devices were always re-enumerated.
On my laptop where it reproduced often, the wireless adapter
wouldn't reconnect on resume. When resuming, the reset_resume
path is always used, since we use USB_QUIRK_RESET_RESUME.
When the bug reproduces, the output URB gets stuck forever
and only completes when explicitly killed. I think this is the
type of behavior that the quirk is designed to work around.
By re-enumerating instead of resuming, we get correct behavior
on both LED lights (since they are correctly assigned again in
probe) and no URB hang anymore. We could probably write a
reset_resume callback that did the right thing in all cases,
but relying on USB core to do it for us is much less complexity.
>> Cameron Gutman <aicommander@xxxxxxxxx> schrieb am Sa., 23. Juli 2016
>> um 10:27 Uhr:
>>>
>>> Suspending and resuming the system can sometimes cause the out
>>> URB to get hung after a reset_resume. This causes LED setting
>>> and force feedback to break on resume. To avoid this, just drop
>>> the reset_resume callback so the USB core rebinds xpad to the
>>> wireless pads on resume if a reset happened.
>>>
>>> A nice side effect of this change is the LED ring on wireless
>>> controllers is now set correctly on system resume.
>>>
>>> Signed-off-by: Cameron Gutman <aicommander@xxxxxxxxx>
>>> Cc: stable@xxxxxxxxxxxxxxx
>>> ---
>>> drivers/input/joystick/xpad.c | 1 -
>>> 1 file changed, 1 deletion(-)
>>>
>>> diff --git a/drivers/input/joystick/xpad.c b/drivers/input/joystick/xpad.c
>>> index a529a45..843054a 100644
>>> --- a/drivers/input/joystick/xpad.c
>>> +++ b/drivers/input/joystick/xpad.c
>>> @@ -1626,7 +1626,6 @@ static struct usb_driver xpad_driver = {
>>> .disconnect = xpad_disconnect,
>>> .suspend = xpad_suspend,
>>> .resume = xpad_resume,
>>> - .reset_resume = xpad_resume,
>>> .id_table = xpad_table,
>>> };
>>>
>>> --
>>> 2.7.4
--
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
