mbox series

[0/3] drm/vram: Provide GEM VRAM vmap()/vunmap/()

Message ID 20190911120352.20084-1-tzimmermann@suse.de (mailing list archive)
Headers show
Series drm/vram: Provide GEM VRAM vmap()/vunmap/() | expand

Message

Thomas Zimmermann Sept. 11, 2019, 12:03 p.m. UTC
The ast and mgag200 drivers pin() and kmap() cursor buffers; essentially
reimplementing vmap(). We can share some code by using the respective
functionality from GEM VRAM buffer objects.

Thomas Zimmermann (3):
  drm/vram: Provide vmap and vunmap operations for GEM VRAM objects
  drm/ast: Use drm_gem_vram_{vmap,vunmap}() to map cursor source BO
  drm/mgag200: Use drm_gem_vram_{vmap,vunmap}() to map cursor source BO

 drivers/gpu/drm/ast/ast_mode.c           |  21 ++---
 drivers/gpu/drm/drm_gem_vram_helper.c    | 106 ++++++++++++++++-------
 drivers/gpu/drm/mgag200/mgag200_cursor.c |  22 ++---
 include/drm/drm_gem_vram_helper.h        |   5 +-
 4 files changed, 93 insertions(+), 61 deletions(-)

--
2.23.0

Comments

Thomas Zimmermann Sept. 11, 2019, 12:10 p.m. UTC | #1
FYI this is actually version 3 of the patch set posted at
[1] and [2]

[1] https://lists.freedesktop.org/archives/dri-devel/2019-July/227823.html
[2] https://lists.freedesktop.org/archives/dri-devel/2019-July/228074.html

Am 11.09.19 um 14:03 schrieb Thomas Zimmermann:
> The ast and mgag200 drivers pin() and kmap() cursor buffers; essentially
> reimplementing vmap(). We can share some code by using the respective
> functionality from GEM VRAM buffer objects.
> 
> Thomas Zimmermann (3):
>   drm/vram: Provide vmap and vunmap operations for GEM VRAM objects
>   drm/ast: Use drm_gem_vram_{vmap,vunmap}() to map cursor source BO
>   drm/mgag200: Use drm_gem_vram_{vmap,vunmap}() to map cursor source BO
> 
>  drivers/gpu/drm/ast/ast_mode.c           |  21 ++---
>  drivers/gpu/drm/drm_gem_vram_helper.c    | 106 ++++++++++++++++-------
>  drivers/gpu/drm/mgag200/mgag200_cursor.c |  22 ++---
>  include/drm/drm_gem_vram_helper.h        |   5 +-
>  4 files changed, 93 insertions(+), 61 deletions(-)
> 
> --
> 2.23.0
>
Thomas Zimmermann Sept. 17, 2019, 8:12 a.m. UTC | #2
ping for a review

Am 11.09.19 um 14:03 schrieb Thomas Zimmermann:
> The ast and mgag200 drivers pin() and kmap() cursor buffers; essentially
> reimplementing vmap(). We can share some code by using the respective
> functionality from GEM VRAM buffer objects.
> 
> Thomas Zimmermann (3):
>   drm/vram: Provide vmap and vunmap operations for GEM VRAM objects
>   drm/ast: Use drm_gem_vram_{vmap,vunmap}() to map cursor source BO
>   drm/mgag200: Use drm_gem_vram_{vmap,vunmap}() to map cursor source BO
> 
>  drivers/gpu/drm/ast/ast_mode.c           |  21 ++---
>  drivers/gpu/drm/drm_gem_vram_helper.c    | 106 ++++++++++++++++-------
>  drivers/gpu/drm/mgag200/mgag200_cursor.c |  22 ++---
>  include/drm/drm_gem_vram_helper.h        |   5 +-
>  4 files changed, 93 insertions(+), 61 deletions(-)
> 
> --
> 2.23.0
>
Gerd Hoffmann Sept. 17, 2019, 9:55 a.m. UTC | #3
On Wed, Sep 11, 2019 at 02:03:49PM +0200, Thomas Zimmermann wrote:
> The ast and mgag200 drivers pin() and kmap() cursor buffers; essentially
> reimplementing vmap(). We can share some code by using the respective
> functionality from GEM VRAM buffer objects.
> 
> Thomas Zimmermann (3):
>   drm/vram: Provide vmap and vunmap operations for GEM VRAM objects
>   drm/ast: Use drm_gem_vram_{vmap,vunmap}() to map cursor source BO
>   drm/mgag200: Use drm_gem_vram_{vmap,vunmap}() to map cursor source BO
> 
>  drivers/gpu/drm/ast/ast_mode.c           |  21 ++---
>  drivers/gpu/drm/drm_gem_vram_helper.c    | 106 ++++++++++++++++-------
>  drivers/gpu/drm/mgag200/mgag200_cursor.c |  22 ++---
>  include/drm/drm_gem_vram_helper.h        |   5 +-
>  4 files changed, 93 insertions(+), 61 deletions(-)

Series is
Acked-by: Gerd Hoffmann <kraxel@redhat.com>

cheers,
  Gerd