Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1405347

Re: [PATCH v3] input: tablet: add Pegasus Notetaker tablet driver

Show all headers | View raw

On Mon, 2016-05-23 at 14:43 +0200, Martin Kepplinger wrote:
> Am 2016-05-23 um 14:26 schrieb Oliver Neukum:
> > On Mon, 2016-05-23 at 13:39 +0200, Martin Kepplinger wrote:
> > 
> >> It's *really* fun to use as an input tablet though! So let's support this
> >> for everybody.
> > 
> > Hi,
> > 
> > I am afraid there are a few issues.
> 
> Thanks for having a look, comments below.
> 
> > 
> > 1. Why the kernel thread?
> 
> I provide switching to the xy tablet mode by pressing a button during
> runtime. So usb_control_msg() is called from the thread, not the
> interrupt routine.
> 
> We don't get in the way of any user app. If we leave that functionality
> out, users may have to un- and replug in the device after using
> userspace apps that transfer data or the like.

The functionality is OK, but the implementation, though creative,
is buggy.

a - multiple devices will fail
b - you have a race condition that will fail to wake the thread
    if it is still running when the button is pressed again

Kernel threads are wasteful. Using the normal system work queue might be
much easier

> > 2. This driver has questionable power management.
> 
> I think I do it how others do it. Suspend/resume works. Anything
> specific required here?

Runtime PM is missing.

	Regards
		Oliver

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v3] input: tablet: add Pegasus Notetaker tablet driver Martin Kepplinger <martink@posteo.de> - 2016-05-23 13:50 +0200
  Re: [PATCH v3] input: tablet: add Pegasus Notetaker tablet driver Oliver Neukum <oneukum@suse.com> - 2016-05-23 14:40 +0200
    Re: [PATCH v3] input: tablet: add Pegasus Notetaker tablet driver Martin Kepplinger <martink@posteo.de> - 2016-05-23 14:50 +0200
      Re: [PATCH v3] input: tablet: add Pegasus Notetaker tablet driver Oliver Neukum <oneukum@suse.com> - 2016-05-23 15:10 +0200
        Re: [PATCH v3] input: tablet: add Pegasus Notetaker tablet driver Martin Kepplinger <martink@posteo.de> - 2016-05-23 19:20 +0200

csiph-web