20. 6. 19. 오후 7:36에 Marek Szyprowski 이(가) 쓴 글:
> Use common helper for checking the contiguity of the imported dma-buf.
> 
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>

Acked-by : Inki Dae <inki....@samsung.com>

Thanks,
Inki Dae

> ---
>  drivers/gpu/drm/exynos/exynos_drm_gem.c | 23 +++--------------------
>  1 file changed, 3 insertions(+), 20 deletions(-)
> 
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_gem.c 
> b/drivers/gpu/drm/exynos/exynos_drm_gem.c
> index efa476858db5..1716a023bca0 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_gem.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_gem.c
> @@ -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);
> 

Reply via email to