[coreboot-gerrit] Change in ...coreboot[master]: mb/intel/icelake_rvp: Add SMI handlers

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32252 )

Change subject: mb/intel/icelake_rvp: Add SMI handlers
......................................................................

Patch Set 3:

(3 comments)

https://review.coreboot.org/#/c/32252/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/32252/3//COMMIT_MSG@7 
PS3, Line 7: mb/intel/icelake_rvp: Add SMI handlers
Is it Chrome EC specific?
> Add SMI handlers for Chrome EC

https://review.coreboot.org/#/c/32252/3//COMMIT_MSG@11 
PS3, Line 11: 1. eSPI SMI event.
Please remove the dot at the end.

https://review.coreboot.org/#/c/32252/3//COMMIT_MSG@15 
PS3, Line 15: Sleep(S3/S5)
Please add a space before (.