diff options
| author | Heinrich Schuchardt <xypron.glpk@gmx.de> | 2021-06-21 21:37:54 +0200 |
|---|---|---|
| committer | Anup Patel <anup@brainfault.org> | 2021-06-22 19:14:30 +0530 |
| commit | 99017946f36525c0689d287fbad21426575944c4 (patch) | |
| tree | 3f66f48908df9ff9952ca13c44c4d25e334b7024 /lib | |
| parent | cdcf907b1989c8816bf67bf078780b6353a516ce (diff) | |
lib: sign conflict in wake_coldboot_harts()
Compiling wake_coldboot_harts() with GCC 11 and -Wextra yields:
lib/sbi/sbi_init.c:208:27:
error: comparison of integer expressions of different signedness:
‘int’ and ‘u32’ {aka ‘unsigned int’} [-Werror=sign-compare]
208 | for (int i = 0; i <= sbi_scratch_last_hartid(); i++) {
| ^~
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reviewed-by: Atish Patra <atish.patra@wdc.com>
Reviewed-by: Xiang W <wxjstz@126.com>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/sbi/sbi_init.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/sbi/sbi_init.c b/lib/sbi/sbi_init.c index b158314..e396d95 100644 --- a/lib/sbi/sbi_init.c +++ b/lib/sbi/sbi_init.c @@ -205,7 +205,7 @@ static void wake_coldboot_harts(struct sbi_scratch *scratch, u32 hartid) spin_lock(&coldboot_lock); /* Send an IPI to all HARTs waiting for coldboot */ - for (int i = 0; i <= sbi_scratch_last_hartid(); i++) { + for (u32 i = 0; i <= sbi_scratch_last_hartid(); i++) { if ((i != hartid) && sbi_hartmask_test_hart(i, &coldboot_wait_hmask)) sbi_ipi_raw_send(i); |