[coreboot-gerrit] [M] Change in coreboot[master]: ACPI: Obsolete FADT p_lvl2_lat and p_lvl3_lat fields

[Felix Held (Code Review)]

Attention is currently required from: Lance Zhao, Jason Glenesk, Raul Rangel, Jonathan Zhang, Matt DeVillier, Johnny Lin, Tim Wawrzynczak, Christian Walter, Arthur Heymans, Kyösti Mälkki, Fred Reitberger, Tim Chu.

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74430 )

Change subject: ACPI: Obsolete FADT p_lvl2_lat and p_lvl3_lat fields
......................................................................

Patch Set 3:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/74430/comment/418567f6_0bb50573 
PS3, Line 12: 
not sure if i missed a patch in the series, but src/southbridge/intel/i82801[d,g,i,j]x/fadt.c override this, so it might be good to also mention that in the commit mmessage

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

https://review.coreboot.org/c/coreboot/+/74430/comment/9edbdf39_9193f387 
PS2, Line 78: 	fadt->duty_width = 3;	/* CLK_VAL bits 3:1 */
> CB:74473 in case you need it.

hm, that's a good question. ping me again about this in 3 weeks or a month; right bow i don't really have the time and capacity to closely look into that; my recent cleanups took much more time than anticipated and now i have to very much prioritize working on the missing parts of the phoenix support.
only had a brief look at CB:74473 and seems to be a good idea, but i'm not exactly sure yet if it's needed