[coreboot-gerrit] Change in coreboot[master]: soc/intel: Configure PAVP at compile-time

[Benjamin Doron (Code Review)]

Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42745 )

Change subject: soc/intel: Configure PAVP at compile-time
......................................................................

Patch Set 9:

(2 comments)

https://review.coreboot.org/c/coreboot/+/42745/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/42745/5//COMMIT_MSG@11 
PS5, Line 11:
> I think that the default can be changed. […]

The Kconfig defaults preserve the old behaviour and I've noted this in the commit message.

Besides, see CB:41763. The logic was to align Jasperlake with the other platforms, not to disable the feature. However, other platforms have it enabled.

https://review.coreboot.org/c/coreboot/+/42745/7//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/42745/7//COMMIT_MSG@10 
PS7, Line 10: protection
> That would be covered as part of the quoted definition. […]

Done