Re: [PATCH 1/2] usb: mtu3: disable USB2 LPM

Greg Kroah-Hartman
2020-06-29T08:11:08Z

On Mon, Jun 29, 2020 at 09:30:58AM +0200, Markus Elfring wrote:
> > A SuperSpeed device shall include the USB 2.0 extension descriptor
> > and shall support LPM when operating in USB 2.0 HS mode(see usb3.2
> > spec9.6.2.1). But we always don't support it, so disable it by
> > default, otherwise device will enter LPM suspend mode when
> > connected to Win10 system.
>
> How do you think about a wording variant like the following?
>
> Change description:
> A SuperSpeed device shall include the USB 2.0 extension descriptor
> and shall support Link Power Management when operating in USB 2.0
> High Speed mode. (See also: USB 3.2 specification 9.6.2.1)
> But we do not support it generally. Thus disable this functionality
> by default.
> Otherwise, the device will enter LPM suspend mode when connected
> to Win10 system.
>
>
> Would you like to add the tag “Fixes” to the commit message?
>
> Regards,
> Markus

Hi,

This is the semi-friendly patch-bot of Greg Kroah-Hartman.

Markus, you seem to have sent a nonsensical or otherwise pointless
review comment to a patch submission on a Linux kernel developer mailing
list. I strongly suggest that you not do this anymore. Please do not
bother developers who are actively working to produce patches and
features with comments that, in the end, are a waste of time.

Patch submitter, please ignore Markus's suggestion; you do not need to
follow it at all. The person/bot/AI that sent it is being ignored by
almost all Linux kernel maintainers for having a persistent pattern of
behavior of producing distracting and pointless commentary, and
inability to adapt to feedback. Please feel free to also ignore emails
from them.

thanks,

greg k-h's patch email bot