Nico Huber has posted comments on this change. ( https://review.coreboot.org/28847 )
Change subject: src/soc/intel/cannonlake: Fix IA32_PLATFORM_DCA_CAP addresse ......................................................................
Patch Set 4:
I agree to the change but I think somebody at Intel should really investigate what the processor supports and what MSRs exist. It looks like the configure_dca_cap() function is plain copy-pasta, I traced it back to Sandybridge. But it was already there implemented wrong anyway if you compare it to the BWG. I also have doubts if there ever was any Intel processor with DCA supported by coreboot.