soc/intel/meteorlake: Use common PCH client SMI handler

Migrate Meteor Lake to use the common PCH client SMI handler
implementation from the Intel common feature code. This change
eliminates platform-specific code by leveraging the shared smihandler.c
driver.

This commit:
- Adds SOC_PMC_DEV macro definition to soc/pci_devs.h
- Selects SOC_INTEL_COMMON_FEATURE_SMIHANDLER Kconfig
- Removes src/soc/intel/meteorlake/smihandler.c
- Updates Makefile to remove smihandler.c compilation

Meteor Lake uses PCI_DEV_PMC as the PMC device identifier.

Change-Id: Ia21a6ee0c6fbee6d4ffcfce5fdbe9b3bdf5fcece
Signed-off-by: Jeremy Compostella <jeremy.compostella@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/91295
Reviewed-by: Huang, Cliff <cliff.huang@intel.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

src/soc/intel/meteorlake/Kconfig

@@ -90,6 +90,7 @@ config SOC_INTEL_METEORLAKE
 	select SOC_INTEL_COMMON_FEATURE_ESPI
 	select SOC_INTEL_COMMON_FEATURE_GSPI_DEVFN
 	select SOC_INTEL_COMMON_FEATURE_PMUTIL
+	select SOC_INTEL_COMMON_FEATURE_SMIHANDLER
 	select SOC_INTEL_COMMON_FEATURE_SOUNDWIRE
 	select SOC_INTEL_COMMON_FEATURE_SPI_DEVFN
 	select SOC_INTEL_COMMON_FEATURE_SPI_DEVFN_PSF

src/soc/intel/meteorlake/Makefile.mk

@@ -43,7 +43,6 @@ ramstage-y += soc_info.c
 smm-y += elog.c
 smm-y += gpio.c
 smm-y += p2sb.c
-smm-y += smihandler.c
 smm-y += soc_info.c
 smm-y += xhci.c
 

src/soc/intel/meteorlake/include/soc/pci_devs.h

@@ -243,5 +243,6 @@
 #define SA_DEVFN_IGD		PCI_DEVFN_IGD
 #define SOC_I2C_DEVFN(n)	PCI_DEVFN_I2C##n
 #define SOC_GSPI_DEVFN(n)	PCI_DEVFN_GSPI##n
+#define SOC_PMC_DEV		PCI_DEV_PMC
 
 #endif // _SOC_METEORLAKE_PCI_DEVS_H_

src/soc/intel/meteorlake/smihandler.c

@@ -1,30 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
-
-#include <device/pci_def.h>
-#include <intelblocks/smihandler.h>
-#include <soc/soc_chip.h>
-#include <soc/pci_devs.h>
-#include <soc/pm.h>
-
-int smihandler_soc_disable_busmaster(pci_devfn_t dev)
-{
-	/* Skip disabling PMC bus master to keep IO decode enabled */
-	if (dev == PCI_DEV_PMC)
-		return 0;
-	return 1;
-}
-
-const smi_handler_t southbridge_smi[SMI_STS_BITS] = {
-	[SMI_ON_SLP_EN_STS_BIT] = smihandler_southbridge_sleep,
-	[APM_STS_BIT] = smihandler_southbridge_apmc,
-	[PM1_STS_BIT] = smihandler_southbridge_pm1,
-	[GPE0_STS_BIT] = smihandler_southbridge_gpe0,
-	[GPIO_STS_BIT] = smihandler_southbridge_gpi,
-	[ESPI_SMI_STS_BIT] = smihandler_southbridge_espi,
-	[MCSMI_STS_BIT] = smihandler_southbridge_mc,
-#if CONFIG(SOC_INTEL_COMMON_BLOCK_SMM_TCO_ENABLE)
-	[TCO_STS_BIT] = smihandler_southbridge_tco,
-#endif
-	[PERIODIC_STS_BIT] = smihandler_southbridge_periodic,
-	[MONITOR_STS_BIT] = smihandler_southbridge_monitor,
-};