[lkml]   [2008]   [Mar]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] Input: add flags bitfield
On Fri, 21 Mar 2008, Henrique de Moraes Holschuh wrote:
> On Fri, 21 Mar 2008 16:07:01 -0400, "Dmitry Torokhov"
> <> said:
> > On Sun, Mar 16, 2008 at 05:14:11PM -0300, Henrique de Moraes Holschuh wrote:
> > > Add a flags bitfield to the input_dev structure, which can be used for
> > > internal coordination among kernel input devices and input handlers without
> > > the need to use ever-expanding blacklists on the input handlers.
> > >
> > > Add initial flag bits which allows an input driver to request that joystick
> > > emulation (joydev) or mouse emulation (mousedev) not be attached to an
> > > input device.
> > >
> > > This will be used by accelerometer drivers exporting a raw interface which
> > > is not to be used as a joystick device (not to confuse this with the usual
> > > fuzzed joystick interface these drivers export for enhanced Neverball
> > > productivity), for example.
> >
> > I'd rather not apply this patch because it pushes kowledge of existing
> > input interfaces into device drivers. What we could do instead is add
> > a 'type' field to the input device structure and then input interfaces
> > (evdev/mousedev, etc) could have an option of matching either by device
> > type or by device capabilities or both. Your raw devices could have type
> > of accelerometer and joydev would bind to devices with type "joystick"
> > or "unknown" + certain capabilities. Will this work?
> It would solve my problem, yes.
> But I'd prefer if joydev and mousedev did not bind to
> unknown+capabilities, just in case. Looks like bad form to me, and
> might bite us back later on. We can properly fix all drivers in-tree
> to have suitable types for joydev and/or mousedev binds, rfkill binds,
> and so on after all.

Drat, removing unknown+capabilities means I'd have to hunt down every
frigging input device in the tree to annotate its type... otherwise, it
would cause regressions re. the handlers. It is a work that needs to be
done anyway, only adding type metadata to new devices and leaving the
rest to report "unknown" is just icky.

Come to think of it, what about uinput? It would need to be able to set
the device type as well, otherwise mousedev and joydev, for example,
would not attach to uinput-created input devices.

"One disk to rule them all, One disk to find them. One disk to bring
them all and in the darkness grind them. In the Land of Redmond
where the shadows lie." -- The Silicon Valley Tarot
Henrique Holschuh

 \ /
  Last update: 2008-03-22 08:13    [W:0.108 / U:1.392 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site