[coreboot-gerrit] Change in coreboot[master]: util/inteltool: Add ICH10 (Consumer Base) support

[Arthur Heymans (Code Review)]

Arthur Heymans has uploaded a new change for review. ( https://review.coreboot.org/19232 )

Change subject: util/inteltool: Add ICH10 (Consumer Base) support
......................................................................

util/inteltool: Add ICH10 (Consumer Base) support

Reuses ICH10R functions.

Change-Id: If9ae8ba8b95e3a7bf6596ae639eb8cafab583298
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M util/inteltool/gpio.c
M util/inteltool/inteltool.c
M util/inteltool/inteltool.h
M util/inteltool/powermgt.c
M util/inteltool/rootcmplx.c
M util/inteltool/spi.c
6 files changed, 7 insertions(+), 0 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/32/19232/1

diff --git a/util/inteltool/gpio.c b/util/inteltool/gpio.c
index b9d5ae1..db0e3a4 100644
--- a/util/inteltool/gpio.c
+++ b/util/inteltool/gpio.c
@@ -939,6 +939,7 @@
 		gpio_defaults = pp_pch_mobile_defaults;
 		defaults_size = ARRAY_SIZE(pp_pch_mobile_defaults);
 		break;
+	case PCI_DEVICE_ID_INTEL_ICH10:
 	case PCI_DEVICE_ID_INTEL_ICH10R:
 		gpiobase = pci_read_word(sb, 0x48) & 0xfffc;
 		gpio_registers = ich10_gpio_registers;
diff --git a/util/inteltool/inteltool.c b/util/inteltool/inteltool.c
index 6596900..79b0bce 100644
--- a/util/inteltool/inteltool.c
+++ b/util/inteltool/inteltool.c
@@ -93,6 +93,7 @@
 	{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_BAYTRAIL, "Bay Trail" },
 	/* Southbridges (LPC controllers) */
 	{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_82371XX, "371AB/EB/MB" },
+	{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH10, "ICH10" },
 	{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH10R, "ICH10R" },
 	{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH9DH, "ICH9DH" },
 	{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH9DO, "ICH9DO" },
diff --git a/util/inteltool/inteltool.h b/util/inteltool/inteltool.h
index 5931cb0..cd981d1 100644
--- a/util/inteltool/inteltool.h
+++ b/util/inteltool/inteltool.h
@@ -82,6 +82,7 @@
 #define PCI_DEVICE_ID_INTEL_ICH9M		0x2919
 #define PCI_DEVICE_ID_INTEL_ICH9ME		0x2917
 #define PCI_DEVICE_ID_INTEL_ICH10R		0x3a16
+#define PCI_DEVICE_ID_INTEL_ICH10               0x3a18
 #define PCI_DEVICE_ID_INTEL_3400_DESKTOP	0x3b00
 #define PCI_DEVICE_ID_INTEL_3400_MOBILE		0x3b01
 #define PCI_DEVICE_ID_INTEL_P55			0x3b02
diff --git a/util/inteltool/powermgt.c b/util/inteltool/powermgt.c
index fd0d00f..7f04308 100644
--- a/util/inteltool/powermgt.c
+++ b/util/inteltool/powermgt.c
@@ -707,6 +707,7 @@
 		pm_registers = pch_pm_registers;
 		size = ARRAY_SIZE(pch_pm_registers);
 		break;
+	case PCI_DEVICE_ID_INTEL_ICH10:
 	case PCI_DEVICE_ID_INTEL_ICH10R:
 		pmbase = pci_read_word(sb, 0x40) & 0xff80;
 		pm_registers = ich10_pm_registers;
diff --git a/util/inteltool/rootcmplx.c b/util/inteltool/rootcmplx.c
index 323811d..337f981 100644
--- a/util/inteltool/rootcmplx.c
+++ b/util/inteltool/rootcmplx.c
@@ -41,6 +41,7 @@
 	case PCI_DEVICE_ID_INTEL_ICH9:
 	case PCI_DEVICE_ID_INTEL_ICH9M:
 	case PCI_DEVICE_ID_INTEL_ICH9ME:
+	case PCI_DEVICE_ID_INTEL_ICH10:
 	case PCI_DEVICE_ID_INTEL_ICH10R:
 	case PCI_DEVICE_ID_INTEL_NM10:
 	case PCI_DEVICE_ID_INTEL_I63XX:
diff --git a/util/inteltool/spi.c b/util/inteltool/spi.c
index 8dc6ecb..154b3c9 100644
--- a/util/inteltool/spi.c
+++ b/util/inteltool/spi.c
@@ -106,6 +106,7 @@
 	case PCI_DEVICE_ID_INTEL_ICH9:
 	case PCI_DEVICE_ID_INTEL_ICH9M:
 	case PCI_DEVICE_ID_INTEL_ICH9ME:
+	case PCI_DEVICE_ID_INTEL_ICH10:
 	case PCI_DEVICE_ID_INTEL_ICH10R:
 	case PCI_DEVICE_ID_INTEL_NM10:
 		bios_cntl = pci_read_byte(sb, 0xdc);
@@ -187,6 +188,7 @@
 	case PCI_DEVICE_ID_INTEL_ICH9:
 	case PCI_DEVICE_ID_INTEL_ICH9M:
 	case PCI_DEVICE_ID_INTEL_ICH9ME:
+	case PCI_DEVICE_ID_INTEL_ICH10:
 	case PCI_DEVICE_ID_INTEL_ICH10R:
 	case PCI_DEVICE_ID_INTEL_NM10:
 	case PCI_DEVICE_ID_INTEL_I63XX:

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If9ae8ba8b95e3a7bf6596ae639eb8cafab583298
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>