Change in coreboot[master]: soc/amd/cezanne/fch: add HAVE_SMI_HANDLER case to fch_init_acpi_ports

[Martin Roth (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Felix Held.
Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50454 )

Change subject: soc/amd/cezanne/fch: add HAVE_SMI_HANDLER case to fch_init_acpi_ports
......................................................................

Patch Set 3:

(1 comment)

File src/soc/amd/cezanne/fch.c:

https://review.coreboot.org/c/coreboot/+/50454/comment/27b69513_3f566926 
PS3, Line 28: 
            : 		/*
            : 		 * SMI on SlpTyp requires sending SMI before completion
            : 		 * response of the I/O write. The BKDG also specifies
            : 		 * clearing ForceStpClkRetry for SMI trapping.
            : 		 */
I don't understand how this comment relates to the following code.

If we need to clear ForceStpClkRetry, why is it getting set?

Where is the SMI that needs to be sent before the completion response?