Re: [PATCH 52/52] drm: Add docs for managed resources

2020-02-21 Thread Sam Ravnborg
Hi Daniel. > What I miss in all of this is how do other subsystems deal > with the different lifetime of their stuff? > Or maybe only drm really has this issue? > Anything we could learn from others? Reading through the thread - this is all covered in more than sufficient details in other mails. S

Re: [PATCH 52/52] drm: Add docs for managed resources

2020-02-21 Thread Sam Ravnborg
Hi Daniel. In general I think I could follow the documentation, this is good. If the overall design is the best possible I cannot say, but looks sane to me. I like the drmm_ naming as a counterpart to devm_, each handling resources with different lifetimes. What I miss in all of this is how do o

Re: [PATCH 52/52] drm: Add docs for managed resources

2020-02-19 Thread Daniel Vetter
On Wed, Feb 19, 2020 at 4:08 PM Laurent Pinchart wrote: > > Hi Daniel, > > Thank you for the patch. > > On Wed, Feb 19, 2020 at 11:21:22AM +0100, Daniel Vetter wrote: > > All collected together to provide a consistent story in one patch, > > instead of the somewhat bumpy refactor-evolution leading

Re: [PATCH 52/52] drm: Add docs for managed resources

2020-02-19 Thread Laurent Pinchart
Hi Daniel, Thank you for the patch. On Wed, Feb 19, 2020 at 11:21:22AM +0100, Daniel Vetter wrote: > All collected together to provide a consistent story in one patch, > instead of the somewhat bumpy refactor-evolution leading to this. > > Also some thoughts on what the next steps could be: > >

[PATCH 52/52] drm: Add docs for managed resources

2020-02-19 Thread Daniel Vetter
All collected together to provide a consistent story in one patch, instead of the somewhat bumpy refactor-evolution leading to this. Also some thoughts on what the next steps could be: - Create a macro called devm_drm_dev_alloc() which essentially wraps the kzalloc(); devm_drm_dev_init(); drmm_