[PATCH] drm/xe: Remove unused xe parameter
Upadhyay, Tejas
tejas.upadhyay at intel.com
Tue Aug 13 11:01:27 UTC 2024
> -----Original Message-----
> From: Intel-xe <intel-xe-bounces at lists.freedesktop.org> On Behalf Of Himal
> Prasad Ghimiray
> Sent: Tuesday, August 13, 2024 4:14 PM
> To: intel-xe at lists.freedesktop.org
> Cc: Ghimiray, Himal Prasad <himal.prasad.ghimiray at intel.com>
> Subject: [PATCH] drm/xe: Remove unused xe parameter
>
> Remove the xe parameter from the pde_encode_pat_index and
> pte_encode_pat_index functions, as it is no longer used.
>
> Signed-off-by: Himal Prasad Ghimiray <himal.prasad.ghimiray at intel.com>
If CI-green, LGTM,
Reviewed-by: Tejas Upadhyay <tejas.upadhyay at intel.com>
> ---
> drivers/gpu/drm/xe/xe_vm.c | 17 ++++++-----------
> 1 file changed, 6 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_vm.c b/drivers/gpu/drm/xe/xe_vm.c index
> f225107bdd65..89d86b7cdb02 100644
> --- a/drivers/gpu/drm/xe/xe_vm.c
> +++ b/drivers/gpu/drm/xe/xe_vm.c
> @@ -1191,7 +1191,7 @@ static const struct drm_gpuvm_ops gpuvm_ops = {
> .vm_free = xe_vm_free,
> };
>
> -static u64 pde_encode_pat_index(struct xe_device *xe, u16 pat_index)
> +static u64 pde_encode_pat_index(u16 pat_index)
> {
> u64 pte = 0;
>
> @@ -1204,8 +1204,7 @@ static u64 pde_encode_pat_index(struct xe_device
> *xe, u16 pat_index)
> return pte;
> }
>
> -static u64 pte_encode_pat_index(struct xe_device *xe, u16 pat_index,
> - u32 pt_level)
> +static u64 pte_encode_pat_index(u16 pat_index, u32 pt_level)
> {
> u64 pte = 0;
>
> @@ -1246,12 +1245,11 @@ static u64 pte_encode_ps(u32 pt_level) static
> u64 xelp_pde_encode_bo(struct xe_bo *bo, u64 bo_offset,
> const u16 pat_index)
> {
> - struct xe_device *xe = xe_bo_device(bo);
> u64 pde;
>
> pde = xe_bo_addr(bo, bo_offset, XE_PAGE_SIZE);
> pde |= XE_PAGE_PRESENT | XE_PAGE_RW;
> - pde |= pde_encode_pat_index(xe, pat_index);
> + pde |= pde_encode_pat_index(pat_index);
>
> return pde;
> }
> @@ -1259,12 +1257,11 @@ static u64 xelp_pde_encode_bo(struct xe_bo
> *bo, u64 bo_offset, static u64 xelp_pte_encode_bo(struct xe_bo *bo, u64
> bo_offset,
> u16 pat_index, u32 pt_level)
> {
> - struct xe_device *xe = xe_bo_device(bo);
> u64 pte;
>
> pte = xe_bo_addr(bo, bo_offset, XE_PAGE_SIZE);
> pte |= XE_PAGE_PRESENT | XE_PAGE_RW;
> - pte |= pte_encode_pat_index(xe, pat_index, pt_level);
> + pte |= pte_encode_pat_index(pat_index, pt_level);
> pte |= pte_encode_ps(pt_level);
>
> if (xe_bo_is_vram(bo) || xe_bo_is_stolen_devmem(bo)) @@ -
> 1276,14 +1273,12 @@ static u64 xelp_pte_encode_bo(struct xe_bo *bo, u64
> bo_offset, static u64 xelp_pte_encode_vma(u64 pte, struct xe_vma *vma,
> u16 pat_index, u32 pt_level)
> {
> - struct xe_device *xe = xe_vma_vm(vma)->xe;
> -
> pte |= XE_PAGE_PRESENT;
>
> if (likely(!xe_vma_read_only(vma)))
> pte |= XE_PAGE_RW;
>
> - pte |= pte_encode_pat_index(xe, pat_index, pt_level);
> + pte |= pte_encode_pat_index(pat_index, pt_level);
> pte |= pte_encode_ps(pt_level);
>
> if (unlikely(xe_vma_is_null(vma)))
> @@ -1303,7 +1298,7 @@ static u64 xelp_pte_encode_addr(struct xe_device
> *xe, u64 addr,
>
> pte = addr;
> pte |= XE_PAGE_PRESENT | XE_PAGE_RW;
> - pte |= pte_encode_pat_index(xe, pat_index, pt_level);
> + pte |= pte_encode_pat_index(pat_index, pt_level);
> pte |= pte_encode_ps(pt_level);
>
> if (devmem)
> --
> 2.34.1
More information about the Intel-xe
mailing list