fix(console): typecast operands to match data type
This corrects the MISRA violation C2012-10.1:
Operands shall not be of an inappropriate essential type.
The condition is explicitly checked against 0U, appending 'U' and
typecasting for unsigned comparison.
Change-Id: Ie78d91f0f79804797648ef996fb208d3ee1698dd
Signed-off-by: Nithin G <nithing@amd.com>
Signed-off-by: Maheedhar Bollapalli <maheedharsai.bollapalli@amd.com>
diff --git a/drivers/console/multi_console.c b/drivers/console/multi_console.c
index 59a4a86..b0ba5e5 100644
--- a/drivers/console/multi_console.c
+++ b/drivers/console/multi_console.c
@@ -95,7 +95,7 @@
console_t *console;
for (console = console_list; console != NULL; console = console->next) {
- if ((console->flags & console_state) && (console->putc != NULL)) {
+ if (((console->flags & console_state) != 0U) && (console->putc != NULL)) {
int ret = do_putc(c, console);
if ((err == ERROR_NO_VALID_CONSOLE) || (ret < err)) {
err = ret;
@@ -123,7 +123,7 @@
do { /* Keep polling while at least one console works correctly. */
for (console = console_list; console != NULL;
console = console->next)
- if ((console->flags & console_state) && (console->getc != NULL)) {
+ if (((console->flags & console_state) != 0U) && (console->getc != NULL)) {
int ret = console->getc(console);
if (ret >= 0) {
return ret;
@@ -143,7 +143,7 @@
console_t *console;
for (console = console_list; console != NULL; console = console->next)
- if ((console->flags & console_state) && (console->flush != NULL)) {
+ if (((console->flags & console_state) != 0U) && (console->flush != NULL)) {
console->flush(console);
}
}