diff options
| author |   Jarkko Nikula <jarkko.nikula@linux.intel.com> | 2018-06-19 14:23:24 +0300 |
|---|---|---|
| committer |   Wolfram Sang <wsa@the-dreams.de> | 2018-07-03 23:06:11 +0200 |
| commit | 1706a96b30d6011e3f6829b365ace2030a306fe3 (patch) | |
| tree | 95b462c3ae3e430312c06548a80724ad97f1318e /drivers/i2c | |
| parent | i2c: designware: Separate timing parameter setting from HW initalization (diff) | |
| download | linux-dev-1706a96b30d6011e3f6829b365ace2030a306fe3.tar.xz linux-dev-1706a96b30d6011e3f6829b365ace2030a306fe3.zip | |
i2c: designware: Add debug print for SDA hold time value
SDA hold time is an important timing parameter and often reason for
arbitration lost errors if not set to a correct value. Add a debug print
for it in order to see what value gets programmed to a HW.
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Diffstat (limited to 'drivers/i2c')
| -rw-r--r-- | drivers/i2c/busses/i2c-designware-common.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/i2c/busses/i2c-designware-common.c b/drivers/i2c/busses/i2c-designware-common.c index 6f1c773c0417..bcc1bcbf0350 100644 --- a/drivers/i2c/busses/i2c-designware-common.c +++ b/drivers/i2c/busses/i2c-designware-common.c @@ -210,6 +210,10 @@ int i2c_dw_set_sda_hold(struct dw_i2c_dev *dev) */ if (!(dev->sda_hold_time & DW_IC_SDA_HOLD_RX_MASK)) dev->sda_hold_time |= 1 << DW_IC_SDA_HOLD_RX_SHIFT; + + dev_dbg(dev->dev, "SDA Hold Time TX:RX = %d:%d\n", + dev->sda_hold_time & ~(u32)DW_IC_SDA_HOLD_RX_MASK, + dev->sda_hold_time >> DW_IC_SDA_HOLD_RX_SHIFT); } else if (dev->sda_hold_time) { dev_warn(dev->dev, "Hardware too old to adjust SDA hold time.\n"); |