From cdcf907b1989c8816bf67bf078780b6353a516ce Mon Sep 17 00:00:00 2001 From: Heinrich Schuchardt Date: Mon, 21 Jun 2021 21:50:10 +0200 Subject: [PATCH] lib: sign conflict in sbi_tlb_entry_process() MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Compiling sbi_tlb_entry_process() with GCC 11 and -Wextra yields: lib/sbi/sbi_tlb.c: In function ‘sbi_tlb_process_count’: lib/sbi/sbi_tlb.c:206:31: error: comparison of integer expressions of different signedness: ‘u32’ {aka ‘unsigned int’} and ‘int’ [-Werror=sign-compare] 206 | if (deq_count > count) Signed-off-by: Heinrich Schuchardt Reviewed-by: Atish Patra Reviewed-by: Xiang W --- lib/sbi/sbi_tlb.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/sbi/sbi_tlb.c b/lib/sbi/sbi_tlb.c index 2ba1b7b..8bbe92b 100644 --- a/lib/sbi/sbi_tlb.c +++ b/lib/sbi/sbi_tlb.c @@ -193,10 +193,11 @@ static void sbi_tlb_entry_process(struct sbi_tlb_info *tinfo) } } -static void sbi_tlb_process_count(struct sbi_scratch *scratch, int count) +static void sbi_tlb_process_count(struct sbi_scratch *scratch, + unsigned int count) { struct sbi_tlb_info tinfo; - u32 deq_count = 0; + unsigned int deq_count = 0; struct sbi_fifo *tlb_fifo = sbi_scratch_offset_ptr(scratch, tlb_fifo_off); -- 2.7.4