selftests: riscv: Add braces around EXPECT_EQ()

EXPECT_EQ() expands to multiple lines, breaking up one-line if
statements. This issue was not present in the patch on the mailing list
but was instead introduced by the maintainer when attempting to fix up
checkpatch warnings. Add braces around EXPECT_EQ() to avoid the error
even though checkpatch suggests them to be removed:

validate_v_ptrace.c:626:17: error: ‘else’ without a previous ‘if’

Fixes: 3789d5eecd5a ("selftests: riscv: verify syscalls discard vector context")
Fixes: 30eb191c895b ("selftests: riscv: verify ptrace rejects invalid vector csr inputs")
Fixes: 849f05ae1ea6 ("selftests: riscv: verify ptrace accepts valid vector csr values")
Signed-off-by: Charlie Jenkins <thecharlesjenkins@gmail.com>
Reviewed-and-tested-by: Sergey Matyukevich <geomatsi@gmail.com>
Link: https://patch.msgid.link/20260309-fix_selftests-v2-2-9d5a553a531e@gmail.com
Signed-off-by: Paul Walmsley <pjw@kernel.org>

diff --git a/tools/testing/selftests/riscv/vector/validate_v_ptrace.c b/tools/testing/selftests/riscv/vector/validate_v_ptrace.c
index 7ae6fed..74b6f6b 100644 (file)
--- a/tools/testing/selftests/riscv/vector/validate_v_ptrace.c
+++ b/tools/testing/selftests/riscv/vector/validate_v_ptrace.c
@@ -290,10 +290,11 @@ TEST(ptrace_v_syscall_clobbering)
 
                /* verify initial vsetvli settings */
 
-               if (is_xtheadvector_supported())
+               if (is_xtheadvector_supported()) {
                        EXPECT_EQ(5UL, regset_data->vtype);
-               else
+               } else {
                        EXPECT_EQ(9UL, regset_data->vtype);
+               }
 
                EXPECT_EQ(regset_data->vlenb, regset_data->vl);
                EXPECT_EQ(vlenb, regset_data->vlenb);
@@ -619,10 +620,11 @@ TEST_F(v_csr_invalid, ptrace_v_invalid_values)
 
                /* verify initial vsetvli settings */
 
-               if (is_xtheadvector_supported())
+               if (is_xtheadvector_supported()) {
                        EXPECT_EQ(5UL, regset_data->vtype);
-               else
+               } else {
                        EXPECT_EQ(9UL, regset_data->vtype);
+               }
 
                EXPECT_EQ(regset_data->vlenb, regset_data->vl);
                EXPECT_EQ(vlenb, regset_data->vlenb);
@@ -827,10 +829,11 @@ TEST_F(v_csr_valid, ptrace_v_valid_values)
 
                /* verify initial vsetvli settings */
 
-               if (is_xtheadvector_supported())
+               if (is_xtheadvector_supported()) {
                        EXPECT_EQ(5UL, regset_data->vtype);
-               else
+               } else {
                        EXPECT_EQ(9UL, regset_data->vtype);
+               }
 
                EXPECT_EQ(regset_data->vlenb, regset_data->vl);
                EXPECT_EQ(vlenb, regset_data->vlenb);