Lean Sheng Tan has uploaded this change for review.

View Change

soc/intel/elkhartlake: Update SerialIO devices details

Add I2C #6 & #7, and remove GSPI #3 as per EHL EDS.
Also update Device Function number for GSPI #2 in asl file.

Signed-off-by: Tan, Lean Sheng <lean.sheng.tan@intel.com>
Change-Id: If166fefe567a857ca29527d0367197139efbf6c7
---
M src/soc/intel/elkhartlake/acpi/serialio.asl
M src/soc/intel/elkhartlake/fsp_params.c
M src/soc/intel/elkhartlake/i2c.c
M src/soc/intel/elkhartlake/include/soc/serialio.h
4 files changed, 25 insertions(+), 8 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/40/48540/1
diff --git a/src/soc/intel/elkhartlake/acpi/serialio.asl b/src/soc/intel/elkhartlake/acpi/serialio.asl
index f7bf452..ac628d2 100644
--- a/src/soc/intel/elkhartlake/acpi/serialio.asl
+++ b/src/soc/intel/elkhartlake/acpi/serialio.asl
@@ -38,6 +38,18 @@
Name (_DDN, "Serial IO I2C Controller 5")
}

+Device (I2C6)
+{
+ Name (_ADR, 0x00100000)
+ Name (_DDN, "Serial IO I2C Controller 6")
+}
+
+Device (I2C7)
+{
+ Name (_ADR, 0x00100001)
+ Name (_DDN, "Serial IO I2C Controller 7")
+}
+
Device (SPI0)
{
Name (_ADR, 0x001e0002)
@@ -52,16 +64,10 @@

Device (SPI2)
{
- Name (_ADR, 0x00120006)
+ Name (_ADR, 0x00120000)
Name (_DDN, "Serial IO SPI Controller 2")
}

-Device (SPI3)
-{
- Name (_ADR, 0x00130000)
- Name (_DDN, "Serial IO SPI Controller 3")
-}
-
Device (UAR0)
{
Name (_ADR, 0x001e0000)
diff --git a/src/soc/intel/elkhartlake/fsp_params.c b/src/soc/intel/elkhartlake/fsp_params.c
index 83a3699..3aed1ac 100644
--- a/src/soc/intel/elkhartlake/fsp_params.c
+++ b/src/soc/intel/elkhartlake/fsp_params.c
@@ -34,6 +34,8 @@
PCH_DEVFN_I2C3,
PCH_DEVFN_I2C4,
PCH_DEVFN_I2C5,
+ PCH_DEVFN_I2C6,
+ PCH_DEVFN_I2C7,
PCH_DEVFN_GSPI0,
PCH_DEVFN_GSPI1,
PCH_DEVFN_GSPI2,
diff --git a/src/soc/intel/elkhartlake/i2c.c b/src/soc/intel/elkhartlake/i2c.c
index c13a1a6..78ee37b 100644
--- a/src/soc/intel/elkhartlake/i2c.c
+++ b/src/soc/intel/elkhartlake/i2c.c
@@ -19,6 +19,10 @@
return 4;
case PCH_DEVFN_I2C5:
return 5;
+ case PCH_DEVFN_I2C6:
+ return 6;
+ case PCH_DEVFN_I2C7:
+ return 7;
}
return -1;
}
@@ -38,6 +42,10 @@
return PCH_DEVFN_I2C4;
case 5:
return PCH_DEVFN_I2C5;
+ case 6:
+ return PCH_DEVFN_I2C6;
+ case 7:
+ return PCH_DEVFN_I2C7;
}
return -1;
}
diff --git a/src/soc/intel/elkhartlake/include/soc/serialio.h b/src/soc/intel/elkhartlake/include/soc/serialio.h
index e42af5f..bee96f0 100644
--- a/src/soc/intel/elkhartlake/include/soc/serialio.h
+++ b/src/soc/intel/elkhartlake/include/soc/serialio.h
@@ -18,13 +18,14 @@
PchSerialIoIndexI2C3,
PchSerialIoIndexI2C4,
PchSerialIoIndexI2C5,
+ PchSerialIoIndexI2C6,
+ PchSerialIoIndexI2C7,
};

enum {
PchSerialIoIndexGSPI0,
PchSerialIoIndexGSPI1,
PchSerialIoIndexGSPI2,
- PchSerialIoIndexGSPI3,
};

enum {

To view, visit change 48540. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If166fefe567a857ca29527d0367197139efbf6c7
Gerrit-Change-Number: 48540
Gerrit-PatchSet: 1
Gerrit-Owner: Lean Sheng Tan <lean.sheng.tan@intel.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-MessageType: newchange