Re: [PATCH 01/11] drm: Export drm_dev_init() for subclassing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 27 May 2016 at 17:08, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote:
> On Fri, May 27, 2016 at 04:54:46PM +0100, Emil Velikov wrote:
>> On 27 May 2016 at 16:36, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> 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".
Seems like I had a dull moment. Thanks for explaining !

-Emil
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux