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

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

 



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".
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux