[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cnl, icl}: Set CONFIG_CPU_ADDR_BITS to 39

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34753 )

Change subject: soc/intel/{cnl, icl}: Set CONFIG_CPU_ADDR_BITS to 39
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34753/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34753/3//COMMIT_MSG@14 
PS3, Line 14: Also this change helps to generate correct MTRR mask value while
            : using set_var_mtrr
> Is the problem fixed by using cpu_phys_address_size() in earlymtrr. […]

should be yes, i was actually using set_var_mtrr as many places in common code block we are using set_var_mtrr  for enabling temp MTRR.

As per my understating all set_var_mtrr() usage might need this change to make it proper or else lets replace CONFIG_CPU_ADDR_BITS in set_var_mtrr() with  cpu_phys_address_size() to fix for all.