summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Anholt <[email protected]>2017-12-14 09:41:16 -0800
committerEric Anholt <[email protected]>2017-12-14 14:36:24 -0800
commit49e2586bfc7f806038bba7864dcff0c50fed1fe9 (patch)
tree1de25437914666d19caef19697e7ccc984c14dde
parent1171f1749ddf7fdff0725804868bb375f2407526 (diff)
broadcom/vc5: Fix a typo in memcmp for sig unpack checking.
This shockingly ended up working out, because only the first byte of *sig is used and (sizeof(*sig) != 0) == 1. Fixes a compiler warning. Link: https://bugs.freedesktop.org/show_bug.cgi?id=104183
-rw-r--r--src/broadcom/qpu/qpu_pack.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/broadcom/qpu/qpu_pack.c b/src/broadcom/qpu/qpu_pack.c
index 0ecce86662a..6630ed4b85f 100644
--- a/src/broadcom/qpu/qpu_pack.c
+++ b/src/broadcom/qpu/qpu_pack.c
@@ -168,7 +168,7 @@ v3d_qpu_sig_unpack(const struct v3d_device_info *devinfo,
/* Signals with zeroed unpacked contents after element 0 are reserved. */
return (packed_sig == 0 ||
- memcmp(sig, &v33_sig_map[0], sizeof(*sig) != 0));
+ memcmp(sig, &v33_sig_map[0], sizeof(*sig)) != 0);
}
bool