drm/i915: drop cast from DEFINE_RES_MEM() usage
authorJani Nikula <jani.nikula@intel.com>
Mon, 16 Jan 2023 17:34:22 +0000 (19:34 +0200)
committerJani Nikula <jani.nikula@intel.com>
Wed, 18 Jan 2023 09:45:23 +0000 (11:45 +0200)
Since commit 52c4d11f1dce ("resource: Convert DEFINE_RES_NAMED() to be
compound literal") it's no longer necessary to cast DEFINE_RES_MEM() to
struct resource.

This also fixes sparse warnings "cast from non-scalar" and "cast to
non-scalar".

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230116173422.1858527-2-jani.nikula@intel.com
drivers/gpu/drm/i915/gem/i915_gem_stolen.c
drivers/gpu/drm/i915/gt/intel_ggtt.c
drivers/gpu/drm/i915/gt/intel_ggtt_gmch.c
drivers/gpu/drm/i915/intel_memory_region.c
drivers/gpu/drm/i915/selftests/mock_gtt.c

index de87349..90a9673 100644 (file)
@@ -110,9 +110,7 @@ static int adjust_stolen(struct drm_i915_private *i915,
                else
                        ggtt_start &= PGTBL_ADDRESS_LO_MASK;
 
-               ggtt_res =
-                       (struct resource) DEFINE_RES_MEM(ggtt_start,
-                                                        ggtt_total_entries(ggtt) * 4);
+               ggtt_res = DEFINE_RES_MEM(ggtt_start, ggtt_total_entries(ggtt) * 4);
 
                if (ggtt_res.start >= stolen[0].start && ggtt_res.start < stolen[0].end)
                        stolen[0].end = ggtt_res.start;
@@ -471,8 +469,7 @@ static int init_reserved_stolen(struct drm_i915_private *i915)
                goto bail_out;
        }
 
-       i915->dsm.reserved =
-               (struct resource)DEFINE_RES_MEM(reserved_base, reserved_size);
+       i915->dsm.reserved = DEFINE_RES_MEM(reserved_base, reserved_size);
 
        if (!resource_contains(&i915->dsm.stolen, &i915->dsm.reserved)) {
                drm_err(&i915->drm,
@@ -485,8 +482,7 @@ static int init_reserved_stolen(struct drm_i915_private *i915)
        return 0;
 
 bail_out:
-       i915->dsm.reserved =
-               (struct resource)DEFINE_RES_MEM(reserved_base, 0);
+       i915->dsm.reserved = DEFINE_RES_MEM(reserved_base, 0);
 
        return ret;
 }
index 8145851..2e75e3c 100644 (file)
@@ -920,8 +920,8 @@ static void gen6_gmch_remove(struct i915_address_space *vm)
 
 static struct resource pci_resource(struct pci_dev *pdev, int bar)
 {
-       return (struct resource)DEFINE_RES_MEM(pci_resource_start(pdev, bar),
-                                              pci_resource_len(pdev, bar));
+       return DEFINE_RES_MEM(pci_resource_start(pdev, bar),
+                             pci_resource_len(pdev, bar));
 }
 
 static int gen8_gmch_probe(struct i915_ggtt *ggtt)
index 4e2163a..4192d06 100644 (file)
@@ -89,8 +89,7 @@ int intel_ggtt_gmch_probe(struct i915_ggtt *ggtt)
 
        intel_gmch_gtt_get(&ggtt->vm.total, &gmadr_base, &ggtt->mappable_end);
 
-       ggtt->gmadr =
-               (struct resource)DEFINE_RES_MEM(gmadr_base, ggtt->mappable_end);
+       ggtt->gmadr = DEFINE_RES_MEM(gmadr_base, ggtt->mappable_end);
 
        ggtt->vm.alloc_pt_dma = alloc_pt_dma;
        ggtt->vm.alloc_scratch_dma = alloc_pt_dma;
index b9a164e..3d1fdea 100644 (file)
@@ -235,7 +235,7 @@ intel_memory_region_create(struct drm_i915_private *i915,
                return ERR_PTR(-ENOMEM);
 
        mem->i915 = i915;
-       mem->region = (struct resource)DEFINE_RES_MEM(start, size);
+       mem->region = DEFINE_RES_MEM(start, size);
        mem->io_start = io_start;
        mem->io_size = io_size;
        mem->min_page_size = min_page_size;
index 568840e..ece97e4 100644 (file)
@@ -112,7 +112,7 @@ void mock_init_ggtt(struct intel_gt *gt)
        ggtt->vm.i915 = gt->i915;
        ggtt->vm.is_ggtt = true;
 
-       ggtt->gmadr = (struct resource) DEFINE_RES_MEM(0, 2048 * PAGE_SIZE);
+       ggtt->gmadr = DEFINE_RES_MEM(0, 2048 * PAGE_SIZE);
        ggtt->mappable_end = resource_size(&ggtt->gmadr);
        ggtt->vm.total = 4096 * PAGE_SIZE;