[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/cse: Implement HECI notify

[Subrata Banik (Code Review)]

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

Change subject: soc/intel/common/cse: Implement HECI notify
......................................................................

Patch Set 7:

(1 comment)

File src/soc/intel/common/block/cse/cse_eop.c:

https://review.coreboot.org/c/coreboot/+/60405/comment/3e54b907_4d21c73c 
PS6, Line 283: /*
             :  * Ideally, to give coreboot maximum flexibility, sending EOP would be done as
             :  * late possible. If HECI_DISABLE_USING_SMM is selected, then sending EOP must
             :  * be performed before the HECI bus is disabled, so these boards use
             :  * BS_PAYLOAD_LOAD, which happens before the HECI_DISABLE_USING_SMM Kconfig takes
             :  * effect (EOP is sent using the HECI bus).
             :  * Otherwise, EOP can be pushed a little later, and can be performed in
             :  * BS_PAYLOAD_BOOT instead.
             :  */
             : #if !CONFIG(HECI_DISABLE_USING_SMM)
             : BOOT_STATE_INIT_ENTRY(BS_PAYLOAD_BOOT, BS_ON_ENTRY, set_cse_end_of_post, NULL);
             : #else
             : BOOT_STATE_INIT_ENTRY(BS_PAYLOAD_LOAD, BS_ON_ENTRY, set_cse_end_of_post, NULL);
             : #endif
> Move these below the function definition?

Ack