[S] Change in coreboot[master]: soc/amd/*/acpi: don't write unused p_lvl[2,3]_lat FADT fields

[Martin Roth (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Matt DeVillier, Fred Reitberger, Felix Held.

Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/73231 )

Change subject: soc/amd/*/acpi: don't write unused p_lvl[2,3]_lat FADT fields
......................................................................

Patch Set 1:

(1 comment)

File src/soc/amd/cezanne/acpi.c:

https://review.coreboot.org/c/coreboot/+/73231/comment/01c2d763_18d7380e 
PS1, Line 79: ACPI_FADT_C2_NOT_SUPPORTED
Are we zeroing the structure before filling it in, or by removing these are we leaving random data there?

If the _CST objects override these values, why not leave them as unsupported in the FADT?  If we put a 0 here, and for some reason the CST weren't filled out, would that be better or worse than saying unsupported?

I think 100 and 1000 might be the best values to put here if we wanted to avoid saying unsupported.