diff options
| author | Raul E Rangel <rrangel@chromium.org> | 2020-04-09 11:37:05 -0600 |
|---|---|---|
| committer | Furquan Shaikh <furquan@google.com> | 2020-05-15 17:32:47 +0000 |
| commit | 6191b85f8770ed807a5954339aef6c99609d7660 (patch) | |
| tree | fc48ee809df2af79212bbaa8f79ddbfa7e9e89bd /src/drivers/intel/fsp2_0/header_display.c | |
| parent | 6ad701054230e25fa9b2afe414ca8da6d5a626ee (diff) | |
drivers/i2c/designware: Check if the device is powered
If the device doesn't return a valid component type, that means the
device is non-functional.
The dw_i2c_regs had invalid offsets for the version field. I got the
correct value from the DesignWare DW_apb_i2c Databook v2.02a. It also
matches what the Picasso PPR says.
I also print out the version field of the controller.
BUG=b:153001807
BRANCH=none
TEST=Tested on PSP where I2C is non functional. Also tested on trembyle
and verified i2c was initialized.
Saw the following in the logs
I2C bus 2 version 0x3132322a
Signed-off-by: Raul E Rangel <rrangel@chromium.org>
Change-Id: If5527972508e0f4b35cc9ecdb1491b1ce85ff3af
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/coreboot/+/2144540
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Furquan Shaikh <furquan@chromium.org>
Tested-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40870
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/drivers/intel/fsp2_0/header_display.c')
0 files changed, 0 insertions, 0 deletions