Re: [PATCH] Input: add flags bitfield

[Henrique de Moraes Holschuh]

On Fri, 21 Mar 2008 16:07:01 -0400, "Dmitry Torokhov"
<dmitry.torokhov@xxxxxxxxx> 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.

What did I miss?  Are there drivers for some sort of generic device that
would be unable to set their type bitfields at runtime?

BTW: as far as I can see, type should be a bitfield or a list, to allow
a
device to have more than one type.  Is that fine with you?

-- 
  "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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/