[RFC v2 0/4] uhid LED input events handling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi all,

This patchset implements the support for handling LED input events in uhid
driver.

As requested in the previous RFC, it was introduced a new helper in hid-input.c
to carry out a generic LED input events handling (hidinput_led_output_report).
The helper is used by uhid driver.

The helper can also be used in i2c-hid driver since it does a very simply
handling of LED events too. I have no i2c device to test the i2c-hid patch,
so it requires some testing before applying it.

As long as the helper relies on device's .hid_output_raw_report() and
hid-logitech-dj driver doesn't properly implement this callback, the helper is
not used in hid-logitech-dj driver.

Also, this helper is not used in usbhid driver as long as it does a more
sophisticated handling than what hidinput_led_output_report helper does. For
further information about usbhid LEDs handling see comments in the patchset [1].

This patchset is rebased on Jiri's for-next branch.

Best regards,

Andre Guedes

[1] - http://marc.info/?l=linux-usb&m=132013970108623&w=2


Andre Guedes (4):
  HID: Add generic handler for LED input events
  HID: uhid: Handle LED input events
  HID: uhid: Use GFP_ATOMIC in uhid_hid_output_raw
  HID: i2c-hid: LED input events handling

 drivers/hid/hid-input.c       | 51 +++++++++++++++++++++++++++++++++++++++++++
 drivers/hid/i2c-hid/i2c-hid.c | 11 +---------
 drivers/hid/uhid.c            | 34 +++++++++++++++++++----------
 include/linux/hid.h           |  2 ++
 4 files changed, 76 insertions(+), 22 deletions(-)

-- 
1.8.0.1

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


[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux