drm: exynos: use common helper for a scatterlist contiguity check
authorMarek Szyprowski <m.szyprowski@samsung.com>
Thu, 30 Apr 2020 12:02:25 +0000 (14:02 +0200)
committerMarek Szyprowski <m.szyprowski@samsung.com>
Thu, 10 Sep 2020 06:18:35 +0000 (08:18 +0200)
Use common helper for checking the contiguity of the imported dma-buf.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
Acked-by : Inki Dae <inki.dae@samsung.com>

drivers/gpu/drm/exynos/exynos_drm_gem.c

index efa4768..1716a02 100644 (file)
@@ -431,27 +431,10 @@ exynos_drm_gem_prime_import_sg_table(struct drm_device *dev,
 {
        struct exynos_drm_gem *exynos_gem;
 
-       if (sgt->nents < 1)
+       /* check if the entries in the sg_table are contiguous */
+       if (drm_prime_get_contiguous_size(sgt) < attach->dmabuf->size) {
+               DRM_ERROR("buffer chunks must be mapped contiguously");
                return ERR_PTR(-EINVAL);
-
-       /*
-        * Check if the provided buffer has been mapped as contiguous
-        * into DMA address space.
-        */
-       if (sgt->nents > 1) {
-               dma_addr_t next_addr = sg_dma_address(sgt->sgl);
-               struct scatterlist *s;
-               unsigned int i;
-
-               for_each_sg(sgt->sgl, s, sgt->nents, i) {
-                       if (!sg_dma_len(s))
-                               break;
-                       if (sg_dma_address(s) != next_addr) {
-                               DRM_ERROR("buffer chunks must be mapped contiguously");
-                               return ERR_PTR(-EINVAL);
-                       }
-                       next_addr = sg_dma_address(s) + sg_dma_len(s);
-               }
        }
 
        exynos_gem = exynos_drm_gem_init(dev, attach->dmabuf->size);