Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • KWin KWin
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 57
    • Issues 57
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 156
    • Merge requests 156
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • PlasmaPlasma
  • KWinKWin
  • Issues
  • #40
Closed
Open
Issue created May 11, 2021 by Bhushan Shah@bshah📱Developer0 of 1 checklist item completed0/1 checklist item

Follow-up from "platforms/drm: drop the double tap timer"

The following discussion from !778 (merged) should be addressed:

  • @davidedmundson started a discussion: (+1 comment)

    We also eventually want to close the libinput fd (would need separate patch though), to allow for touchscreen device to go into power saving state using runtime_pm.

    Yes. This won't make any difference until that's implemented.

    At which point I don't understand reason to remove this. If libinput is completely closed, then this code won't do anything as we won't process those double tap events in the first place.

    So removing it will only affect hardware that doesn't support hardware interrupts (like my lenovo convertible), so I don't really see the point.

    (I would support moving all this filter from the DRM backend code though)

  • Move out drm input event filter out of the DRM backend code

  • Close libinput fd when we are not using it (during DPMS off)

Assignee
Assign to
Time tracking