lkml.org 
[lkml]   [2022]   [Jul]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH v3 10/10] HID: usbhid: Set USB mice as s2idle wakeup resources
    From
    (FYI - combined both of Greg's responses in this message rather than 
    responding to them separately)

    > Note, this is very different than older versions of Windows. Any idea
    when this behavior and prescription changed?

    Older versions of Windows also didn't support Modern Standby, so I would
    hypothesize it's either Windows 8 or Windows 10 that changed it.

    > Where are patches 1-9 of this series?

    Patch 1 was based on your comment in v2 that there was no helper (it
    introduces a helper).

    Patches 2-9 are just changes across the kernel to use this helper.

    The entire series was submitted to LKML, but individual patches only to
    the relevant maintainers for the subsystem. If there is a v4 patch I'll
    CC everyone on everything.

    Here is a lore link to the whole series:
    https://lore.kernel.org/lkml/20220701023328.2783-1-mario.limonciello@amd.com/

    On 7/1/22 01:58, Greg KH wrote:
    > On Thu, Jun 30, 2022 at 09:33:28PM -0500, Mario Limonciello wrote:
    >> The USB HID transport layer doesn't configure mice for wakeup by default.
    >> Thus users can not wake system from s2idle via USB mouse. However, users
    >> can wake the same system from Modern Standby on Windows with the same USB
    >> mouse.
    >>
    >> Microsoft documentation indicates that all USB mice and touchpads should
    >> be waking the system from Modern Standby.
    >
    > As I said before, their documentation indicates that all _EXTERNAL_ mice
    > and touchpads. You forgot about internally connected mice and touchpads
    > here, you wouldn't want them to wake up a device asleep, right?
    >

    Is this actually a thing? I've been in the PC industry a while but
    never seen a design that supported Modern Standby/s2idle that opted to
    use USB to connect an "internal" touchpad/mouse. Microsoft's own
    documentation in the same link advises against it at least too.

    "We recommend using HIDI2C for input peripherals whenever possible for
    better power efficiency, but this is not a requirement".

    But in any case - wakeup from this type of device is a grey area. They
    require that internal touchpads connected to I2C be a wake source and
    also USB touchpads.
    There is nothing on that page prescribing the behavior for an "internal"
    USB touchpad.

    Reading between the lines given the intent, I would argue if such a
    design is created or exists internal USB mice and touchpads should also
    be a wake up.

    \
     
     \ /
      Last update: 2022-07-01 20:01    [W:2.159 / U:0.556 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site