From: "Henrique de Moraes Holschuh" <hmh@hmh.eng.br>
To: "Dmitry Torokhov" <dmitry.torokhov@gmail.com>
Cc: linux-kernel@vger.kernel.org, "Richard Purdie" <rpurdie@rpsys.net>
Subject: Re: [PATCH] Input: add flags bitfield
Date: Fri, 21 Mar 2008 18:56:26 -0300 [thread overview]
Message-ID: <1206136586.14058.1243686863@webmail.messagingengine.com> (raw)
In-Reply-To: <20080321160048.ZZRA012@mailhub.coreip.homeip.net>
On Fri, 21 Mar 2008 16:07:01 -0400, "Dmitry Torokhov"
<dmitry.torokhov@gmail.com> 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
next prev parent reply other threads:[~2008-03-21 21:56 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-16 20:14 [PATCH] Input: add flags bitfield Henrique de Moraes Holschuh
2008-03-21 20:07 ` Dmitry Torokhov
2008-03-21 21:56 ` Henrique de Moraes Holschuh [this message]
2008-03-22 7:09 ` Henrique de Moraes Holschuh
2008-03-24 1:02 ` Dmitry Torokhov
2008-03-24 3:25 ` Henrique de Moraes Holschuh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1206136586.14058.1243686863@webmail.messagingengine.com \
--to=hmh@hmh.eng.br \
--cc=dmitry.torokhov@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rpurdie@rpsys.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.