[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Configure DPR on all stacks

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46818 )

Change subject: soc/intel/xeon_sp: Configure DPR on all stacks
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/46818/5/src/soc/intel/xeon_sp/uncor... 
File src/soc/intel/xeon_sp/uncore.c:

https://review.coreboot.org/c/coreboot/+/46818/5/src/soc/intel/xeon_sp/uncor... 
PS5, Line 320: 	configure_dpr(dev);
> why not in init?

This should probably happen after clearing the memory has happened (BS_DEV_INIT, BS_ON_EXIT). If it is done in .final it also happens before LockConfig.