From 1f7efe82bdd5677500cc864ee53530d73367afab Mon Sep 17 00:00:00 2001 From: Tom Rini Date: Mon, 8 May 2017 22:14:19 -0400 Subject: [PATCH] socrates: Fix a misleading indentation warning With gcc-6 and later we see a warning about the fact that we have a construct of "if (test);\n\tstatement". Upon reviewing the code, the intention here is as the compiler suggests, we only want to execute the indented statement if the test was true. Cc: Sergei Poselenov Signed-off-by: Tom Rini --- board/socrates/socrates.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/socrates/socrates.c b/board/socrates/socrates.c index 8b34a80e8f..fb691c22d9 100644 --- a/board/socrates/socrates.c +++ b/board/socrates/socrates.c @@ -378,7 +378,7 @@ static void board_backlight_brightness(int br) /* LEDs on */ reg = in_be32((void *)(CONFIG_SYS_FPGA_BASE + 0x0c)); - if (!(reg & BACKLIGHT_ENABLE)); + if (!(reg & BACKLIGHT_ENABLE)) out_be32((void *)(CONFIG_SYS_FPGA_BASE + 0x0c), reg | BACKLIGHT_ENABLE); } else {