[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cnl,icl,skl}: Remove unused SMI opregion

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36462 )

Change subject: soc/intel/{cnl,icl,skl}: Remove unused SMI opregion
......................................................................

Patch Set 3:

(2 comments)

https://review.coreboot.org/c/coreboot/+/36462/1/src/soc/intel/skylake/acpi/... 
File src/soc/intel/skylake/acpi/platform.asl:

https://review.coreboot.org/c/coreboot/+/36462/1/src/soc/intel/skylake/acpi/... 
PS1, Line 38: /* IO-Trap at 0x800.
            :  * This is the ACPI->SMI communication interface.
            :  */
            : OperationRegion (IO_T, SystemIO, 0x800, 0x10)
            : Field (IO_T, ByteAcc, NoLock, Preserve)
            : {
            : 	Offset (0x8),
            : 	TRP0, 8		/* IO-Trap at 0x808 */
            : }
> yes should be good to clean as well

Done https://review.coreboot.org/c/coreboot/+/36466

https://review.coreboot.org/c/coreboot/+/36462/1/src/soc/intel/skylake/acpi/... 
PS1, Line 49: TRAP
> may be i will check with GFX team if they have any question, but in general good to clean both in sp […]

Done