Change in coreboot[master]: soc/intel/common/cse: Add config to disable CSE at pre-boot

[Felix Singer (Code Review)]

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

Change subject: soc/intel/common/cse: Add config to disable CSE at pre-boot
......................................................................

Patch Set 2:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/60721/comment/de245c97_1113ae64 
PS2, Line 13: 	  This config decides the state of CSE/Heci1 device at the end of boot.
> > DISABLE_HECI1 sounds good. […]

I also wouldn't use "CSE" in the name, since for me CSE refers to the firmware, some internals of it or the hardware part and this is just about the communication interface. However, I would just use HECI1.

Also, I wouldn't invert the option. I think we should be more consistent regarding the direction (enable or disable) of options. If find it pretty hard and also annoying to use the Kconfig menu if some have "enable" and others "disable" in their name. From my perspective, people expect the "enable" terminology in user interfaces. So I would name it "Enable HECI1" and set it to enable by default.