Re: [PATCH 3/5] [media] au0828: Unregister notifiers

2016-03-22 Thread Shuah Khan
On 03/16/2016 06:04 AM, Mauro Carvalho Chehab wrote: > If au0828 gets removed, we need to remove the notifiers. > > Signed-off-by: Mauro Carvalho Chehab Tested bind_unbind au0828 loop 1000 times, followed by bind_unbind snd_usb_audio loop 1000 times. Didn't see any lock warnings on a KASAN enabl

[PATCH 3/5] [media] au0828: Unregister notifiers

2016-03-19 Thread Mauro Carvalho Chehab
If au0828 gets removed, we need to remove the notifiers. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/usb/au0828/au0828-core.c | 34 -- 1 file changed, 24 insertions(+), 10 deletions(-) diff --git a/drivers/media/usb/au0828/au0828-core.c b/drivers/medi

Re: [PATCH 3/5] [media] au0828: Unregister notifiers

2016-03-19 Thread Javier Martinez Canillas
Hello Mauro, Patch looks good to me, I just have a minor comment: On Wed, Mar 16, 2016 at 9:04 AM, Mauro Carvalho Chehab wrote: > If au0828 gets removed, we need to remove the notifiers. > > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/media/usb/au0828/au0828-core.c | 34 > +++