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.sh...@linux.intel.com> Reviewed-by: Krzysztof Karas <krzysztof.ka...@intel.com> Signed-off-by: Sebastian Brzezinka <sebastian.brzezi...@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