27 May
2016
27 May
'16
4:08 p.m.
On Fri, May 27, 2016 at 04:54:46PM +0100, Emil Velikov wrote:
On 27 May 2016 at 16:36, Chris Wilson chris@chris-wilson.co.uk wrote:
On Fri, May 27, 2016 at 05:29:43PM +0200, Nils Wallménius wrote:
Hi, Chris
See comment below
err_setunique: if (drm_core_check_feature(dev, DRIVER_GEM)) @@ -653,8 +650,46 @@ err_minors: drm_fs_inode_free(dev->anon_inode); err_free: mutex_destroy(&dev->master_mutex);
kfree(dev);return NULL;
return 0;Shouldn't this return != 0?
You saw nothing, it always said return ret!
Chris can you please elaborate ? I believe Nils has a point here.
I mean, "thanks for bringing that to my attention, oops it was meant to say return ret ofc, fixed". -Chris
--
Chris Wilson, Intel Open Source Technology Centre