On Wed, Nov 04, 2020 at 02:00:22PM +0100, Christian König wrote: > This is deprecated. > > Signed-off-by: Christian König <christian.koenig@xxxxxxx> Ok you convinced me that amdgpu is safe. Maybe for completeness paste the explanation you put into the other mail into the commit message here (just the entire thing seems fine). Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > index c01c060e4ac5..d6781b479839 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > @@ -1036,8 +1036,8 @@ static int amdgpu_ttm_tt_pin_userptr(struct ttm_bo_device *bdev, > goto release_sg; > > /* convert SG to linear array of pages and dma addresses */ > - drm_prime_sg_to_page_addr_arrays(ttm->sg, ttm->pages, > - gtt->ttm.dma_address, ttm->num_pages); > + drm_prime_sg_to_page_addr_arrays(ttm->sg, NULL, gtt->ttm.dma_address, > + ttm->num_pages); > > return 0; > > @@ -1382,7 +1382,7 @@ static int amdgpu_ttm_tt_populate(struct ttm_bo_device *bdev, > ttm->sg = sgt; > } > > - drm_prime_sg_to_page_addr_arrays(ttm->sg, ttm->pages, > + drm_prime_sg_to_page_addr_arrays(ttm->sg, NULL, > gtt->ttm.dma_address, > ttm->num_pages); > return 0; > -- > 2.25.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel