Hi Mika,

Can you please try the attached patch and see if it fixes this issue?

Thanks
-Manoj

-----Original Message-----
From: Mika Westerberg [mailto:[email protected]] 
Sent: Tuesday, September 24, 2013 2:26 PM
To: Jiri Kosina; Manoj Chourasia
Cc: [email protected]
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

Reply via email to