i2c: i2c-qcom-geni: Add interconnect support
authorAkash Asthana <akashast@codeaurora.org>
Tue, 23 Jun 2020 10:38:52 +0000 (16:08 +0530)
committerBjorn Andersson <bjorn.andersson@linaro.org>
Thu, 25 Jun 2020 04:50:11 +0000 (21:50 -0700)
Get the interconnect paths for I2C based Serial Engine device
and vote according to the bus speed of the driver.

Signed-off-by: Akash Asthana <akashast@codeaurora.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
Acked-by: Wolfram Sang <wsa@kernel.org>
Link: https://lore.kernel.org/r/1592908737-7068-4-git-send-email-akashast@codeaurora.org
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
drivers/i2c/busses/i2c-qcom-geni.c

index 18d1e4f..32b2a99 100644 (file)
@@ -557,6 +557,22 @@ static int geni_i2c_probe(struct platform_device *pdev)
        gi2c->adap.dev.of_node = dev->of_node;
        strlcpy(gi2c->adap.name, "Geni-I2C", sizeof(gi2c->adap.name));
 
+       ret = geni_icc_get(&gi2c->se, "qup-memory");
+       if (ret)
+               return ret;
+       /*
+        * Set the bus quota for core and cpu to a reasonable value for
+        * register access.
+        * Set quota for DDR based on bus speed.
+        */
+       gi2c->se.icc_paths[GENI_TO_CORE].avg_bw = GENI_DEFAULT_BW;
+       gi2c->se.icc_paths[CPU_TO_GENI].avg_bw = GENI_DEFAULT_BW;
+       gi2c->se.icc_paths[GENI_TO_DDR].avg_bw = Bps_to_icc(gi2c->clk_freq_out);
+
+       ret = geni_icc_set_bw(&gi2c->se);
+       if (ret)
+               return ret;
+
        ret = geni_se_resources_on(&gi2c->se);
        if (ret) {
                dev_err(dev, "Error turning on resources %d\n", ret);
@@ -579,6 +595,10 @@ static int geni_i2c_probe(struct platform_device *pdev)
                return ret;
        }
 
+       ret = geni_icc_disable(&gi2c->se);
+       if (ret)
+               return ret;
+
        dev_dbg(dev, "i2c fifo/se-dma mode. fifo depth:%d\n", tx_depth);
 
        gi2c->suspended = 1;
@@ -623,7 +643,7 @@ static int __maybe_unused geni_i2c_runtime_suspend(struct device *dev)
                gi2c->suspended = 1;
        }
 
-       return 0;
+       return geni_icc_disable(&gi2c->se);
 }
 
 static int __maybe_unused geni_i2c_runtime_resume(struct device *dev)
@@ -631,6 +651,10 @@ static int __maybe_unused geni_i2c_runtime_resume(struct device *dev)
        int ret;
        struct geni_i2c_dev *gi2c = dev_get_drvdata(dev);
 
+       ret = geni_icc_enable(&gi2c->se);
+       if (ret)
+               return ret;
+
        ret = geni_se_resources_on(&gi2c->se);
        if (ret)
                return ret;