[PATCH 2/2] drm: omapdrm: Perform initialization/cleanup at probe/remove time

2016-12-14 Thread Laurent Pinchart
Hi Daniel, On Tuesday 13 Dec 2016 22:48:50 Daniel Vetter wrote: > On Tue, Dec 13, 2016 at 11:41:09PM +0200, Laurent Pinchart wrote: > > On Tuesday 13 Dec 2016 22:21:10 Daniel Vetter wrote: > >> On Tue, Dec 13, 2016 at 05:21:43PM +0200, Laurent Pinchart wrote: > >>> + > >>> + /* Allocate and initia

[PATCH 2/2] drm: omapdrm: Perform initialization/cleanup at probe/remove time

2016-12-13 Thread Laurent Pinchart
Hi Daniel, On Tuesday 13 Dec 2016 22:21:10 Daniel Vetter wrote: > On Tue, Dec 13, 2016 at 05:21:43PM +0200, Laurent Pinchart wrote: > > The drm driver .load() operation is prone to race conditions as it > > initializes the driver after registering the device nodes. Its usage is > > deprecated, inl

[PATCH 2/2] drm: omapdrm: Perform initialization/cleanup at probe/remove time

2016-12-13 Thread Daniel Vetter
On Tue, Dec 13, 2016 at 11:41:09PM +0200, Laurent Pinchart wrote: > On Tuesday 13 Dec 2016 22:21:10 Daniel Vetter wrote: > > On Tue, Dec 13, 2016 at 05:21:43PM +0200, Laurent Pinchart wrote: > > > + > > > + /* Allocate and initialize the DRM device. */ > > > + ddev = drm_dev_alloc(&omap_drm_driver,

[PATCH 2/2] drm: omapdrm: Perform initialization/cleanup at probe/remove time

2016-12-13 Thread Daniel Vetter
On Tue, Dec 13, 2016 at 05:21:43PM +0200, Laurent Pinchart wrote: > The drm driver .load() operation is prone to race conditions as it > initializes the driver after registering the device nodes. Its usage is > deprecated, inline it in the probe function and call drm_dev_alloc() and > drm_dev_regis

[PATCH 2/2] drm: omapdrm: Perform initialization/cleanup at probe/remove time

2016-12-13 Thread Laurent Pinchart
The drm driver .load() operation is prone to race conditions as it initializes the driver after registering the device nodes. Its usage is deprecated, inline it in the probe function and call drm_dev_alloc() and drm_dev_register() explicitly. For consistency inline the .unload() handler in the rem