[patch 1/2] hald modifications for vmware's usb mouse

Philip Langdale plangdale at vmware.com
Wed Jan 30 13:58:57 PST 2008


Dan Nicholson wrote:
> On Jan 28, 2008 4:56 AM, Vinay Reddy <vinayvinay at gmail.com> wrote:
>> Aargh, hit "Send" a little too fast...
>>
>> I do have commit privileges to get the patch checked in; I just wish
>> to get it approved by someone who knows more about HAL.
> 
> I would say, sending it to the HAL mailing list would probably help (CC'd).

FWIW, These device characteristics also match those of some ILO hardware input
devices. A patch from HP was made to the linux kernel's driver to handle
this type of device.

--phil

>> On Jan 28, 2008 6:23 PM, Vinay Reddy <vinayvinay at gmail.com> wrote:
>>> Hi,
>>> Currently, VMware's absolute USB mouse is not correctly tagged by hald
>>> (it only gets the "input" capability). Here's a patch that adds the
>>> "input.mouse" capability to VMware's absolute USB mouse.
>>>
>>> Thanks,
>>> Vinay
>>>
>>> diff --git a/hald/linux/device.c b/hald/linux/device.c
>>> index b55da25..99dc026 100644
>>> --- a/hald/linux/device.c
>>> +++ b/hald/linux/device.c
>>> @@ -258,9 +258,18 @@ input_test_abs (HalDevice *d, const char *sysfs_path)
>>>                 goto out;
>>>         num_bits = input_str_to_bitmask (s, bitmask, sizeof (bitmask));
>>>
>>> -       if (test_bit (ABS_X, bitmask) && test_bit (ABS_Y, bitmask) &&
>>> test_bit (ABS_PRESSURE, bitmask)) {
>>> -               hal_device_add_capability (d, "input.touchpad");
>>> -                goto out;
>>> +       if (test_bit (ABS_X, bitmask) && test_bit (ABS_Y, bitmask)) {
>>> +                if (test_bit (ABS_PRESSURE, bitmask)) {
>>> +                        hal_device_add_capability (d, "input.touchpad");
>>> +                        goto out;
>>> +                } else {
>>> +                        /*
>>> +                         * This path is taken by VMware's USB mouse, which has
>>> +                         * absolute axes, but no touch/pressure button.
>>> +                         */
>>> +                        hal_device_add_capability (d, "input.mouse");
>>> +                        goto out;
>>> +                }
>>>          }
>>>
>>>          /* TODO: Hmm; this code looks sketchy... why do we do
>>> !test_bit on the Y axis ?? */
>>>
>>
>>
>> --
>> After silence, that which comes nearest to expressing the
>> inexpressible is music - Aldous Huxley
>>
>> _______________________________________________
>> xorg mailing list
>> xorg at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/xorg
>>
> _______________________________________________
> xorg mailing list
> xorg at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/xorg




More information about the xorg mailing list