1. 25 9月, 2012 3 次提交
    • M
      HID: hid-lg4ff: Set absolute axes parametes on DFP · a2063172
      Michal Malý 提交于
      The lg4ff driver doesn't fill the "input_absinfo" struct so it is left
      with default values. Applications with rely on information in this struct
      therefore do not work correctly with the wheel.
      
      Other Logitech wheels probably need this fix too, but again I do not have
      enough information to write it.
      Signed-off-by: NMichal Malý <madcatxster@gmail.com>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      a2063172
    • M
      HID: hid-lg4ff: Adjust X axis input value accordingly to selected range. · 2b24a960
      Michal Malý 提交于
      Range limiting command for the Driving Force Pro wheel is only a FF_SPRING
      effect so that the wheel creates resistance when the user tries to turn it past
      the limit. It is however possible to overpower the FFB motors quite easily which
      leads to the X axis value exceeding the expected limit. This confuses
      games which dynamically adjust calibration using the highest/lowest min and max
      values reported by the wheel. Joydev device driver also doesn't take in account
      any changes in an axis range after the joystick device is created.
      
      This patch recalculates received ABS_X axis value so it is always in
      <0; 16383> range where 0 is the left limit and 16383 the right limit.
      Logitech driver for Windows does the same thing.  As for any concerns about
      possible loss of precision, I compared a large set of raw/adjusted values
      generated by "mult_frac" to values returned by the Windows driver and I got
      a 100% match.
      
      Other Logitech wheels will probably need a similar fix, but I currently lack
      the information needed to write one.
      Signed-off-by: NMichal Malý <madcatxster@gmail.com>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      2b24a960
    • M
      HID: hid-lg4ff: Minor code cleanup to improve readability · 74479ba8
      Michal Malý 提交于
      This patch replaces all occurrences of "report->field[0]->value[n]" with just
      "value[n]" to get rid of the lengthy trains we have now.
      Signed-off-by: NMichal Malý <madcatxster@gmail.com>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      74479ba8
  2. 22 9月, 2012 2 次提交
  3. 07 9月, 2012 1 次提交
  4. 06 9月, 2012 1 次提交
  5. 02 9月, 2012 1 次提交
  6. 27 8月, 2012 1 次提交
  7. 22 8月, 2012 1 次提交
  8. 31 7月, 2012 1 次提交
  9. 23 7月, 2012 1 次提交
  10. 22 7月, 2012 1 次提交
  11. 20 7月, 2012 11 次提交
    • D
      HID: Allow drivers to be their own listener · 4bc19f62
      David Herrmann 提交于
      hid-picolcd and hid-wiimote do not allow any of hidinput, hiddev or hidraw
      to claim the device but still want to remain on the bus. Hence, if a
      driver uses the raw_event callback but no other listener claimed the
      device, we still leave it on the bus as the driver handles everything by
      itself. It thus becomes its own listener.
      
      Under some circumstances (eg., hidinput_connect() fails and raw_event set)
      a device may be left on the bus even though it requires external
      listeners. But then if hidinput_connect() fails there are bigger issues
      than a device that is left unhandled. So we can safely use this heuristic
      to avoid adding another flag for special devices like hid-picolcd and
      hid-wiimote.
      
      This also removes the ugly hack from hid-picolcd as this is no longer
      required.
      Signed-off-by: NDavid Herrmann <dh.herrmann@googlemail.com>
      Acked-by: NHenrik Rydberg <rydberg@euromail.se>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      4bc19f62
    • A
      HID: usbhid: fix error paths in suspend · eb055fd0
      Alan Stern 提交于
      This patch (as1597) fixes some of the error paths in usbhid's suspend
      routine.  The driver was not careful to restart everything that might
      have been stopped, in cases where a suspend failed.
      
      For example, once the HID_SUSPENDED flag is set, an output report
      submission would not restart the corresponding URB queue.  If a
      suspend fails, it's therefore necessary to check whether the queues
      need to be restarted.
      Signed-off-by: NAlan Stern <stern@rowland.harvard.edu>
      CC: Oliver Neukum <oliver@neukum.org>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      eb055fd0
    • A
      HID: usbhid: check for suspend or reset before restarting · d4150c8f
      Alan Stern 提交于
      This patch (as1596) improves the queue-restart logic in usbhid by
      checking to see if the device is suspended or a reset is about to
      occur.  There's no point submitting an URB if either of those is
      true.
      Signed-off-by: NAlan Stern <stern@rowland.harvard.edu>
      CC: Oliver Neukum <oliver@neukum.org>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      d4150c8f
    • A
      HID: usbhid: replace HID_REPORTED_IDLE with HID_SUSPENDED · f2b5264d
      Alan Stern 提交于
      This patch (as1595) improves the usbhid driver by using the
      HID_SUSPENDED bitflag to indicate that the device is suspended rather
      than using HID_REPORTED_IDLE, which the patch removes.
      
      Since HID_SUSPENDED was not being used for anything, and since the
      name "HID_REPORTED_IDLE" doesn't convey much meaning, the end result
      is easier to read and understand.
      Signed-off-by: NAlan Stern <stern@rowland.harvard.edu>
      CC: Oliver Neukum <oliver@neukum.org>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      f2b5264d
    • A
      HID: usbhid: inline some simple routines · 93101af3
      Alan Stern 提交于
      This patch (as1594) simplifies the usbhid driver by inlining a couple
      of routines.  As a result of an earlier patch, irq_out_pump_restart()
      and ctrl_pump_restart() are each used in only one place.  Since they
      don't really do what their names say, and since they each involve only
      about two lines of actual code, there's no reason to keep them as
      separate functions.
      Signed-off-by: NAlan Stern <stern@rowland.harvard.edu>
      CC: Oliver Neukum <oliver@neukum.org>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      93101af3
    • A
      HID: usbhid: fix autosuspend calls · 01a7c984
      Alan Stern 提交于
      This patch (as1593) fixes some logic errors in the usbhid driver
      relating to runtime PM.  The driver does not balance its calls to
      usb_autopm_get_interface_async() and usb_autopm_put_interface_async().
      
      For example, when the control queue is restarted the driver does a
      _get.  But the resume won't happen immediately, so the driver leaves
      the queue stopped.  When the resume does occur, the queue is restarted
      and a second _get occurs, with no balancing _put.
      
      The patch fixes the problem by rearranging the logic for restarting
      the queues.  All the _get/_put calls and bitflag settings in
      __usbhid_submit_report() are moved into the queue-restart routines.  A
      balancing _put call is added for the case where the queue is still
      suspended.  A call to irq_out_pump_restart(), which doesn't take all
      the right actions for restarting the irq-OUT queue, is replaced by a
      call to usbhid_restart_out_queue(), which does.  Similarly for
      ctrl_pump_restart().
      
      Finally, new code is added to prevent an autosuspend from happening
      every time an URB is cancelled, and the comments explaining what
      happens when an URB needs to be cancelled are expanded and clarified.
      Signed-off-by: NAlan Stern <stern@rowland.harvard.edu>
      CC: Oliver Neukum <oliver@neukum.org>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      01a7c984
    • A
      HID: usbhid: fix use-after-free bug · 668160e5
      Alan Stern 提交于
      This patch (as1592) fixes an obscure problem in the usbhid driver.
      Under some circumstances, a control or interrupt-OUT URB can be
      submitted twice.  This will happen if the first submission fails; the
      queue pointers aren't updated, so the next time the queue is restarted
      the same URB will be submitted again.
      
      The problem is that raw_report gets deallocated during the first
      submission.  The second submission will then dereference and try to
      free an already-freed region of memory.  The patch fixes the problem
      by setting raw_report to NULL when it is deallocated and checking for
      NULL before dereferencing it.
      Signed-off-by: NAlan Stern <stern@rowland.harvard.edu>
      CC: Oliver Neukum <oliver@neukum.org>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      668160e5
    • M
      HID: hid-core: optimize in case of hidraw · b94e3c94
      Matthieu CASTET 提交于
      When using hidraw, hid buffer can be big and take lot's of
      time to process (interrupt) kernel context.
      Don't try to parse report if we are only interrested in hidraw.
      
      Also don't prepare data for debug stuff if no debugfs file
      are opened.
      Signed-off-by: NMatthieu CASTET <matthieu.castet@parrot.com>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      b94e3c94
    • M
      HID: hidraw: fix list->buffer memleak · 4c7b417e
      Matthieu CASTET 提交于
      If we don't read fast enough hidraw device, hidraw_report_event
      will cycle and we will leak list->buffer.
      Also list->buffer are not free on release.
      After this patch, kmemleak report nothing.
      Signed-off-by: NMatthieu CASTET <matthieu.castet@parrot.com>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      4c7b417e
    • V
      HID: uhid: Fix sending events with invalid data · adefb69b
      Vinicius Costa Gomes 提交于
      This was detected because events with invalid types were arriving
      to userspace.
      
      The code before this patch would only work for the first event in the
      queue (when uhid->tail is 0).
      Signed-off-by: NVinicius Costa Gomes <vinicius.gomes@openbossa.org>
      Reviewed-by: NDavid Herrmann <dh.herrmann@googlemail.com>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      adefb69b
    • S
      HID: roccat: added sensor sysfs attribute for Savu · 11a5c818
      Stefan Achatz 提交于
      The sensor attr can be used to tweak the optical sensor of the Savu.
      Signed-off-by: NStefan Achatz <erazor_de@users.sourceforge.net>
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      11a5c818
  12. 19 7月, 2012 1 次提交
  13. 12 7月, 2012 1 次提交
  14. 11 7月, 2012 2 次提交
  15. 09 7月, 2012 2 次提交
  16. 05 7月, 2012 2 次提交
  17. 04 7月, 2012 1 次提交
  18. 28 6月, 2012 6 次提交
  19. 25 6月, 2012 1 次提交