Change in coreboot[master]: soc/amd/picasso: initialize i2c controllers in SoC flow

[Raul Rangel (Code Review)]

Hello Aaron Durbin,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/coreboot/+/40868

to review the following change.

Change subject: soc/amd/picasso: initialize i2c controllers in SoC flow
......................................................................

soc/amd/picasso: initialize i2c controllers in SoC flow

BUG=b:153642124
TEST=Saw I2C communication

Change-Id: I31f8b97d1ff7b687d7e078d5b594d1ad73c815e7
Signed-off-by: Aaron Durbin 
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/coreboot/+...
Commit-Queue: Furquan Shaikh 
Tested-by: Furquan Shaikh 
Reviewed-by: Furquan Shaikh 
---
M src/soc/amd/picasso/southbridge.c
1 file changed, 1 insertion(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/68/40868/1

diff --git a/src/soc/amd/picasso/southbridge.c b/src/soc/amd/picasso/southbridge.c
index 6bedab0..f31c81b 100644
--- a/src/soc/amd/picasso/southbridge.c
+++ b/src/soc/amd/picasso/southbridge.c
@@ -442,6 +442,7 @@

void southbridge_init(void *chip_info)
 {
+	i2c_soc_init();
    sb_init_acpi_ports();
    acpi_clear_pm1_status();
 }

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40868 )

Change subject: soc/amd/picasso: initialize i2c controllers in SoC flow
......................................................................

Patch Set 1: Code-Review+2

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40868 )

Change subject: soc/amd/picasso: initialize i2c controllers in SoC flow
......................................................................

Patch Set 1: Code-Review+2

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/40868 )

Change subject: soc/amd/picasso: initialize i2c controllers in SoC flow
......................................................................

soc/amd/picasso: initialize i2c controllers in SoC flow

BUG=b:153642124
TEST=Saw I2C communication

Change-Id: I31f8b97d1ff7b687d7e078d5b594d1ad73c815e7
Signed-off-by: Aaron Durbin 
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/coreboot/+...
Commit-Queue: Furquan Shaikh 
Tested-by: Furquan Shaikh 
Reviewed-by: Furquan Shaikh 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40868
Tested-by: build bot (Jenkins) 
Reviewed-by: Furquan Shaikh 
---
M src/soc/amd/picasso/southbridge.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Aaron Durbin: Looks good to me, approved
  Furquan Shaikh: Looks good to me, approved

diff --git a/src/soc/amd/picasso/southbridge.c b/src/soc/amd/picasso/southbridge.c
index d37f143..0d54294 100644
--- a/src/soc/amd/picasso/southbridge.c
+++ b/src/soc/amd/picasso/southbridge.c
@@ -467,6 +467,7 @@

void southbridge_init(void *chip_info)
 {
+	i2c_soc_init();
    sb_init_acpi_ports();
    acpi_clear_pm1_status();
 }