[coreboot-gerrit] Change in coreboot[master]: soc/intel/*: Add TPM decode range to soc_get_fixed_mmio_ranges()

[Michael Niewöhner (Code Review)]

Attention is currently required from: Nico Huber, Patrick Rudolph, Tim Wawrzynczak, Subrata Banik.
Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48658 )

Change subject: soc/intel/*: Add TPM decode range to soc_get_fixed_mmio_ranges()
......................................................................

Patch Set 2:

(1 comment)

File src/soc/intel/xeon_sp/lpc.c:

https://review.coreboot.org/c/coreboot/+/48658/comment/33ee61b3_01afaf22 
PS2, Line 13: 	{ CONFIG_CRB_TPM_BASE_ADDRESS, 0x8000 },
            : 	{ CONFIG_TPM_TIS_BASE_ADDRESS, 0x8000 },
            : 
AFAIUI these Kconfigs are only there for overriding the fixed address in case the tpm doesn't comply with the TCG spec.

If a user/board overrides this value, we should not add that variable value to the lpc fixed ranges. Also, the Kconfig doesn't change the hardware (iow when the Kconfig changes, the soc will still decode 0xfed4000 and it should still be here).

IMO we should define the ranges/sizes per soc in include/soc/iomap.h and use these fixed ones here.