[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block: Provide mmc.c for setting dll registers.

[Kane Chen (Code Review)]

Kane Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35040 )

Change subject: soc/intel/common/block: Provide mmc.c for setting dll registers.
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35040/5/src/soc/intel/common/block/... 
File src/soc/intel/common/block/scs/mmc.c:

https://review.coreboot.org/c/coreboot/+/35040/5/src/soc/intel/common/block/... 
PS5, Line 61: SOC_INTEL_COMMON_MMC_OVERRIDE
> I am just concerned that we will be adding confusion about when to use what i.e. […]

I would prefer we only have one place to program this DLL. 
And the FSP has the full control for this DLL in the future.
Otherwise, it would also create confusion when we do the debug inside FSP.

so far i think this override is just to mitigate the schedule.
Thanks.