Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson. Hello build bot (Jenkins), Jason Glenesk, Raul Rangel, Marshall Dawson,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/61570
to look at the new patch set (#2).
Change subject: soc/amd/common/block/i2c/i23c_pad_ctr: add & use I23C pad configuration ......................................................................
soc/amd/common/block/i2c/i23c_pad_ctr: add & use I23C pad configuration
I2C bus 0..2 on Sabrina uses a different pad type which supports 1.1V and 1.8V levels, but doesn't support 3.3V I2C levels. Compared to the existing I2C pad control registers the bit definitions are different, so add a separate function to configure those pads which however still has the same function signature and is compatible with same data structs used for the devicetree settings. PPR #57243 Rev 1.50 was used as a reference.
TEST=None
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: Ie210c3437f2608d1e9fb99dcb151fc4190721375 --- M src/soc/amd/common/block/i2c/Kconfig M src/soc/amd/common/block/i2c/Makefile.inc A src/soc/amd/common/block/i2c/i23c_pad_ctrl.c A src/soc/amd/common/block/i2c/i23c_pad_def.h M src/soc/amd/common/block/i2c/i2c_pad_def.h M src/soc/amd/common/block/include/amdblocks/i2c.h M src/soc/amd/sabrina/Kconfig M src/soc/amd/sabrina/i2c.c 8 files changed, 140 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/70/61570/2