Change in coreboot[master]: soc/intel/common/cse: Rework heci_disable function

[Subrata Banik (Code Review)]

Attention is currently required from: Maulik V Vaghela, Jonathan Zhang, Angel Pons, Sridhar Siricilla, Arthur Heymans, Andrey Petrov, Patrick Rudolph, Anjaneya "Reddy" Chagam, Johnny Lin, Tim Wawrzynczak, Christian Walter, Nick Vaccaro, Werner Zeh, Tim Chu.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61431 )

Change subject: soc/intel/common/cse: Rework heci_disable function
......................................................................

Patch Set 7:

(2 comments)

File src/soc/intel/apollolake/include/soc/pcr_ids.h:

https://review.coreboot.org/c/coreboot/+/61431/comment/360e3411_8eb46110 
PS7, Line 30: /* Reserved */
> Still marked as reserved now that we now its functionality?

I have added PID_CSME0 macro to fix the compilation issue but marked it reserved as 
on APL, disabling HECI1 using SBI msg with CSME0 PID is not supported. The only option is via PCR write.

Do you think its reasonable to mark as reserved ?

File src/soc/intel/common/block/cse/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/61431/comment/10ceecdb_a589f7cc 
PS7, Line 2:  disable_heci.c
> Don't we usually have a separate line for every file to add to a stage?

Ack