diff options
| author | Boyan Karatotev <boyan.karatotev@arm.com> | 2026-06-25 13:27:55 +0100 |
|---|---|---|
| committer | Boyan Karatotev <boyan.karatotev@arm.com> | 2026-06-25 13:35:04 +0100 |
| commit | c2dd7b5e1592abc8e3ede4a379ec488158ef3741 (patch) | |
| tree | fbbe35e6828ff6344b0e5976950742e6b2b9a291 /common | |
| parent | 5da9bc26bb5bfbb466a092d2169f0c82ea49b938 (diff) | |
| download | arm-trusted-firmware-c2dd7b5e1592abc8e3ede4a379ec488158ef3741.tar.gz arm-trusted-firmware-c2dd7b5e1592abc8e3ede4a379ec488158ef3741.zip | |
fix(cpufeat): do not perform bitwise OR on boolean types
The |= operator performs a bitwise OR on the two sides. MISRA complains
that this is incorrect and that it should instead be a logical OR since
the types are boolean and not integers. Since there is no such short
operator convert it to the full one.
Change-Id: I09d6d23649695bbff2b474e5b23ab7910bebb68b
Signed-off-by: Boyan Karatotev <boyan.karatotev@arm.com>
Diffstat (limited to 'common')
| -rw-r--r-- | common/feat_detect.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/feat_detect.c b/common/feat_detect.c index 729b6dd8f..af147f412 100644 --- a/common/feat_detect.c +++ b/common/feat_detect.c @@ -30,7 +30,7 @@ check_feature(int state, unsigned long field, const char *feat_name, #define CHECK_FEATURE(name, idreg, guard, field, min, max, worlds) \ do { \ - tainted |= check_feature(guard, _read_ ## name ## _id_field(), #name, min, max); \ + tainted = tainted || check_feature(guard, _read_ ## name ## _id_field(), #name, min, max); \ } while (0); /* |