[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cnl,icl,skl,tgl,common}: Make changes to send_heci_reset_r...

[Sridhar Siricilla (Code Review)]

Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37584 )

Change subject: soc/intel/{cnl,icl,skl,tgl,common}: Make changes to send_heci_reset_req_message()
......................................................................

Patch Set 21:

(3 comments)

https://review.coreboot.org/c/coreboot/+/37584/6/src/soc/intel/common/block/... 
File src/soc/intel/common/block/cse/cse.c:

https://review.coreboot.org/c/coreboot/+/37584/6/src/soc/intel/common/block/... 
PS6, Line 604:
> Yes, it requires. I will push separate patch to address your comment.

The patch implements prerequisites https://review.coreboot.org/c/coreboot/+/38800/1

https://review.coreboot.org/c/coreboot/+/37584/6/src/soc/intel/common/block/... 
PS6, Line 612: heci_reset
> It resets HECI interface to ensure syncronization between CSE & Host for communication.

Done

https://review.coreboot.org/c/coreboot/+/37584/6/src/soc/intel/common/block/... 
PS6, Line 617: CSE_RESET_ONLY
> Currently, no external doc talk about CSE_RESET_ONLY. […]

Done