mbox series

[v4,0/3] drm/lima: simplify driver by using more drm helpers

Message ID 20191010140152.17747-1-yuq825@gmail.com (mailing list archive)
Headers show
Series drm/lima: simplify driver by using more drm helpers | expand

Message

Qiang Yu Oct. 10, 2019, 2:01 p.m. UTC
By using shared drm helpers:
1. drm_gem_(un)lock_reservations
2. drm_gem_shmem_helpers
we can simplify lima driver a lot and benifit from updates to
these functions.

Patch series is based on drm-misc-next branch

v2:
Add drm_gem_objects_lookup_user and use it for driver which
pass user GEM handles in contious array.

v3:
improve commit comment.

v4:
Drop drm_gem_objects_lookup refine patches.

Qiang Yu (3):
  drm/lima: use drm_gem_shmem_helpers
  drm/lima: use drm_gem_(un)lock_reservations
  drm/lima: add __GFP_NOWARN flag to all dma_alloc_wc

 drivers/gpu/drm/lima/Kconfig          |   1 +
 drivers/gpu/drm/lima/Makefile         |   4 +-
 drivers/gpu/drm/lima/lima_device.c    |   2 +-
 drivers/gpu/drm/lima/lima_drv.c       |  22 +--
 drivers/gpu/drm/lima/lima_gem.c       | 195 ++++++++++----------------
 drivers/gpu/drm/lima/lima_gem.h       |  32 ++++-
 drivers/gpu/drm/lima/lima_gem_prime.c |  46 ------
 drivers/gpu/drm/lima/lima_gem_prime.h |  13 --
 drivers/gpu/drm/lima/lima_mmu.c       |   1 -
 drivers/gpu/drm/lima/lima_object.c    | 119 ----------------
 drivers/gpu/drm/lima/lima_object.h    |  35 -----
 drivers/gpu/drm/lima/lima_sched.c     |   6 +-
 drivers/gpu/drm/lima/lima_vm.c        |  87 ++++++------
 13 files changed, 148 insertions(+), 415 deletions(-)
 delete mode 100644 drivers/gpu/drm/lima/lima_gem_prime.c
 delete mode 100644 drivers/gpu/drm/lima/lima_gem_prime.h
 delete mode 100644 drivers/gpu/drm/lima/lima_object.c
 delete mode 100644 drivers/gpu/drm/lima/lima_object.h

Comments

Vasily Khoruzhick Oct. 14, 2019, 4:21 a.m. UTC | #1
On Thu, Oct 10, 2019 at 7:02 AM Qiang Yu <yuq825@gmail.com> wrote:
>
> By using shared drm helpers:
> 1. drm_gem_(un)lock_reservations
> 2. drm_gem_shmem_helpers
> we can simplify lima driver a lot and benifit from updates to
> these functions.
>
> Patch series is based on drm-misc-next branch
>
> v2:
> Add drm_gem_objects_lookup_user and use it for driver which
> pass user GEM handles in contious array.
>
> v3:
> improve commit comment.
>
> v4:
> Drop drm_gem_objects_lookup refine patches.
>
> Qiang Yu (3):
>   drm/lima: use drm_gem_shmem_helpers
>   drm/lima: use drm_gem_(un)lock_reservations
>   drm/lima: add __GFP_NOWARN flag to all dma_alloc_wc

LGTM, whole series:

Reviewed-by: Vasily Khoruzhick <anarsoul@gmail.com>

>  drivers/gpu/drm/lima/Kconfig          |   1 +
>  drivers/gpu/drm/lima/Makefile         |   4 +-
>  drivers/gpu/drm/lima/lima_device.c    |   2 +-
>  drivers/gpu/drm/lima/lima_drv.c       |  22 +--
>  drivers/gpu/drm/lima/lima_gem.c       | 195 ++++++++++----------------
>  drivers/gpu/drm/lima/lima_gem.h       |  32 ++++-
>  drivers/gpu/drm/lima/lima_gem_prime.c |  46 ------
>  drivers/gpu/drm/lima/lima_gem_prime.h |  13 --
>  drivers/gpu/drm/lima/lima_mmu.c       |   1 -
>  drivers/gpu/drm/lima/lima_object.c    | 119 ----------------
>  drivers/gpu/drm/lima/lima_object.h    |  35 -----
>  drivers/gpu/drm/lima/lima_sched.c     |   6 +-
>  drivers/gpu/drm/lima/lima_vm.c        |  87 ++++++------
>  13 files changed, 148 insertions(+), 415 deletions(-)
>  delete mode 100644 drivers/gpu/drm/lima/lima_gem_prime.c
>  delete mode 100644 drivers/gpu/drm/lima/lima_gem_prime.h
>  delete mode 100644 drivers/gpu/drm/lima/lima_object.c
>  delete mode 100644 drivers/gpu/drm/lima/lima_object.h
>
> --
> 2.17.1
>
> _______________________________________________
> lima mailing list
> lima@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/lima
Qiang Yu Oct. 17, 2019, 3:51 p.m. UTC | #2
Thanks, applied to drm-misc-next.

Regards,
Qiang

On Mon, Oct 14, 2019 at 12:21 PM Vasily Khoruzhick <anarsoul@gmail.com> wrote:
>
> On Thu, Oct 10, 2019 at 7:02 AM Qiang Yu <yuq825@gmail.com> wrote:
> >
> > By using shared drm helpers:
> > 1. drm_gem_(un)lock_reservations
> > 2. drm_gem_shmem_helpers
> > we can simplify lima driver a lot and benifit from updates to
> > these functions.
> >
> > Patch series is based on drm-misc-next branch
> >
> > v2:
> > Add drm_gem_objects_lookup_user and use it for driver which
> > pass user GEM handles in contious array.
> >
> > v3:
> > improve commit comment.
> >
> > v4:
> > Drop drm_gem_objects_lookup refine patches.
> >
> > Qiang Yu (3):
> >   drm/lima: use drm_gem_shmem_helpers
> >   drm/lima: use drm_gem_(un)lock_reservations
> >   drm/lima: add __GFP_NOWARN flag to all dma_alloc_wc
>
> LGTM, whole series:
>
> Reviewed-by: Vasily Khoruzhick <anarsoul@gmail.com>
>
> >  drivers/gpu/drm/lima/Kconfig          |   1 +
> >  drivers/gpu/drm/lima/Makefile         |   4 +-
> >  drivers/gpu/drm/lima/lima_device.c    |   2 +-
> >  drivers/gpu/drm/lima/lima_drv.c       |  22 +--
> >  drivers/gpu/drm/lima/lima_gem.c       | 195 ++++++++++----------------
> >  drivers/gpu/drm/lima/lima_gem.h       |  32 ++++-
> >  drivers/gpu/drm/lima/lima_gem_prime.c |  46 ------
> >  drivers/gpu/drm/lima/lima_gem_prime.h |  13 --
> >  drivers/gpu/drm/lima/lima_mmu.c       |   1 -
> >  drivers/gpu/drm/lima/lima_object.c    | 119 ----------------
> >  drivers/gpu/drm/lima/lima_object.h    |  35 -----
> >  drivers/gpu/drm/lima/lima_sched.c     |   6 +-
> >  drivers/gpu/drm/lima/lima_vm.c        |  87 ++++++------
> >  13 files changed, 148 insertions(+), 415 deletions(-)
> >  delete mode 100644 drivers/gpu/drm/lima/lima_gem_prime.c
> >  delete mode 100644 drivers/gpu/drm/lima/lima_gem_prime.h
> >  delete mode 100644 drivers/gpu/drm/lima/lima_object.c
> >  delete mode 100644 drivers/gpu/drm/lima/lima_object.h
> >
> > --
> > 2.17.1
> >
> > _______________________________________________
> > lima mailing list
> > lima@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/lima