[coreboot-gerrit] Change in coreboot[master]: vx900: map the SPI controller

[Lubomir Rintel (Code Review)]

Lubomir Rintel has uploaded this change for review. ( https://review.coreboot.org/22258


Change subject: vx900: map the SPI controller
......................................................................

vx900: map the SPI controller

This is required for Flashrom to work well.

Change-Id: Id756d86a7f3b34f816ea7a7ed78f159512f550d5
Signed-off-by: Lubomir Rintel <lkundrak at v3.sk>
---
M src/northbridge/via/vx900/lpc.c
M src/northbridge/via/vx900/vx900.h
2 files changed, 53 insertions(+), 2 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/58/22258/1

diff --git a/src/northbridge/via/vx900/lpc.c b/src/northbridge/via/vx900/lpc.c
index b36aaed..822e9ac 100644
--- a/src/northbridge/via/vx900/lpc.c
+++ b/src/northbridge/via/vx900/lpc.c
@@ -184,9 +184,58 @@
 	dump_pci_device(dev);
 }
 
+static void vx900_lpc_read_resources(device_t dev)
+{
+	struct resource *res;
+	pci_dev_read_resources(dev);
+
+	/* MMIO space */
+	res = new_resource(dev, VX900_MMCONFIG_MBAR);
+	res->base = 0xfed03000;
+	res->size = 0x1000;
+	res->align = 12;
+	res->gran = 12;
+	res->limit = 0xffffffff;
+	res->flags = IORESOURCE_MEM | IORESOURCE_RESERVE;
+
+	/* SPI controller */
+	res = new_resource(dev, IOINDEX_SUBTRACTIVE(0, 0));
+	res->base = 0xfed02000;
+	res->size = 0x8;
+	res->align = 12;
+	res->gran = 12;
+	res->limit = 0xffffffff;
+	res->flags = IORESOURCE_MEM | IORESOURCE_RESERVE;
+}
+
+static void vx900_lpc_set_resources(device_t dev)
+{
+	struct resource *mmio, *spi;
+        u32 reg;
+
+	mmio = find_resource(dev, VX900_MMCONFIG_MBAR);
+	if (mmio) {
+		report_resource_stored(dev, mmio, "<mmconfig>");
+		mmio->flags |= IORESOURCE_STORED;
+		reg = pci_read_config32(dev, VX900_MMCONFIG_MBAR);
+		reg &= 0xff000000;
+		reg |= mmio->base >> 8;
+		pci_write_config32(dev, VX900_MMCONFIG_MBAR, reg);
+
+		spi = find_resource(dev, IOINDEX_SUBTRACTIVE(0, 0));
+		if (spi) {
+			report_resource_stored(dev, spi, "<spi>");
+			spi->flags |= IORESOURCE_STORED;
+			/* Set base and the enable bit. */
+			((u32*)(uintptr_t)mmio->base)[0] = (spi->base | 0x01);
+		}
+	}
+	pci_dev_set_resources(dev);
+}
+
 static struct device_operations vx900_lpc_ops = {
-	.read_resources = pci_dev_read_resources,
-	.set_resources = pci_dev_set_resources,
+	.read_resources = vx900_lpc_read_resources,
+	.set_resources = vx900_lpc_set_resources,
 	.enable_resources = pci_dev_enable_resources,
 	.init = vx900_lpc_init,
 	.scan_bus = scan_lpc_bus,
diff --git a/src/northbridge/via/vx900/vx900.h b/src/northbridge/via/vx900/vx900.h
index 12e5733..216f637 100644
--- a/src/northbridge/via/vx900/vx900.h
+++ b/src/northbridge/via/vx900/vx900.h
@@ -26,6 +26,8 @@
 
 #define SMBUS_IO_BASE	0x500
 
+#define VX900_MMCONFIG_MBAR		0xbc
+
 /* The maximum number of DIMM slots that the VX900 supports */
 #define VX900_MAX_DIMM_SLOTS 2
 #define VX900_MAX_MEM_RANKS 4

-- 
To view, visit https://review.coreboot.org/22258
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id756d86a7f3b34f816ea7a7ed78f159512f550d5
Gerrit-Change-Number: 22258
Gerrit-PatchSet: 1
Gerrit-Owner: Lubomir Rintel <lkundrak at v3.sk>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20171031/0bcee86e/attachment.html>