[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp/cpx: Add soc_acpi_fill_fadt()

[Stefan Reinauer (Code Review)]

Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45845 )

Change subject: soc/intel/xeon_sp/cpx: Add soc_acpi_fill_fadt()
......................................................................

Patch Set 10: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/45845/10/src/soc/intel/xeon_sp/cpx/... 
File src/soc/intel/xeon_sp/cpx/acpi.c:

https://review.coreboot.org/c/coreboot/+/45845/10/src/soc/intel/xeon_sp/cpx/... 
PS10, Line 176: The bit_width field intentionally overflows here.
That's a bold claim, but what is the intention?