linux.git/drivers/hid/hidraw.c, branch v3.0 Linux kernel source tree HID: 'name' and 'phys' in 'struct hid_device' can never be NULL 2011-05-18T11:23:31+00:00 Daniel Mack zonque@gmail.com 2011-05-15T16:07:42+00:00 dd2ed487fdd78b50549b2ca8418875c0d9f4a30e As they are static members of fix size, there is no need to NULL-check them. Signed-off-by: Daniel Mack <zonque@gmail.com> Cc: Dmitry Torokhov <dtor@mail.ru> Cc: Jiri Kosina <jkosina@suse.cz> Signed-off-by: Jiri Kosina <jkosina@suse.cz> 
As they are static members of fix size, there is no need to NULL-check them.

Signed-off-by: Daniel Mack <zonque@gmail.com>
Cc: Dmitry Torokhov <dtor@mail.ru>
Cc: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
HID: hidraw: fix comments 2011-03-27T18:30:32+00:00 Jiri Kosina jkosina@suse.cz 2011-03-27T18:29:02+00:00 d2a1cfebe38edc0bbac8f5cfbce062fe3d146d7a Adjust the comments a little bit. Signed-off-by: Jiri Kosina <jkosina@suse.cz> 
Adjust the comments a little bit.

Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Merge branches 'dragonrise', 'hidraw-feature', 'multitouch', 'ntrig', 'roccat', 'upstream' and 'upstream-fixes' into for-linus 2011-03-17T13:31:46+00:00 Jiri Kosina jkosina@suse.cz 2011-03-17T13:31:46+00:00 65b06194c9c9f41bc07ac6a6d42edb4b9e43fea4 






HID: Add Support for Setting and Getting Feature Reports from hidraw
2011-02-11T14:05:49+00:00

Alan Ott
alan@signal11.us

2011-01-18T08:04:39+00:00

b4dbde9da8ece42bbe4c70c26bac3b28dd6a3ddb

Per the HID Specification, Feature reports must be sent and received on
the Configuration endpoint (EP 0) through the Set_Report/Get_Report
interfaces.  This patch adds two ioctls to hidraw to set and get feature
reports to and from the device.  Modifications were made to hidraw and
usbhid.

New hidraw ioctls:
  HIDIOCSFEATURE - Perform a Set_Report transfer of a Feature report.
  HIDIOCGFEATURE - Perform a Get_Report transfer of a Feature report.

Signed-off-by: Alan Ott <alan@signal11.us>
Signed-off-by: Antonio Ospite <ospite@studenti.unina.it>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>





Per the HID Specification, Feature reports must be sent and received on
the Configuration endpoint (EP 0) through the Set_Report/Get_Report
interfaces.  This patch adds two ioctls to hidraw to set and get feature
reports to and from the device.  Modifications were made to hidraw and
usbhid.

New hidraw ioctls:
  HIDIOCSFEATURE - Perform a Set_Report transfer of a Feature report.
  HIDIOCGFEATURE - Perform a Get_Report transfer of a Feature report.

Signed-off-by: Alan Ott <alan@signal11.us>
Signed-off-by: Antonio Ospite <ospite@studenti.unina.it>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>







HID: hidraw: fix hidraw_disconnect()
2011-02-01T10:16:51+00:00

Stefan Achatz
erazor_de@users.sourceforge.net

2011-01-29T01:17:30+00:00

3a22ebe9cc76acac2511b1d3979a35609924ce42

hidraw_disconnect() first sets an entry in hidraw_table to NULL
and calls device_destroy() afterwards. The thereby called
hidraw_release() tries to read this already cleared value resulting
in never removing any device from the list.
This got fixed by changing the order of events.

Signed-off-by: Stefan Achatz <erazor_de@users.sourceforge.net>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>





hidraw_disconnect() first sets an entry in hidraw_table to NULL
and calls device_destroy() afterwards. The thereby called
hidraw_release() tries to read this already cleared value resulting
in never removing any device from the list.
This got fixed by changing the order of events.

Signed-off-by: Stefan Achatz <erazor_de@users.sourceforge.net>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>







HID: hidraw: Replace Confusing += Operator with =
2011-01-31T14:43:46+00:00

Alan Ott
alan@signal11.us

2011-01-27T03:25:18+00:00

cf28a673666fba5770b40fb084584391f451c07d

Setting of the return value of hidraw_read() uses the += operator when
= is more appropriate.  There is no case where ret can be anything
other than zero when the assignment is made, making = equivalent to
+= and much more clear.

Signed-off-by: Alan Ott <alan@signal11.us>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>





Setting of the return value of hidraw_read() uses the += operator when
= is more appropriate.  There is no case where ret can be anything
other than zero when the assignment is made, making = equivalent to
+= and much more clear.

Signed-off-by: Alan Ott <alan@signal11.us>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>







HID: hidraw: add compatibility ioctl() for 32-bit applications.
2011-01-04T10:21:28+00:00

Alan Ott
alan@signal11.us

2011-01-04T05:37:22+00:00

ae5e49c79c051ea1d5ca91cbd4a0d22189067ba3

Added the ioctl function to the compat_ioctl pointer in the file_operations
struct. Before this, some ioctls would fail for 32-bit apps on 64-bit systems.

Signed-off-by: Alan Ott <alan@signal11.us>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>





Added the ioctl function to the compat_ioctl pointer in the file_operations
struct. Before this, some ioctls would fail for 32-bit apps on 64-bit systems.

Signed-off-by: Alan Ott <alan@signal11.us>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>







Merge branch 'master' into upstream
2010-12-10T14:19:18+00:00

Jiri Kosina
jkosina@suse.cz

2010-12-10T14:19:18+00:00

2ade0c1d9d93b7642212657ef76f4a1e30233711












HID: Add and use hid_<level>: dev_<level> equivalents
2010-12-10T14:10:38+00:00

Joe Perches
joe@perches.com

2010-12-10T03:29:03+00:00

4291ee305e9bb0699504a66f0e2b7aefcf0512a5

Neaten current uses of dev_<level> by adding and using
hid specific hid_<level> macros.

Convert existing uses of dev_<level> uses to hid_<level>.
Convert hid-pidff printk uses to hid_<level>.

Remove err_hid and use hid_err instead.

Add missing newlines to logging messages where necessary.
Coalesce format strings.

Add and use pr_fmt(fmt) KBUILD_MODNAME ": " fmt

Other miscellaneous changes:

Add const struct hid_device * argument to hid-core functions
extract() and implement() so hid_<level> can be used by them.
Fix bad indentation in hid-core hid_input_field function
that calls extract() function above.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>





Neaten current uses of dev_<level> by adding and using
hid specific hid_<level> macros.

Convert existing uses of dev_<level> uses to hid_<level>.
Convert hid-pidff printk uses to hid_<level>.

Remove err_hid and use hid_err instead.

Add missing newlines to logging messages where necessary.
Coalesce format strings.

Add and use pr_fmt(fmt) KBUILD_MODNAME ": " fmt

Other miscellaneous changes:

Add const struct hid_device * argument to hid-core functions
extract() and implement() so hid_<level> can be used by them.
Fix bad indentation in hid-core hid_input_field function
that calls extract() function above.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>







HID: add hid_hw_open/close/power() handlers
2010-12-08T13:35:14+00:00

Dmitry Torokhov
dmitry.torokhov@gmail.com

2010-12-08T07:02:48+00:00

5bea7660bba973dc5e8e9d92b11fb1dd5b524ebf

Instead of exposing the guts of hid->ll_driver relationship to HID
sub-drivers provide these helpers to encapsulate the details.

Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>





Instead of exposing the guts of hid->ll_driver relationship to HID
sub-drivers provide these helpers to encapsulate the details.

Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>