Hi Mika, Can you please try the attached patch and see if it fixes this issue? Thanks -Manoj -----Original Message----- From: Mika Westerberg [mailto:mika.westerberg@xxxxxxxxxxxxxxx] Sent: Tuesday, September 24, 2013 2:26 PM To: Jiri Kosina; Manoj Chourasia Cc: linux-input@xxxxxxxxxxxxxxx Subject: Q: weird hidraw behaviour Hi, I noticed that after commit 212a871a393 (HID: hidraw: correctly deallocate memory on device disconnect) hidraw doesn't close the underlying hid device when the device node is closed last time. For example I have a touch panel (HID over I2C) device with added debug prints in i2c_hid_open()/i2c_hid_close(): # od -x /dev/hidraw0 [ 41.363813] i2c_hid 1-004c: i2c_hid_power lvl:32 [ 41.368464] i2c_hid 1-004c: i2c_hid_set_power [ 41.372831] i2c_hid 1-004c: __i2c_hid_command: cmd=54 01 00 08 [ 41.451455] i2c_hid 1-004c: i2c_hid_open ^C # od -x /dev/hidraw0 [ 58.420928] i2c_hid 1-004c: i2c_hid_power lvl:32 [ 58.425577] i2c_hid 1-004c: i2c_hid_set_power [ 58.429945] i2c_hid 1-004c: __i2c_hid_command: cmd=54 01 00 08 [ 58.525276] i2c_hid 1-004c: i2c_hid_open ^C i2c_hid_close() is never called. Is this intended or am I missing something? Thanks. ----------------------------------------------------------------------------------- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message. -----------------------------------------------------------------------------------
Attachment:
HID-hidraw-close-underlying-device-at-removal-of-las.patch
Description: HID-hidraw-close-underlying-device-at-removal-of-las.patch
