Mirrors/opensbi
3
0
Fork 1
mirror of https://github.com/riscv-software-src/opensbi.git synced 2026-06-12 22:31:45 +01:00
Code Issues Packages Projects Releases Wiki Activity

lib: sbi: Fix LLVM compile error observed in sbi_mpxy.c

Browse Source 
The following LLVM compile error is observed in sbi_mpxy.c:

 CC        lib/sbi/sbi_mpxy.o
lib/sbi/sbi_mpxy.c:535:36: error: result of comparison of constant 18446744073709551615 with
      expression of type 'u32' (aka 'unsigned int') is always false [-Werror,-Wtautological-constant-out-of-range-compare]
  535 |                     (attrs->msi_info.msi_addr_hi == INVALID_ADDR))
      |                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^  ~~~~~~~~~~~~
lib/sbi/sbi_mpxy.c:534:36: error: result of comparison of constant 18446744073709551615 with
      expression of type 'u32' (aka 'unsigned int') is always false [-Werror,-Wtautological-constant-out-of-range-compare]
  534 |                     (attrs->msi_info.msi_addr_lo == INVALID_ADDR) &&
      |                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^  ~~~~~~~~~~~~
2 errors generated.

To address the above issue, add u32 typecast to INVALID_ADDR.

Fixes: e92c8fd083 ("sbi: mpxy: define INVALID_ADDR using unsigned long width")
Fixes: 7939bf1329 ("lib: sbi: Add SBI Message Proxy (MPXY) framework")
Signed-off-by: Anup Patel <anup.patel@oss.qualcomm.com>
Link: https://lore.kernel.org/r/20260612062218.172726-1-anup.patel@oss.qualcomm.com
Signed-off-by: Anup Patel <anup@brainfault.org>
This commit is contained in:
Anup Patel
2026-06-12 11:52:18 +05:30
committed by Anup Patel
parent 895201cc5a
commit c175c97a27
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/sbi/sbi_mpxy.c
+2 -2
View File
@@ -531,8 +531,8 @@ static int mpxy_check_write_std_attr(struct sbi_mpxy_channel *channel,
if (attr_val > 1) if (attr_val > 1)
ret = SBI_ERR_INVALID_PARAM; ret = SBI_ERR_INVALID_PARAM;
if (attr_val == 1 && if (attr_val == 1 &&
(attrs->msi_info.msi_addr_lo == INVALID_ADDR) && (attrs->msi_info.msi_addr_lo == (u32)INVALID_ADDR) &&
(attrs->msi_info.msi_addr_hi == INVALID_ADDR)) (attrs->msi_info.msi_addr_hi == (u32)INVALID_ADDR))
ret = SBI_ERR_DENIED; ret = SBI_ERR_DENIED;
break; break;
case SBI_MPXY_ATTR_MSI_ADDR_LO: case SBI_MPXY_ATTR_MSI_ADDR_LO: