diff options
author | Aaron Durbin <adurbin@chromium.org> | 2018-01-30 17:29:08 -0700 |
---|---|---|
committer | Aaron Durbin <adurbin@chromium.org> | 2018-02-02 05:05:39 +0000 |
commit | 97d58bcc09b2a462d0bd55d1f675b687277b04b7 (patch) | |
tree | 8f36fb65807aba4b9f94421d93505197d07e6d87 /src/northbridge | |
parent | f18c0415efad4bca3b900e5c0658238a48afe3e4 (diff) | |
download | coreboot-97d58bcc09b2a462d0bd55d1f675b687277b04b7.tar.xz |
drivers/i2c/designware: conform to controller restrictions
The designware i2c controller indicates that the slave address
shouldn't be programmed while the controller is enabled. Therefore,
switch the ordering of the slave target address and the enable.
Additionally, ensure the controller is disabled prior to the
start of the slave programming sequence.
Lastly, chunk up the i2c_msg segments at differing slave address
boundaries. That allows for simpler programming for the controller
by only doing one slave address transaction chunk at a time.
BUG=b:70232394,b:69250772
Change-Id: Iebc08e2db847cb182fad98e0ff3d799b9a64aca7
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/23513
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Diffstat (limited to 'src/northbridge')
0 files changed, 0 insertions, 0 deletions