i2c: tegra: Remove redundant check in tegra_i2c_issue_bus_clear()
authorDmitry Osipenko <digetx@gmail.com>
Tue, 29 Sep 2020 22:19:02 +0000 (01:19 +0300)
committerWolfram Sang <wsa@kernel.org>
Mon, 5 Oct 2020 20:57:42 +0000 (22:57 +0200)
The tegra_i2c_wait_for_config_load() checks for 'has_config_load_reg' by
itself, hence there is no need to duplicate the check.

Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Acked-by: Thierry Reding <treding@nvidia.com>
Tested-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
drivers/i2c/busses/i2c-tegra.c

index dc25578..79fa984 100644 (file)
@@ -1082,11 +1082,10 @@ static int tegra_i2c_issue_bus_clear(struct i2c_adapter *adap)
        reg = FIELD_PREP(I2C_BC_SCLK_THRESHOLD, 9) | I2C_BC_STOP_COND |
              I2C_BC_TERMINATE;
        i2c_writel(i2c_dev, reg, I2C_BUS_CLEAR_CNFG);
-       if (i2c_dev->hw->has_config_load_reg) {
-               err = tegra_i2c_wait_for_config_load(i2c_dev);
-               if (err)
-                       return err;
-       }
+
+       err = tegra_i2c_wait_for_config_load(i2c_dev);
+       if (err)
+               return err;
 
        reg |= I2C_BC_ENABLE;
        i2c_writel(i2c_dev, reg, I2C_BUS_CLEAR_CNFG);