[coreboot-gerrit] [XS] Change in coreboot[main]: libpayload: Use unsigned integet for PDG_MASK

[Nico Huber (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans.

Nico Huber has posted comments on this change by Arthur Heymans. ( https://review.coreboot.org/c/coreboot/+/84007?usp=email )

Change subject: libpayload: Use unsigned integet for PDG_MASK
......................................................................

Patch Set 3: Code-Review+2

(1 comment)

File payloads/libpayload/arch/arm/virtual.c:

https://review.coreboot.org/c/coreboot/+/84007/comment/921f3371_4cd9547d?usp... :
PS3, Line 82: 	return 1 << PAGE_SHIFT;
> Same "problem" here

I don't think it is. Shifting a signed but positive integer is defined.
The problem above, I guess, is that `~0` (to be read as `~(int)0`)
is negative.