Re: [PATCH 14/16] rcar-vin: make use of video_device_alloc() and video_device_release()

2017-05-20 Thread Laurent Pinchart
Hi Niklas, On Saturday 20 May 2017 20:27:41 Niklas Söderlund wrote: > On 2017-05-10 16:36:03 +0300, Laurent Pinchart wrote: > > On Tuesday 14 Mar 2017 19:59:55 Niklas Söderlund wrote: > >> Make use of the helper functions video_device_alloc() and > >> video_device_release() to control the

Re: [PATCH 14/16] rcar-vin: make use of video_device_alloc() and video_device_release()

2017-05-20 Thread Niklas Söderlund
Hi Laurent, Thanks for your feedback. On 2017-05-10 16:36:03 +0300, Laurent Pinchart wrote: > Hi Niklas, > > On Tuesday 14 Mar 2017 19:59:55 Niklas Söderlund wrote: > > Make use of the helper functions video_device_alloc() and > > video_device_release() to control the lifetime of the struct > >

Re: [PATCH 14/16] rcar-vin: make use of video_device_alloc() and video_device_release()

2017-05-10 Thread Laurent Pinchart
Hi Niklas, On Tuesday 14 Mar 2017 19:59:55 Niklas Söderlund wrote: > Make use of the helper functions video_device_alloc() and > video_device_release() to control the lifetime of the struct > video_device. It's nice to see you considering lifetime management issues, but this isn't enough. The