clk: Simplify clk_is_match()

Linux style is to handle early-on failure.  Inverting the first
condition lets us simplify the second, and improves readability.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Brian Masney <bmasney@redhat.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>

diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
index 47093cda9df32223c1120c3710261296027c4cd3..1a73e9c4e752a85ce34ef2bfa52fbb4179f4f411 100644 (file)
--- a/drivers/clk/clk.c
+++ b/drivers/clk/clk.c
@@ -3259,11 +3259,10 @@ bool clk_is_match(const struct clk *p, const struct clk *q)
                return true;
 
        /* true if clk->core pointers match. Avoid dereferencing garbage */
-       if (!IS_ERR_OR_NULL(p) && !IS_ERR_OR_NULL(q))
-               if (p->core == q->core)
-                       return true;
+       if (IS_ERR_OR_NULL(p) || IS_ERR_OR_NULL(q))
+               return false;
 
-       return false;
+       return p->core == q->core;
 }
 EXPORT_SYMBOL_GPL(clk_is_match);