clk: tegra: dfll: Reference CVB table instead of copying data
authorThierry Reding <treding@nvidia.com>
Fri, 8 Apr 2016 13:02:06 +0000 (15:02 +0200)
committerThierry Reding <treding@nvidia.com>
Thu, 28 Apr 2016 10:41:53 +0000 (12:41 +0200)
Instead of copying parts of the CVB table into a separate structure,
keep track of the selected CVB table and directly reference data from
it.

Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/clk/tegra/clk-dfll.c
drivers/clk/tegra/clk-dfll.h
drivers/clk/tegra/clk-tegra124-dfll-fcpu.c

index 19bfa07e24b1b67baccc838a016ce4897cb61a11..f010562534ebe056bc67ed57bad8f9a48c9458ff 100644 (file)
@@ -55,6 +55,7 @@
 #include <linux/seq_file.h>
 
 #include "clk-dfll.h"
+#include "cvb.h"
 
 /*
  * DFLL control registers - access via dfll_{readl,writel}
@@ -442,8 +443,8 @@ static void dfll_tune_low(struct tegra_dfll *td)
 {
        td->tune_range = DFLL_TUNE_LOW;
 
-       dfll_writel(td, td->soc->tune0_low, DFLL_TUNE0);
-       dfll_writel(td, td->soc->tune1, DFLL_TUNE1);
+       dfll_writel(td, td->soc->cvb->cpu_dfll_data.tune0_low, DFLL_TUNE0);
+       dfll_writel(td, td->soc->cvb->cpu_dfll_data.tune1, DFLL_TUNE1);
        dfll_wmb(td);
 
        if (td->soc->set_clock_trimmers_low)
@@ -1449,7 +1450,7 @@ static int dfll_build_i2c_lut(struct tegra_dfll *td)
        }
        v_max = dev_pm_opp_get_voltage(opp);
 
-       v = td->soc->min_millivolts * 1000;
+       v = td->soc->cvb->min_millivolts * 1000;
        lut = find_vdd_map_entry_exact(td, v);
        if (lut < 0)
                goto out;
@@ -1461,7 +1462,7 @@ static int dfll_build_i2c_lut(struct tegra_dfll *td)
                        break;
                v_opp = dev_pm_opp_get_voltage(opp);
 
-               if (v_opp <= td->soc->min_millivolts * 1000)
+               if (v_opp <= td->soc->cvb->min_millivolts * 1000)
                        td->dvco_rate_min = dev_pm_opp_get_freq(opp);
 
                for (;;) {
@@ -1490,7 +1491,7 @@ static int dfll_build_i2c_lut(struct tegra_dfll *td)
 
        if (!td->dvco_rate_min)
                dev_err(td->dev, "no opp above DFLL minimum voltage %d mV\n",
-                       td->soc->min_millivolts);
+                       td->soc->cvb->min_millivolts);
        else
                ret = 0;
 
index 7e66c07c4d2b7a96d64c2857c992cec75d38ea52..d192982b1f96414980fdcb857b9851601a7fc9f5 100644 (file)
 /**
  * struct tegra_dfll_soc_data - SoC-specific hooks/integration for the DFLL driver
  * @dev: struct device * that holds the OPP table for the DFLL
- * @min_millivolts: minimum voltage (in mV) that the DFLL can operate
- * @tune0_low: DFLL tuning register 0 (low voltage range)
- * @tune0_high: DFLL tuning register 0 (high voltage range)
- * @tune1: DFLL tuning register 1
+ * @cvb: CPU frequency table for this SoC
  * @init_clock_trimmers: callback to initialize clock trimmers
  * @set_clock_trimmers_high: callback to tune clock trimmers for high voltage
  * @set_clock_trimmers_low: callback to tune clock trimmers for low voltage
  */
 struct tegra_dfll_soc_data {
        struct device *dev;
-       unsigned int min_millivolts;
-       u32 tune0_low;
-       u32 tune0_high;
-       u32 tune1;
+       const struct cvb_table *cvb;
 
        void (*init_clock_trimmers)(void);
        void (*set_clock_trimmers_high)(void);
index 61253330c12bbe07e0e42bfa239b6c4bb276156f..6bfa5ca40d02009ddeec868576500392d62ed239 100644 (file)
@@ -86,7 +86,6 @@ static int tegra124_dfll_fcpu_probe(struct platform_device *pdev)
 {
        int process_id, speedo_id, speedo_value;
        struct tegra_dfll_soc_data *soc;
-       const struct cvb_table *cvb;
 
        process_id = tegra_sku_info.cpu_process_id;
        speedo_id = tegra_sku_info.cpu_speedo_id;
@@ -108,21 +107,17 @@ static int tegra124_dfll_fcpu_probe(struct platform_device *pdev)
                return -ENODEV;
        }
 
-       cvb = tegra_cvb_build_opp_table(tegra124_cpu_cvb_tables,
-                                       ARRAY_SIZE(tegra124_cpu_cvb_tables),
-                                       process_id, speedo_id, speedo_value,
-                                       cpu_max_freq_table[speedo_id],
-                                       soc->dev);
-       if (IS_ERR(cvb)) {
-               dev_err(&pdev->dev, "couldn't build OPP table: %ld\n",
-                       PTR_ERR(cvb));
-               return PTR_ERR(cvb);
+       soc->cvb = tegra_cvb_build_opp_table(tegra124_cpu_cvb_tables,
+                                            ARRAY_SIZE(tegra124_cpu_cvb_tables),
+                                            process_id, speedo_id, speedo_value,
+                                            cpu_max_freq_table[speedo_id],
+                                            soc->dev);
+       if (IS_ERR(soc->cvb)) {
+               dev_err(&pdev->dev, "couldn't add OPP table: %ld\n",
+                       PTR_ERR(soc->cvb));
+               return PTR_ERR(soc->cvb);
        }
 
-       soc->min_millivolts = cvb->min_millivolts;
-       soc->tune0_low = cvb->cpu_dfll_data.tune0_low;
-       soc->tune0_high = cvb->cpu_dfll_data.tune0_high;
-       soc->tune1 = cvb->cpu_dfll_data.tune1;
 
        return tegra_dfll_register(pdev, soc);
 }