diff options
| author | Tom Rini <trini@konsulko.com> | 2017-05-08 22:14:19 -0400 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2017-05-12 08:37:22 -0400 |
| commit | 1f7efe82bdd5677500cc864ee53530d73367afab (patch) | |
| tree | a37a551071d3b771f3d3b5c88640504a83273a8e /board/socrates | |
| parent | c33e825ac35bd13a29665b7bb32d00daad2bd59e (diff) | |
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 <sposelenov@emcraft.com>
Signed-off-by: Tom Rini <trini@konsulko.com>
Diffstat (limited to 'board/socrates')
| -rw-r--r-- | board/socrates/socrates.c | 2 |
1 files changed, 1 insertions, 1 deletions
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 { |