skylake: Generate ACPI timing values for I2C devices

Have the Skylake SOC generate ACPI timing values for the enabled I2C
controllers instead of passing it in the DSDT with static timings.

The timing values are generated from the controller clock speed and
are more accurate than the hardcoded values that were in the ASL which
were originally copied from Broadwell where the controller is running
at a different clock speed...

Additionally it is now possible for a board to override the values
using devicetree.cb.  If zero is passed in for SCL HCNT or LCNT then
the kernel will generate its own timing using the same forumla, but if
the SDA hold time value is zero the kernel will NOT generate a correct
value and the SDA hold time may be incorrect.

This was tested on the Chell platform to ensure all the I2C devices on
the board are still operational with these new timing values.

Change-Id: I4feb3df9e083592792f8fadd7105e081a984a906
Signed-off-by: Duncan Laurie <dlaurie@chromium.org>
Reviewed-on: https://review.coreboot.org/15291
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>

1 files changed, 0 insertions, 19 deletions

diff --git a/src/soc/intel/skylake/acpi/serialio.asl b/src/soc/intel/skylake/acpi/serialio.asl
index f274be83eb..849679be62 100644
--- a/src/soc/intel/skylake/acpi/serialio.asl
+++ b/src/soc/intel/skylake/acpi/serialio.asl
@@ -20,55 +20,36 @@ Device (I2C0)
 {
 	Name (_ADR, 0x00150000)
 	Name (_DDN, "Serial IO I2C Controller 0")
-
-	Name (SSCN, Package () { 432, 507, 30 })
-	Name (FMCN, Package () { 72, 160, 30 })
 }
 
 Device (I2C1)
 {
 	Name (_ADR, 0x00150001)
 	Name (_DDN, "Serial IO I2C Controller 1")
-
-	Name (SSCN, Package () { 528, 640, 30 })
-	Name (FMCN, Package () { 128, 160, 30 })
-	Name (FPCN, Package () { 48, 64, 30})
 }
 
 Device (I2C2)
 {
 	Name (_ADR, 0x00150002)
 	Name (_DDN, "Serial IO I2C Controller 2")
-
-	Name (SSCN, Package () { 432, 507, 30 })
-	Name (FMCN, Package () { 72, 160, 30 })
 }
 
 Device (I2C3)
 {
 	Name (_ADR, 0x00150003)
 	Name (_DDN, "Serial IO I2C Controller 3")
-
-	Name (SSCN, Package () { 432, 507, 30 })
-	Name (FMCN, Package () { 72, 160, 30 })
 }
 
 Device (I2C4)
 {
 	Name (_ADR, 0x00190002)
 	Name (_DDN, "Serial IO I2C Controller 4")
-
-	Name (SSCN, Package () { 432, 507, 30 })
-	Name (FMCN, Package () { 72, 160, 30 })
 }
 
 Device (I2C5)
 {
 	Name (_ADR, 0x00190002)
 	Name (_DDN, "Serial IO I2C Controller 5")
-
-	Name (SSCN, Package () { 432, 507, 30 })
-	Name (FMCN, Package () { 72, 160, 30 })
 }
 
 Device (SPI0)