soundwire: generic_bandwidth_allocation use bus->params.max_dr_freq
authorBard Liao <yung-chuan.liao@linux.intel.com>
Mon, 27 Nov 2023 12:44:05 +0000 (20:44 +0800)
committerVinod Koul <vkoul@kernel.org>
Tue, 28 Nov 2023 09:59:55 +0000 (15:29 +0530)
bus->params.max_dr_freq is calculated and set in sdw_bus_master_add().
We can use it directly instead of calculating it again.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Link: https://lore.kernel.org/r/20231127124405.2080431-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/soundwire/generic_bandwidth_allocation.c

index 31162f2b563811569dcf1bec7a0fc0757efa72f4..c70a63d009ae4b0ae4184952540f3a48b2b400d3 100644 (file)
@@ -333,7 +333,7 @@ static int sdw_select_row_col(struct sdw_bus *bus, int clk_freq)
  */
 static int sdw_compute_bus_params(struct sdw_bus *bus)
 {
-       unsigned int max_dr_freq, curr_dr_freq = 0;
+       unsigned int curr_dr_freq = 0;
        struct sdw_master_prop *mstr_prop = &bus->prop;
        int i, clk_values, ret;
        bool is_gear = false;
@@ -351,14 +351,12 @@ static int sdw_compute_bus_params(struct sdw_bus *bus)
                clk_buf = NULL;
        }
 
-       max_dr_freq = mstr_prop->max_clk_freq * SDW_DOUBLE_RATE_FACTOR;
-
        for (i = 0; i < clk_values; i++) {
                if (!clk_buf)
-                       curr_dr_freq = max_dr_freq;
+                       curr_dr_freq = bus->params.max_dr_freq;
                else
                        curr_dr_freq = (is_gear) ?
-                               (max_dr_freq >>  clk_buf[i]) :
+                               (bus->params.max_dr_freq >>  clk_buf[i]) :
                                clk_buf[i] * SDW_DOUBLE_RATE_FACTOR;
 
                if (curr_dr_freq <= bus->params.bandwidth)