[coreboot-gerrit] Change in coreboot[master]: soc/amd/stoneyridge: use SMBus speed in compilation unit

[Aaron Durbin (Code Review)]

Aaron Durbin has submitted this change. ( https://review.coreboot.org/c/coreboot/+/38611 )

Change subject: soc/amd/stoneyridge: use SMBus speed in compilation unit
......................................................................

soc/amd/stoneyridge: use SMBus speed in compilation unit

The fixed bus speed of 400 kHz doesn't need to reside in a header file.
Just move the assumption into the code itself.

Change-Id: I426fe078909a9b725c1747380d69af31292b6d1e
Signed-off-by: Aaron Durbin 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38611
Reviewed-by: Angel Pons 
Reviewed-by: Kyösti Mälkki 
Tested-by: build bot (Jenkins) 
---
M src/soc/amd/stoneyridge/include/soc/smbus.h
M src/soc/amd/stoneyridge/southbridge.c
2 files changed, 4 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Kyösti Mälkki: Looks good to me, but someone else must approve
  Angel Pons: Looks good to me, approved

diff --git a/src/soc/amd/stoneyridge/include/soc/smbus.h b/src/soc/amd/stoneyridge/include/soc/smbus.h
index 391084d..4638fcd 100644
--- a/src/soc/amd/stoneyridge/include/soc/smbus.h
+++ b/src/soc/amd/stoneyridge/include/soc/smbus.h
@@ -19,8 +19,6 @@
 #include <stdint.h>
 #include <soc/iomap.h>

-#define SMB_SPEED_400KHZ		(66000000 / (400000 * 4))
-
 /*
  * Between 1-10 seconds, We should never timeout normally
  * Longer than this is just painful when a timeout condition occurs.
diff --git a/src/soc/amd/stoneyridge/southbridge.c b/src/soc/amd/stoneyridge/southbridge.c
index 1b2afec..b0aaf24 100644
--- a/src/soc/amd/stoneyridge/southbridge.c
+++ b/src/soc/amd/stoneyridge/southbridge.c
@@ -378,8 +378,11 @@

static void fch_smbus_init(void)
 {
+	/* 400 kHz smbus speed. */
+	const uint8_t smbus_speed = (66000000 / (400000 * 4));
+
    pm_write8(SMB_ASF_IO_BASE, SMB_BASE_ADDR >> 8);
-	smbus_write8(SMBTIMING, SMB_SPEED_400KHZ);
+	smbus_write8(SMBTIMING, smbus_speed);
    /* Clear all SMBUS status bits */
    smbus_write8(SMBHSTSTAT, SMBHST_STAT_CLEAR);
    smbus_write8(SMBSLVSTAT, SMBSLV_STAT_CLEAR);