[patch] Re: HAL and hotplugging
Sascha Hlusiak
saschahlusiak at arcor.de
Sun Mar 16 05:18:58 PDT 2008
Am Samstag 15 März 2008 22:36:47 schrieb Dustin Spicuzza:
> - for (i = 0; props[i]; i++) {
> + for (i = 0; capabilities[i]; i++) {
> /* input.keys is the new, of which input.keyboard is a subset, but
> * input.keyboard is the old 'we have keys', so we have to keep it
> * around. */
> - if (strcmp(props[i], "input.keys") == 0 ||
> - strcmp(props[i], "input.keyboard") == 0)
> + if (strcmp(capabilities[i], "input.keys") == 0 ||
> + strcmp(capabilities[i], "input.keyboard") == 0)
> type |= TYPE_KEYS;
> - if (strcmp(props[i], "input.mouse") == 0 ||
> - strcmp(props[i], "input.touchpad") == 0)
> + if (strcmp(capabilities[i], "input.mouse") == 0 ||
> + strcmp(capabilities[i], "input.touchpad") == 0)
> type |= TYPE_POINTER;
> }
Can we get rid of that whole block, please? Why check for certain input.*
capabilities when input.x11_driver should be totally enough for deciding if
we can hotplug this or not (just exit and don't print an error when
input.x11_driver is not set). That way we can hotplug devices that are not
input.keys or input.mouse or input.touchpad, like a joystick with the
joystick driver.
- Sascha
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.x.org/archives/xorg/attachments/20080316/55fe4dba/attachment.pgp>
More information about the xorg
mailing list