staging: gasket: remove else clause after return in if clause
authorTodd Poynor <toddpoynor@google.com>
Fri, 20 Jul 2018 03:49:10 +0000 (20:49 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 21 Jul 2018 06:50:35 +0000 (08:50 +0200)
Else after return is unnecessary and may cause static code checkers to
complain.

Reported-by: Guenter Roeck <groeck@chromium.org>
Signed-off-by: Simon Que <sque@chromium.org>
Signed-off-by: Todd Poynor <toddpoynor@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/gasket/gasket_page_table.c

index 2a27db6..617d602 100644 (file)
@@ -598,9 +598,7 @@ bool gasket_page_table_is_dev_addr_bad(
        if (gasket_addr_is_simple(pg_tbl, dev_addr))
                return gasket_is_simple_dev_addr_bad(
                        pg_tbl, dev_addr, num_pages);
-       else
-               return gasket_is_extended_dev_addr_bad(
-                       pg_tbl, dev_addr, num_pages);
+       return gasket_is_extended_dev_addr_bad(pg_tbl, dev_addr, num_pages);
 }
 EXPORT_SYMBOL(gasket_page_table_is_dev_addr_bad);