Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-22 Thread Shuah Khan
On 03/22/2016 01:56 PM, Shuah Khan wrote: > On 03/16/2016 06:04 AM, Mauro Carvalho Chehab wrote: >> Now that the media_device can be used by multiple drivers, >> via devres, we need to be sure that it will be dropped only >> when all drivers stop using it. >> >> Signed-off-by: Mauro Carvalho Chehab

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-22 Thread Shuah Khan
On 03/16/2016 06:04 AM, Mauro Carvalho Chehab wrote: > Now that the media_device can be used by multiple drivers, > via devres, we need to be sure that it will be dropped only > when all drivers stop using it. > > Signed-off-by: Mauro Carvalho Chehab Tested bind_unbind au0828 loop 1000 times, fo

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-20 Thread Shuah Khan
On 03/16/2016 08:25 AM, Mauro Carvalho Chehab wrote: > Em Wed, 16 Mar 2016 08:05:15 -0600 > Shuah Khan escreveu: > >> On 03/16/2016 06:04 AM, Mauro Carvalho Chehab wrote: >>> Now that the media_device can be used by multiple drivers, >>> via devres, we need to be sure that it will be dropped only

[PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-19 Thread Mauro Carvalho Chehab
Now that the media_device can be used by multiple drivers, via devres, we need to be sure that it will be dropped only when all drivers stop using it. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/media-device.c | 48 +++- include/media/media-devi

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-19 Thread Sakari Ailus
Hi Mauro, On Wed, Mar 16, 2016 at 09:04:05AM -0300, Mauro Carvalho Chehab wrote: > Now that the media_device can be used by multiple drivers, > via devres, we need to be sure that it will be dropped only > when all drivers stop using it. > > Signed-off-by: Mauro Carvalho Chehab For patches 2 (a

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-19 Thread Javier Martinez Canillas
Hello Mauro, Patch looks almost good to me, I just have a question below: On Wed, Mar 16, 2016 at 9:04 AM, Mauro Carvalho Chehab wrote: > Now that the media_device can be used by multiple drivers, > via devres, we need to be sure that it will be dropped only > when all drivers stop using it. > >

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-19 Thread Sakari Ailus
Hi Mauro, On Wed, Mar 16, 2016 at 09:04:05AM -0300, Mauro Carvalho Chehab wrote: > Now that the media_device can be used by multiple drivers, > via devres, we need to be sure that it will be dropped only > when all drivers stop using it. Not long ago we split media device initialisation and regis

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-19 Thread Mauro Carvalho Chehab
Em Wed, 16 Mar 2016 08:05:15 -0600 Shuah Khan escreveu: > On 03/16/2016 06:04 AM, Mauro Carvalho Chehab wrote: > > Now that the media_device can be used by multiple drivers, > > via devres, we need to be sure that it will be dropped only > > when all drivers stop using it. > > > > Signed-off-by:

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-19 Thread Mauro Carvalho Chehab
Em Thu, 17 Mar 2016 13:50:53 +0200 Sakari Ailus escreveu: > Hi Mauro, > > On Wed, Mar 16, 2016 at 09:04:05AM -0300, Mauro Carvalho Chehab wrote: > > Now that the media_device can be used by multiple drivers, > > via devres, we need to be sure that it will be dropped only > > when all drivers sto

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-18 Thread Shuah Khan
On 03/16/2016 06:04 AM, Mauro Carvalho Chehab wrote: > Now that the media_device can be used by multiple drivers, > via devres, we need to be sure that it will be dropped only > when all drivers stop using it. > > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/media/media-device.c | 48 >