[PATCH 2/2] drm: Don't swallow error codes in drm_dev_alloc()

2016-09-22 Thread Tom Gundersen
On Thu, Sep 22, 2016 at 11:47 AM, Eric Engestrom wrote: > On Wed, Sep 21, 2016 at 04:59:19PM +0200, Tom Gundersen wrote: >> There are many reasons other than ENOMEM that drm_dev_init() can >> fail. Return ERR_PTR rather than NULL to be able to distinguish >> these in the caller. >> >> Signed-off-b

[PATCH 2/2] drm: Don't swallow error codes in drm_dev_alloc()

2016-09-22 Thread Eric Engestrom
On Wed, Sep 21, 2016 at 04:59:19PM +0200, Tom Gundersen wrote: > There are many reasons other than ENOMEM that drm_dev_init() can > fail. Return ERR_PTR rather than NULL to be able to distinguish > these in the caller. > > Signed-off-by: Tom Gundersen Looks good to me :) Assuming you fixed all t

[PATCH 2/2] drm: Don't swallow error codes in drm_dev_alloc()

2016-09-22 Thread David Herrmann
Hey On Wed, Sep 21, 2016 at 4:59 PM, Tom Gundersen wrote: > There are many reasons other than ENOMEM that drm_dev_init() can > fail. Return ERR_PTR rather than NULL to be able to distinguish > these in the caller. > > Signed-off-by: Tom Gundersen > --- Reviewed-by: David Herrmann Thanks David

[PATCH 2/2] drm: Don't swallow error codes in drm_dev_alloc()

2016-09-22 Thread Daniel Vetter
On Wed, Sep 21, 2016 at 04:59:19PM +0200, Tom Gundersen wrote: > There are many reasons other than ENOMEM that drm_dev_init() can > fail. Return ERR_PTR rather than NULL to be able to distinguish > these in the caller. > > Signed-off-by: Tom Gundersen Reviewed-by: Daniel Vetter > --- > driver

[PATCH 2/2] drm: Don't swallow error codes in drm_dev_alloc()

2016-09-22 Thread Sean Paul
On Wed, Sep 21, 2016 at 7:59 AM, Tom Gundersen wrote: > There are many reasons other than ENOMEM that drm_dev_init() can > fail. Return ERR_PTR rather than NULL to be able to distinguish > these in the caller. > > Signed-off-by: Tom Gundersen Thanks, applied to -misc Sean > --- > drivers/gpu

[PATCH 2/2] drm: Don't swallow error codes in drm_dev_alloc()

2016-09-21 Thread Tom Gundersen
There are many reasons other than ENOMEM that drm_dev_init() can fail. Return ERR_PTR rather than NULL to be able to distinguish these in the caller. Signed-off-by: Tom Gundersen --- drivers/gpu/drm/arc/arcpgu_drv.c| 4 ++-- drivers/gpu/drm/arm/hdlcd_drv.c | 4 ++-