Hi Mika,

I have posted the patch in mailing list for review. Attaching it here too.
Please test it one more time as I have fixed the warning.

Regards
-Manoj


-----Original Message-----
From: Mika Westerberg [mailto:[email protected]] 
Sent: Tuesday, October 01, 2013 6:13 PM
To: Manoj Chourasia
Cc: [email protected]; Jiri Kosina
Subject: Re: weird hidraw behaviour

On Tue, Oct 01, 2013 at 05:13:16PM +0530, Manoj Chourasia wrote:
> Can you please try the attached patch and see if it fixes this issue?

Yes, that patch fixes the issue for me, thanks!

It introduces following warning, though:

drivers/hid/hidraw.c: In function ‘drop_ref’:
drivers/hid/hidraw.c:320:5: warning: suggest explicit braces to avoid ambiguous 
‘else’ [-Wparentheses]
  if (!hidraw->open)

-----------------------------------------------------------------------------------
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: PATCH-hidraw-close-underlying-device-at-removal-of-las.patch
Description: PATCH-hidraw-close-underlying-device-at-removal-of-las.patch

Reply via email to