Change in coreboot[master]: soc/intel/common/basecode/eop: coreboot driver perform EOP operations

[Subrata Banik (Code Review)]

Attention is currently required from: Arthur Heymans, Felix Singer, Tim Wawrzynczak, Angel Pons, Lean Sheng Tan, Werner Zeh, Patrick Rudolph, EricR Lai.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60405 )

Change subject: soc/intel/common/basecode/eop: coreboot driver perform EOP operations
......................................................................

Patch Set 23:

(1 comment)

File src/soc/intel/common/basecode/eop/eop.c:

https://review.coreboot.org/c/coreboot/+/60405/comment/87f909ed_4759dc32 
PS23, Line 20: static void do_eop_operations(void *unused)
             : {
             : 	if (CONFIG(SKIP_FSP_NOTIFY_PHASE_READY_TO_BOOT)) {
             : 		/* Step 1 */
             : 		cse_send_end_of_post();
             : 
             : 		/* Step 2 */
             : 		cse_lock_config();
             : 
             : 		/* Step 3 */
             : 		if (CONFIG(DISABLE_HECI1_AT_PRE_BOOT)) {
             : 			cse_set_to_d0i3();
             : 			heci1_disable();
             : 		}
             : 	}
             : 
             : 	if (CONFIG(SKIP_FSP_NOTIFY_PHASE_END_OF_FIRMWARE)) {
             : 		/* Step 4 */
             : 		heci_set_to_d0i3();
             : 		/* Step 5 */
             : 		pmc_clear_pmcon_sts();
             : 	}
             : }
> Now that CSE device state is independent of hiding (see DISABLE_HECI1_AT_PRE_BOOT Kconfig), we should use its device operations instead of bootstate callbacks.

Good feedback! then ideally we will be ending up adding `.final` in all relevant IP code (cse, pmc, spi etc. etc.) that need some additional configuration, isn't it.

Can we consider this as design assumption ?