[PATCH 2/3] drm/i915: Add braces around the else block in clflush_write32()

Sebastian Brzezinka sebastian.brzezinka at intel.com
Wed Jul 16 14:07:24 UTC 2025


Hi Krzysztof

On Wed Jul 16, 2025 at 1:47 PM UTC, Krzysztof Karas wrote:
> Hi Sebastian,
>
> I think this patch could be dropped. The brackets do not improve
> readability in this case, because we deal with only a single
> line of code in the "else" block.

From: https://www.kernel.org/doc/html/v4.10/process/coding-style.html

>This does not apply if only one branch of a conditional statement is a
> single statement; in the latter case use braces in both branches:

-- 
Best regards,
Sebastian



More information about the Intel-gfx mailing list