Re: [PATCH 1/2] [media] uvcvideo: Refactor teardown of uvc on USB disconnect

2017-04-30 Thread Greg KH
On Mon, Apr 17, 2017 at 03:23:55PM +0300, Laurent Pinchart wrote: > Hi Daniel, > > Thank you for the patch (and the investigation). > > On Monday 17 Apr 2017 18:52:39 Daniel Axtens wrote: > > Currently, disconnecting a USB webcam while it is in use prints out a > > number of warnings, such as: >

Re: [PATCH 1/2] [media] uvcvideo: Refactor teardown of uvc on USB disconnect

2017-04-22 Thread Daniel Axtens
Hi Laurent, >> To fix this, we need to make sure the devices are always unregistered >> before the end of uvc_disconnect(). To this, move the unregistration >> into the disconnect path: >> >> - split uvc_status_cleanup() into two parts, one on disconnect that >>unregisters and one on delete t

Re: [PATCH 1/2] [media] uvcvideo: Refactor teardown of uvc on USB disconnect

2017-04-17 Thread Laurent Pinchart
Hi Daniel, Thank you for the patch (and the investigation). On Monday 17 Apr 2017 18:52:39 Daniel Axtens wrote: > Currently, disconnecting a USB webcam while it is in use prints out a > number of warnings, such as: > > WARNING: CPU: 2 PID: 3118 at > /build/linux-ezBi1T/linux-4.8.0/fs/sysfs/group

[PATCH 1/2] [media] uvcvideo: Refactor teardown of uvc on USB disconnect

2017-04-17 Thread Daniel Axtens
Currently, disconnecting a USB webcam while it is in use prints out a number of warnings, such as: WARNING: CPU: 2 PID: 3118 at /build/linux-ezBi1T/linux-4.8.0/fs/sysfs/group.c:237 sysfs_remove_group+0x8b/0x90 sysfs group a7cd0780 not found for kobject 'event13' This has been noticed be