tpm: don't use unneeded double brackets
authorJeroen Hofstee <jeroen@myspectrum.nl>
Wed, 18 Jun 2014 20:59:48 +0000 (22:59 +0200)
committerTom Rini <trini@ti.com>
Mon, 7 Jul 2014 23:43:01 +0000 (19:43 -0400)
clang is tempted to inteprete such a condition as a assignment
as well. Since it isn't don't use double brackets.

cc: Tom Wai-Hong Tam <waihong@chromium.org>
Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
drivers/tpm/tpm.c

index b6573341956d6d0d4ffc76239b1bf71da550a8f9..bc0f9645b591e9460a74ccdfee947b49338684ba 100644 (file)
@@ -411,7 +411,7 @@ static ssize_t tpm_transmit(const unsigned char *buf, size_t bufsiz)
                        goto out_recv;
                }
 
-               if ((status == chip->vendor.req_canceled)) {
+               if (status == chip->vendor.req_canceled) {
                        error("Operation Canceled\n");
                        rc = -ECANCELED;
                        goto out;