[coreboot-gerrit] Change in coreboot[master]: libpayload: Fix potential compilation issues

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42234 )

Change subject: libpayload: Fix potential compilation issues
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42234/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/42234/2//COMMIT_MSG@9 
PS2, Line 9: The include chain can define one of the macros (BIT, MIN, MAX are all
> I still think this is the better, more robust approach (and in the same time fix every time it's mis […]

I'm okay with putting this in with a #warning directive if you want to, that may be useful for other payloads which don't care about doing things the right way as much. Note that for depthcharge, all warnings are treated as errors, so it would still prevent compilation there (as I think it should).