Re: [PATCH 1/2] ipu3-cio2: Unregister device nodes first, then release resources

2018-10-15 Thread Bing Bu Cao
On 10/15/2018 04:39 PM, Sakari Ailus wrote: > Hi Bingbu, > > On Mon, Oct 15, 2018 at 03:15:05PM +0800, Bing Bu Cao wrote: >> On 10/10/2018 04:32 PM, Sakari Ailus wrote: >>> While there are issues related to object lifetime management, unregister >>> the media device first, followed immediately

Re: [PATCH 1/2] ipu3-cio2: Unregister device nodes first, then release resources

2018-10-15 Thread Sakari Ailus
Hi Bingbu, On Mon, Oct 15, 2018 at 03:15:05PM +0800, Bing Bu Cao wrote: > > On 10/10/2018 04:32 PM, Sakari Ailus wrote: > > While there are issues related to object lifetime management, unregister > > the media device first, followed immediately by other device nodes when > > the driver is being

Re: [PATCH 1/2] ipu3-cio2: Unregister device nodes first, then release resources

2018-10-15 Thread Bing Bu Cao
On 10/10/2018 04:32 PM, Sakari Ailus wrote: > While there are issues related to object lifetime management, unregister > the media device first, followed immediately by other device nodes when > the driver is being unbound. Only then the resources needed by the driver > may be released. This is

Re: [PATCH 1/2] ipu3-cio2: Unregister device nodes first, then release resources

2018-10-11 Thread Bing Bu Cao
Tested-by: Bingbu Cao Reviewed-by: Bingbu Cao On 10/10/2018 04:32 PM, Sakari Ailus wrote: > While there are issues related to object lifetime management, unregister > the media device first, followed immediately by other device nodes when > the driver is being unbound. Only then the resources

[PATCH 1/2] ipu3-cio2: Unregister device nodes first, then release resources

2018-10-10 Thread Sakari Ailus
While there are issues related to object lifetime management, unregister the media device first, followed immediately by other device nodes when the driver is being unbound. Only then the resources needed by the driver may be released. This is slightly safer. Signed-off-by: Sakari Ailus ---