Re: [PATCH v3 2/2] drm/lima: driver for ARM Mali4xx GPUs

2019-03-05 Thread Qiang Yu
On Wed, Mar 6, 2019 at 4:18 AM Eric Anholt wrote: > > Rob Herring writes: > > > On Fri, Mar 1, 2019 at 11:23 PM Qiang Yu wrote: > >> > >> > > +static struct lima_fence *lima_fence_create(struct lima_sched_pipe > >> > > *pipe) > >> > > +{ > >> > > + struct lima_fence *fence; > >> > > + >

Re: [PATCH v3 2/2] drm/lima: driver for ARM Mali4xx GPUs

2019-03-05 Thread Eric Anholt
Rob Herring writes: > On Fri, Mar 1, 2019 at 11:23 PM Qiang Yu wrote: >> >> > > +static struct lima_fence *lima_fence_create(struct lima_sched_pipe >> > > *pipe) >> > > +{ >> > > + struct lima_fence *fence; >> > > + >> > > + fence = kmem_cache_zalloc(lima_fence_slab, GFP_KERNEL);

Re: [PATCH v3 2/2] drm/lima: driver for ARM Mali4xx GPUs

2019-03-05 Thread Rob Herring
On Fri, Mar 1, 2019 at 11:23 PM Qiang Yu wrote: > > > > +static struct lima_fence *lima_fence_create(struct lima_sched_pipe *pipe) > > > +{ > > > + struct lima_fence *fence; > > > + > > > + fence = kmem_cache_zalloc(lima_fence_slab, GFP_KERNEL); > > > > Out of curiosity, what is the

Re: [PATCH v3 2/2] drm/lima: driver for ARM Mali4xx GPUs

2019-03-04 Thread Qiang Yu
On Tue, Mar 5, 2019 at 1:20 AM Rob Herring wrote: > > On Sat, Mar 2, 2019 at 12:23 PM Rob Clark wrote: > > > > On Fri, Mar 1, 2019 at 9:32 PM Qiang Yu wrote: > > > > > > On Thu, Feb 28, 2019 at 5:41 AM Rob Herring wrote: > > > > > > > > On Wed, Feb 27, 2019 at 7:42 AM Qiang Yu wrote: > > > >

Re: [PATCH v3 2/2] drm/lima: driver for ARM Mali4xx GPUs

2019-03-04 Thread Rob Herring
On Sat, Mar 2, 2019 at 12:23 PM Rob Clark wrote: > > On Fri, Mar 1, 2019 at 9:32 PM Qiang Yu wrote: > > > > On Thu, Feb 28, 2019 at 5:41 AM Rob Herring wrote: > > > > > > On Wed, Feb 27, 2019 at 7:42 AM Qiang Yu wrote: > > > > diff --git a/drivers/gpu/drm/lima/lima_drv.c > > > >

Re: [PATCH v3 2/2] drm/lima: driver for ARM Mali4xx GPUs

2019-03-02 Thread Rob Clark
On Fri, Mar 1, 2019 at 9:32 PM Qiang Yu wrote: > > On Thu, Feb 28, 2019 at 5:41 AM Rob Herring wrote: > > > > On Wed, Feb 27, 2019 at 7:42 AM Qiang Yu wrote: > > > diff --git a/drivers/gpu/drm/lima/lima_drv.c > > > b/drivers/gpu/drm/lima/lima_drv.c> > > new file mode 100644 > > > index

Re: [PATCH v3 2/2] drm/lima: driver for ARM Mali4xx GPUs

2019-03-01 Thread Qiang Yu
> > +static struct lima_fence *lima_fence_create(struct lima_sched_pipe *pipe) > > +{ > > + struct lima_fence *fence; > > + > > + fence = kmem_cache_zalloc(lima_fence_slab, GFP_KERNEL); > > Out of curiosity, what is the benefit of using a separate slab here? > If this is beneficial,

Re: [PATCH v3 2/2] drm/lima: driver for ARM Mali4xx GPUs

2019-02-27 Thread Sam Ravnborg
Hi Qiang. Two general comments: - We are trying to avoid drmP.h in new drivers, please drop - The use of idr is being replaced with XArray, see patch-set posted by Matthew Wilcox. Try to use XArray so we do not introduce a new user. This was from a very quick looks at the driver. In general