interconnect: qcom: icc-rpm: Enforce 2 or 0 bus clocks

For SMD RPM bus scaling to work, we need a pair of sleep-wake clocks.
The variable number of them we previously supported was only a hack
to keep the clocks required for QoS register access, but now that
these are separated, we can leave bus_clks to the actual bus clocks.

In cases where there is no actual bus scaling (such as A0NoC on MSM8996
and GNoC on SDM660 where the HLOS is only supposed to program the QoS
registers and the bus is either static or controlled remotely), allow
for no clock scaling with a boolean property.

Remove all the code related to allowing an arbitrary number of bus_clks,
replace the number by BUS_CLK_MAX (= 2) and guard the bus clock paths
to ensure they are not taken on non-scaling buses.

Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Link: https://lore.kernel.org/r/20230228-topic-qos-v8-6-ee696a2c15a9@linaro.org
Signed-off-by: Georgi Djakov <djakov@kernel.org>

diff --git a/drivers/interconnect/qcom/icc-rpm.c b/drivers/interconnect/qcom/icc-rpm.c
index c34f75703a66c975395b9a86ac959ffccf0a2a76..bebc7928af2473fabad28af32e7509265049c032 100644 (file)
--- a/drivers/interconnect/qcom/icc-rpm.c
+++ b/drivers/interconnect/qcom/icc-rpm.c
@@ -449,17 +449,9 @@ int qnoc_probe(struct platform_device *pdev)
        for (i = 0; i < cd_num; i++)
                qp->intf_clks[i].id = cds[i];
 
-       if (desc->num_bus_clocks) {
-               cds = desc->bus_clocks;
-               cd_num = desc->num_bus_clocks;
-       } else {
-               cds = bus_clocks;
-               cd_num = ARRAY_SIZE(bus_clocks);
-       }
-
-       for (i = 0; i < cd_num; i++)
-               qp->bus_clks[i].id = cds[i];
-       qp->num_bus_clks = cd_num;
+       qp->num_bus_clks = desc->no_clk_scaling ? 0 : NUM_BUS_CLKS;
+       for (i = 0; i < qp->num_bus_clks; i++)
+               qp->bus_clks[i].id = bus_clocks[i];
 
        qp->type = desc->type;
        qp->qos_offset = desc->qos_offset;

diff --git a/drivers/interconnect/qcom/icc-rpm.h b/drivers/interconnect/qcom/icc-rpm.h
index b9b63860042f89ddf51389051ce39232d40943ae..ee705edf19dd67bcf9c24bca2884bfd36d3482fe 100644 (file)
--- a/drivers/interconnect/qcom/icc-rpm.h
+++ b/drivers/interconnect/qcom/icc-rpm.h
@@ -25,7 +25,7 @@ enum qcom_icc_type {
 /**
  * struct qcom_icc_provider - Qualcomm specific interconnect provider
  * @provider: generic interconnect provider
- * @num_bus_clks: the total number of bus_clks clk_bulk_data entries
+ * @num_bus_clks: the total number of bus_clks clk_bulk_data entries (0 or 2)
  * @num_intf_clks: the total number of intf_clks clk_bulk_data entries
  * @type: the ICC provider type
  * @regmap: regmap for QoS registers read/write access
@@ -100,9 +100,9 @@ struct qcom_icc_desc {
        struct qcom_icc_node * const *nodes;
        size_t num_nodes;
        const char * const *bus_clocks;
-       size_t num_bus_clocks;
        const char * const *intf_clocks;
        size_t num_intf_clocks;
+       bool no_clk_scaling;
        enum qcom_icc_type type;
        const struct regmap_config *regmap_cfg;
        unsigned int qos_offset;

diff --git a/drivers/interconnect/qcom/msm8996.c b/drivers/interconnect/qcom/msm8996.c
index 9aedfc8de4bffa7787b24f594d77066f7dbb9b3d..dc9959a87df290b8f2628ff4f992a1a3c340b156 100644 (file)
--- a/drivers/interconnect/qcom/msm8996.c
+++ b/drivers/interconnect/qcom/msm8996.c
@@ -1819,6 +1819,7 @@ static const struct qcom_icc_desc msm8996_a0noc = {
        .num_nodes = ARRAY_SIZE(a0noc_nodes),
        .intf_clocks = a0noc_intf_clocks,
        .num_intf_clocks = ARRAY_SIZE(a0noc_intf_clocks),
+       .no_clk_scaling = true,
        .regmap_cfg = &msm8996_a0noc_regmap_config
 };
 

diff --git a/drivers/interconnect/qcom/sdm660.c b/drivers/interconnect/qcom/sdm660.c
index 0e8a96f4ce90d6abf472805286160092313e79cd..7ffaf70d62d3dad92fe6714f71e24cb37104ac2b 100644 (file)
--- a/drivers/interconnect/qcom/sdm660.c
+++ b/drivers/interconnect/qcom/sdm660.c
@@ -1616,6 +1616,7 @@ static const struct qcom_icc_desc sdm660_gnoc = {
        .nodes = sdm660_gnoc_nodes,
        .num_nodes = ARRAY_SIZE(sdm660_gnoc_nodes),
        .regmap_cfg = &sdm660_gnoc_regmap_config,
+       .no_clk_scaling = true,
 };
 
 static struct qcom_icc_node * const sdm660_mnoc_nodes[] = {