diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2021-10-05 15:29:35 -0700 |
---|---|---|
committer | Ashutosh Dixit <ashutosh.dixit@intel.com> | 2021-10-08 19:16:05 -0700 |
commit | 426723f979380f18f9c07d36ebac3a52f760ba7e (patch) | |
tree | eefda917ee9bb1bf8b7aefd8c607f4a0d8f4eabb /lib/i915/intel_memory_region.c | |
parent | d5d99d16817eca9ce8d191b6e84c1a1fe2ad4428 (diff) |
lib: Typechecking minmax
Add typechecking to the min/max macros and make their locals truly
unique-ish to reduce the risk of shadowing.
v2: small bug fix, write also height coordinate on rotation
test. (jheikkil)
v3: Fix up a couple of other max/max_t instances (Ashutosh)
Signed-off-by: Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com>
Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Signed-off-by: Chris Wilson <chris.p.wilson@intel.com>
Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Diffstat (limited to 'lib/i915/intel_memory_region.c')
-rw-r--r-- | lib/i915/intel_memory_region.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/lib/i915/intel_memory_region.c b/lib/i915/intel_memory_region.c index bc2f66db..058b273d 100644 --- a/lib/i915/intel_memory_region.c +++ b/lib/i915/intel_memory_region.c @@ -146,19 +146,16 @@ out: * * Returns: Number of found lmem regions. */ -uint8_t gem_get_lmem_region_count(int fd) +unsigned int gem_get_lmem_region_count(int fd) { struct drm_i915_query_memory_regions *query_info; - uint8_t num_regions; - uint8_t lmem_regions = 0; + unsigned int lmem_regions = 0; query_info = gem_get_query_memory_regions(fd); if (!query_info) goto out; - num_regions = query_info->num_regions; - - for (int i = 0; i < num_regions; i++) { + for (unsigned int i = 0; i < query_info->num_regions; i++) { if (query_info->regions[i].region.memory_class == I915_MEMORY_CLASS_DEVICE) lmem_regions += 1; } |