Arthur Heymans has submitted this change. ( https://review.coreboot.org/c/coreboot/+/84007?usp=email )
Change subject: libpayload: Use unsigned integer for PDG_MASK ......................................................................
libpayload: Use unsigned integer for PDG_MASK
The clang compiler warns about undefined behavior about shifting signed integers.
Change-Id: Ic4ce64207393ec4a8d6b188b35e0f436342826de Signed-off-by: Arthur Heymans arthur@aheymans.xyz Reviewed-on: https://review.coreboot.org/c/coreboot/+/84007 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com --- M payloads/libpayload/arch/arm/virtual.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved
diff --git a/payloads/libpayload/arch/arm/virtual.c b/payloads/libpayload/arch/arm/virtual.c index 867a43b..cfc7af1 100644 --- a/payloads/libpayload/arch/arm/virtual.c +++ b/payloads/libpayload/arch/arm/virtual.c @@ -71,7 +71,7 @@ #define SECTION_SHIFT 30 #define BLOCK_SHIFT 21 #define PAGE_SHIFT 12 -#define PGD_MASK (~0 << PAGE_SHIFT) +#define PGD_MASK (~0u << PAGE_SHIFT)
static pmd_t *ttb_buff = 0; static uintptr_t work_block;