On 09/11/2018 05:56 PM, Christian König wrote: > They aren't directly used by the hardware. > > Signed-off-by: Christian König <christian.koenig at amd.com> Reviewed-by: Junwei Zhang <Jerry.Zhang at amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c > index 7db0040ca145..3a6f92de5504 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c > @@ -516,7 +516,7 @@ static int amdgpu_bo_do_create(struct amdgpu_device *adev, > } > > static int amdgpu_bo_create_shadow(struct amdgpu_device *adev, > - unsigned long size, int byte_align, > + unsigned long size, > struct amdgpu_bo *bo) > { > struct amdgpu_bo_param bp; > @@ -527,7 +527,6 @@ static int amdgpu_bo_create_shadow(struct amdgpu_device *adev, > > memset(&bp, 0, sizeof(bp)); > bp.size = size; > - bp.byte_align = byte_align; > bp.domain = AMDGPU_GEM_DOMAIN_GTT; > bp.flags = AMDGPU_GEM_CREATE_CPU_GTT_USWC | > AMDGPU_GEM_CREATE_SHADOW; > @@ -576,7 +575,7 @@ int amdgpu_bo_create(struct amdgpu_device *adev, > WARN_ON(reservation_object_lock((*bo_ptr)->tbo.resv, > NULL)); > > - r = amdgpu_bo_create_shadow(adev, bp->size, bp->byte_align, (*bo_ptr)); > + r = amdgpu_bo_create_shadow(adev, bp->size, *bo_ptr); > > if (!bp->resv) > reservation_object_unlock((*bo_ptr)->tbo.resv);