drm/i915/gt: Replace unnecessary ',' with '; '
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 22 Jan 2021 19:29:11 +0000 (19:29 +0000)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 24 Mar 2021 18:30:35 +0000 (19:30 +0100)
Checkpatch spotted a couple of commas where we can use the more common
';', and so not worry about the subtle implications of sequence points.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210122192913.4518-8-chris@chris-wilson.co.uk
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/gt/gen6_ppgtt.h
drivers/gpu/drm/i915/gt/intel_region_lmem.c

index 3357228..6a61a5c 100644 (file)
@@ -59,9 +59,9 @@ static inline struct gen6_ppgtt *to_gen6_ppgtt(struct i915_ppgtt *base)
        for (iter = gen6_pde_index(start);                              \
             length > 0 && iter < I915_PDES &&                          \
                     (pt = i915_pt_entry(pd, iter), true);              \
-            ({ u32 temp = ALIGN(start+1, 1 << GEN6_PDE_SHIFT);         \
+            ({ u32 temp = ALIGN(start + 1, 1 << GEN6_PDE_SHIFT);       \
                    temp = min(temp - start, length);                   \
-                   start += temp, length -= temp; }), ++iter)
+                   start += temp; length -= temp; }), ++iter)
 
 #define gen6_for_all_pdes(pt, pd, iter)                                        \
        for (iter = 0;                                                  \
index e326d3c..08bcb15 100644 (file)
@@ -115,7 +115,7 @@ intel_setup_fake_lmem(struct drm_i915_private *i915)
 
        /* Your mappable aperture belongs to me now! */
        mappable_end = pci_resource_len(pdev, 2);
-       io_start = pci_resource_start(pdev, 2),
+       io_start = pci_resource_start(pdev, 2);
        start = i915->params.fake_lmem_start;
 
        mem = intel_memory_region_create(i915,