diff options
| author | Jeroen Hofstee <jeroen@myspectrum.nl> | 2014-06-18 20:59:48 (GMT) |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2014-07-07 23:43:01 (GMT) |
| commit | 49f9337956af6d52470c8082c8e7088b0c030334 (patch) | |
| tree | 3f9f42db007e23be18c4bb12ec1d969c0a5a781b | |
| parent | 6275331d41cc80d93ca00d12e8c694b9d9841155 (diff) | |
| download | u-boot-fsl-qoriq-49f9337956af6d52470c8082c8e7088b0c030334.tar.xz | |
tpm: don't use unneeded double brackets
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>
| -rw-r--r-- | drivers/tpm/tpm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/tpm/tpm.c b/drivers/tpm/tpm.c index b657334..bc0f964 100644 --- a/drivers/tpm/tpm.c +++ b/drivers/tpm/tpm.c @@ -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; |