[coreboot-gerrit] [S] Change in coreboot[main]: src/soc/intel/common: Add Intel RMT+ support

[Pratikkumar V Prajapati (Code Review)]

Attention is currently required from: Anil Kumar K, Arthur Heymans, Bora Guvendik, Cliff Huang, Gaggery Tsai, Jamie Ryu, Kane Chen, Karthik Ramasubramanian, Krishna P Bhat D, Ronak Kanabar, Subrata Banik, V Sowmya, Wonkyu Kim.

Pratikkumar V Prajapati has posted comments on this change by Gaggery Tsai. ( https://review.coreboot.org/c/coreboot/+/77904?usp=email )

Change subject: src/soc/intel/common: Add Intel RMT+ support
......................................................................

Patch Set 9:

(1 comment)

File src/soc/intel/common/block/memory/Kconfig:

https://review.coreboot.org/c/coreboot/+/77904/comment/d627a3cf_3a15ff74?usp... :
PS9, Line 33: config SOC_INTEL_RMT_PLUS
            : 	bool
            : 	select SOC_INTEL_COMMON_ACPI_BDAT
            : 	help
            : 	  Intel RMT plus is a feature from FSPM. It exports the RMT data from
            : 	  HOBs with 4 different GUIDs for 4 SAGV points. With this feature
            : 	  enabled, once the memory training is required, the RMT flag will be
            : 	  turned on automatically. The RMT data exported from HOBs will pass
            : 	  through ACPI BDAT table to the OS.
            : 
            : config SOC_INTEL_RMT_PLUS_IN_DEV_MODE
            : 	bool
            : 	select SOC_INTEL_COMMON_ACPI_BDAT
            : 	help
            : 	  This is the same with SOC_INTEL_RMT_PLUS but only enables RMT plus
            : 	  in the DEV mode.
instead of multiple bool configs, would it make sense to have just one config and set various types of mode with the single config?