mb/*: Use common IPMI KCS driver

Remove duplicated code and instead use the IPMI KCS driver, which provides
the same functionality.

Change-Id: I419713c9bef02084cca1ff4cf11c33c2e3e8d3c1
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33490
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Andrey Petrov <anpetrov@fb.com>

3 files changed, 2 insertions, 134 deletions

diff --git a/src/mainboard/ocp/monolake/devicetree.cb b/src/mainboard/ocp/monolake/devicetree.cb
index 2d91b73b02..6a8bef179e 100644
--- a/src/mainboard/ocp/monolake/devicetree.cb
+++ b/src/mainboard/ocp/monolake/devicetree.cb
@@ -11,8 +11,9 @@ chip soc/intel/fsp_broadwell_de
 			chip drivers/pc80/tpm
 				device pnp 0c31.0 on end
 			end
-			chip drivers/generic/generic		# BMC KCS
+			chip drivers/ipmi		# BMC KCS
 				device pnp ca2.0 on end
+				register "bmc_i2c_address" = "0x20"
 			end
 		end # LPC Bridge
 		device pci 1f.2 on end # SATA Controller
diff --git a/src/mainboard/ocp/monolake/dsdt.asl b/src/mainboard/ocp/monolake/dsdt.asl
index 72b1c3c086..1248703266 100644
--- a/src/mainboard/ocp/monolake/dsdt.asl
+++ b/src/mainboard/ocp/monolake/dsdt.asl
@@ -27,9 +27,6 @@ DefinitionBlock(
 {
 	#include "acpi/platform.asl"
 
-	Name (IDTP, 0x0CA2)
-	Name (ICDP, 0x0CA6)
-
 	Name(_S0, Package() { 0x00, 0x00, 0x00, 0x00 })
 	Name(_S5, Package() { 0x07, 0x00, 0x00, 0x00 })
 
@@ -293,97 +290,5 @@ DefinitionBlock(
 		}
 	}
 
-	Scope (_SB.PCI0.LPC0)
-	{
-		Device (SPMI)
-		{
-			Name (_HID, EisaId ("IPI0001"))
-			Name (_STR, Unicode ("IPMI_KCS"))
-			Name (_UID, 0x00)
-			OperationRegion (IPST, SystemIO, ICDP, 0x01)
-			Field (IPST, ByteAcc, NoLock, Preserve)
-			{
-				STAS,		8
-			}
-			Method (_STA, 0, NotSerialized) {
-				Return (0x0f)
-			}
-			Name (ICRS, ResourceTemplate ()
-			{
-				IO (Decode16,
-					0x0000,
-					0x0000,
-					0x00,
-					0x00,
-					_Y01)
-				IO (Decode16,
-					0x0000,
-					0x0000,
-					0x00,
-					0x00,
-					_Y02)
-
-			})
-			Method (_CRS, 0, NotSerialized)
-			{
-				CreateWordField (ICRS, \_SB.PCI0.LPC0.SPMI._Y01._MIN, IPDB)
-				CreateWordField (ICRS, \_SB.PCI0.LPC0.SPMI._Y01._MAX, IPDH)
-				CreateByteField (ICRS, \_SB.PCI0.LPC0.SPMI._Y01._LEN, IPDL)
-				CreateWordField (ICRS, \_SB.PCI0.LPC0.SPMI._Y02._MIN, IPCB)
-				CreateWordField (ICRS, \_SB.PCI0.LPC0.SPMI._Y02._MAX, IPCH)
-				CreateByteField (ICRS, \_SB.PCI0.LPC0.SPMI._Y02._LEN, IPCL)
-
-				IPDB = IDTP
-				IPDH = IDTP
-				IPDL = 0x01
-
-				IPCB = ICDP
-				IPCH = ICDP
-				IPCL = 0x01
-
-				Return (ICRS)
-			}
-			Method (_IFT, 0, NotSerialized) {
-				Return (0x01)
-			}
-			Method(_SRV, 0, NotSerialized) {
-				Return (0x0200)
-			}
-		}
-
-		Device (SYSR)
-		{
-			Name (_HID, EisaId ("PNP0C02"))
-
-			Name (_CRS, ResourceTemplate ()  // _CRS: Current Resource Settings
-			{
-				IO (Decode16,
-					0x0CA2,
-					0x0CA2,
-					0x01,
-					0x01,
-				)
-				IO (Decode16,
-					0x0CA6,
-					0x0CA6,
-					0x01,
-					0x01,
-				)
-				IO (Decode16,
-					0x0CA8,
-					0x0CA8,
-					0x01,
-					0x01,
-				)
-				IO (Decode16,
-					0x0CAC,
-					0x0CAC,
-					0x01,
-					0x01,
-				)
-			})
-		}
-	}
-
 	#include "acpi/mainboard.asl"
 }
diff --git a/src/mainboard/ocp/monolake/mainboard.c b/src/mainboard/ocp/monolake/mainboard.c
index 56aef6b36e..9755b06983 100644
--- a/src/mainboard/ocp/monolake/mainboard.c
+++ b/src/mainboard/ocp/monolake/mainboard.c
@@ -15,55 +15,17 @@
  */
 
 #include <device/device.h>
-#include <smbios.h>
-#if CONFIG(VGA_ROM_RUN)
-#include <x86emu/x86emu.h>
-#endif
 #include <pc80/mc146818rtc.h>
 #include <cf9_reset.h>
 #include "ipmi.h"
 
-#define BMC_KCS_BASE 0xca2
-#define INTERFACE_IS_IO 0x1
-
-#if CONFIG(GENERATE_SMBIOS_TABLES)
-static int mainboard_smbios_data(struct device *dev, int *handle,
-				 unsigned long *current)
-{
-	int len = 0;
-
-	// add IPMI Device Information
-	len += smbios_write_type38(
-		current, handle,
-		SMBIOS_BMC_INTERFACE_KCS,
-		0x20,				// IPMI Version
-		0x20,				// I2C address
-		0xff,				// no NV storage
-		BMC_KCS_BASE | INTERFACE_IS_IO, // IO port interface address
-		0x40,
-		0); // no IRQ
-
-	return len;
-}
-#endif
-
 /*
  * mainboard_enable is executed as first thing after enumerate_buses().
  * This is the earliest point to add customization.
  */
 static void mainboard_enable(struct device *dev)
 {
-#if CONFIG(GENERATE_SMBIOS_TABLES)
-	dev->ops->get_smbios_data = mainboard_smbios_data;
-#endif
-
-	/* Enable access to the BMC IPMI via KCS */
-	struct device *lpc_sio_dev = dev_find_slot_pnp(BMC_KCS_BASE, 0);
-	struct resource *res = new_resource(lpc_sio_dev, BMC_KCS_BASE);
 	ipmi_oem_rsp_t rsp;
-	res->base = BMC_KCS_BASE;
-	res->size = 1;
-	res->flags = IORESOURCE_IO | IORESOURCE_ASSIGNED | IORESOURCE_FIXED;
 
 	if (is_ipmi_clear_cmos_set(&rsp)) {
 		/* TODO: Should also try to restore CMOS to cmos.default