i2c: designware: suppress unneeded SDA hold time warnings
authorAlexander Monakov <amonakov@ispras.ru>
Thu, 8 Mar 2018 13:23:53 +0000 (16:23 +0300)
committerWolfram Sang <wsa@the-dreams.de>
Sat, 17 Mar 2018 20:53:01 +0000 (21:53 +0100)
The hardware may not support SDA hold time configuration, but if it is
not set in the Device Tree either, there is no need to print a warning.

Reported-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Signed-off-by: Alexander Monakov <amonakov@ispras.ru>
Acked-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
drivers/i2c/busses/i2c-designware-master.c

index 05732531829fe090185d5f133ea2d7958a94e87a..fd36c39ddf4e86efd4ae8f12bd9f4ab9df4da860 100644 (file)
@@ -163,7 +163,7 @@ static int i2c_dw_init_master(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;
                dw_writel(dev, dev->sda_hold_time, DW_IC_SDA_HOLD);
-       } else {
+       } else if (dev->sda_hold_time) {
                dev_warn(dev->dev,
                        "Hardware too old to adjust SDA hold time.\n");
        }