[coreboot-gerrit] [S] Change in coreboot[master]: soc/intel/cmn/pcie: Add ability for SoC to overwrite snoop/no-snoop l...

[Paul Menzel (Code Review)]

Attention is currently required from: Bora Guvendik, Hannah Williams, Jérémy Compostella.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74141 )

Change subject: soc/intel/cmn/pcie: Add ability for SoC to overwrite snoop/no-snoop latency
......................................................................

Patch Set 4:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/74141/comment/b7af75aa_88cedd14 
PS4, Line 7: Add ability for
Allow

https://review.coreboot.org/c/coreboot/+/74141/comment/6c7adc54_21899401 
PS4, Line 9: require
requires

https://review.coreboot.org/c/coreboot/+/74141/comment/674437e7_48f52ad4 
PS4, Line 10: tolerance, let it overwrite the common code settings.
… in devicetree? Kconfig?

File src/soc/intel/common/block/pcie/pcie.c:

https://review.coreboot.org/c/coreboot/+/74141/comment/21405c96_4fbf4816 
PS4, Line 14: __weak
I think, coreboot tries to avoid weak functions.