[PATCH v3 1/4] drm/i915: Add braces around the else block in clflush_write32()
Sebastian Brzezinka
sebastian.brzezinka at intel.com
Fri Jul 18 10:28:04 UTC 2025
According to the kernel coding style, if only one branch of a
conditional statement is a single statement, braces should
still be used in both branches.
Reviewed-by: Andi Shyti <andi.shyti at linux.intel.com>
Reviewed-by: Krzysztof Karas <krzysztof.karas at intel.com>
Signed-off-by: Sebastian Brzezinka <sebastian.brzezinka at intel.com>
---
v2 -> v3:
- Nothing has changed
---
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c
index ca7e9216934a..0801d4a140e3 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c
@@ -1382,8 +1382,9 @@ static void clflush_write32(u32 *addr, u32 value, unsigned int flushes)
*/
if (flushes & CLFLUSH_AFTER)
drm_clflush_virt_range(addr, sizeof(*addr));
- } else
+ } else {
*addr = value;
+ }
}
static u64
--
2.34.1
More information about the Intel-gfx
mailing list