On 05/08/17 11:51, Tomi Valkeinen wrote:
> Add omap_gem_put_paddr_locked() which is a version of
> omap_gem_put_paddr() that expects the caller to hold the struct_mutex.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxx>
Looks trivial enough.
Reviewed-by: Jyri Sarha <jsarha@xxxxxx>
> ---
> drivers/gpu/drm/omapdrm/omap_gem.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/omapdrm/omap_gem.c b/drivers/gpu/drm/omapdrm/omap_gem.c
> index 68a75b829b71..5d73dccc1383 100644
> --- a/drivers/gpu/drm/omapdrm/omap_gem.c
> +++ b/drivers/gpu/drm/omapdrm/omap_gem.c
> @@ -873,12 +873,12 @@ int omap_gem_get_paddr(struct drm_gem_object *obj,
> /* Release physical address, when DMA is no longer being performed.. this
> * could potentially unpin and unmap buffers from TILER
> */
> -void omap_gem_put_paddr(struct drm_gem_object *obj)
> +
> +static void omap_gem_put_paddr_locked(struct drm_gem_object *obj)
> {
> struct omap_gem_object *omap_obj = to_omap_bo(obj);
> int ret;
>
> - mutex_lock(&obj->dev->struct_mutex);
> if (omap_obj->paddr_cnt > 0) {
> omap_obj->paddr_cnt--;
> if (omap_obj->paddr_cnt == 0) {
> @@ -896,7 +896,12 @@ void omap_gem_put_paddr(struct drm_gem_object *obj)
> omap_obj->block = NULL;
> }
> }
> +}
>
> +void omap_gem_put_paddr(struct drm_gem_object *obj)
> +{
> + mutex_lock(&obj->dev->struct_mutex);
> + omap_gem_put_paddr_locked(obj);
> mutex_unlock(&obj->dev->struct_mutex);
> }
>
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
