Revert "gpu: nvgpu: fix gpcclk for K4.4"

This reverts commit a918003694984b6fca9e6b6c07fd7cdf3503055e.

Change-Id: Idf39cc0946c5c4df82c7c4b6afa225b1f8d5a923
Signed-off-by: Srikar Srimath Tirumala <srikars@nvidia.com>
Reviewed-on: http://git-master/r/1280827
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

2 files changed, 20 insertions, 76 deletions

diff --git a/drivers/gpu/nvgpu/gk20a/gk20a_sysfs.c b/drivers/gpu/nvgpu/gk20a/gk20a_sysfs.c
index 6611b120..f7e0036f 100644
--- a/drivers/gpu/nvgpu/gk20a/gk20a_sysfs.c
+++ b/drivers/gpu/nvgpu/gk20a/gk20a_sysfs.c
@@ -18,16 +18,12 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include <linux/version.h>
 #include <linux/device.h>
 #include <linux/pm_runtime.h>
 #include <linux/kernel.h>
 #include <linux/fb.h>
 #include <linux/gk20a.h>
 #include <linux/clk/tegra.h>
-#if (LINUX_VERSION_CODE >= KERNEL_VERSION(4, 4, 0))
-#include <soc/tegra/tegra-dvfs.h>
-#endif
 
 #include "gk20a.h"
 #include "gr_gk20a.h"
diff --git a/drivers/gpu/nvgpu/gm20b/clk_gm20b.c b/drivers/gpu/nvgpu/gm20b/clk_gm20b.c
index 05dd455c..be194228 100644
--- a/drivers/gpu/nvgpu/gm20b/clk_gm20b.c
+++ b/drivers/gpu/nvgpu/gm20b/clk_gm20b.c
@@ -371,18 +371,6 @@ static void clk_config_dvfs_ndiv(int mv, u32 n_eff, struct na_dvfs *d)
         d->sdm_din = (d->sdm_din >> BITS_PER_BYTE) & 0xff;
 }
 
-static void gm20b_calc_dvfs_safe_max_freq(struct clk *c)
-{
-        unsigned long safe_rate;
-
-        if (dvfs_safe_max_freq)
-                return;
-
-        safe_rate = tegra_dvfs_get_fmax_at_vmin_safe_t(c);
-        safe_rate = safe_rate * (100 - DVFS_SAFE_MARGIN) / 100;
-        dvfs_safe_max_freq = rate_gpu_to_gpc2clk(safe_rate);
-}
-
 /* Voltage dependent configuration */
 static void clk_config_dvfs(struct gk20a *g, struct pll *gpll)
 {
@@ -394,7 +382,6 @@ static void clk_config_dvfs(struct gk20a *g, struct pll *gpll)
         clk = clk_get_parent(clk);
 #endif
 
-        gm20b_calc_dvfs_safe_max_freq(clk);
         d->mv = tegra_dvfs_predict_mv_at_hz_cur_tfloor(clk,
                         rate_gpc2clk_to_gpu(gpll->freq));
 
@@ -1132,23 +1119,32 @@ static int gm20b_init_clk_reset_enable_hw(struct gk20a *g)
         return 0;
 }
 
-#ifdef CONFIG_TEGRA_CLK_FRAMEWORK
-static int gm20b_init_gpc_pll(struct gk20a *g)
+static int gm20b_init_clk_setup_sw(struct gk20a *g)
 {
         struct clk_gk20a *clk = &g->clk;
-        struct clk *c, *ref;
+        unsigned long safe_rate;
+        struct clk *ref, *c;
+
+        gk20a_dbg_fn("");
+
+        if (clk->sw_ready) {
+                gk20a_dbg_fn("skip init");
+                return 0;
+        }
 
         if (!gk20a_clk_get(g))
                 return -EINVAL;
 
+        c = clk->tegra_clk;
+#ifdef CONFIG_TEGRA_CLK_FRAMEWORK
         /*
          * On Tegra GPU clock exposed to frequency governor is a shared user on
          * GPCPLL bus (gbus). The latter can be accessed as GPU clock parent.
          * Respectively the grandparent is PLL reference clock.
          */
-        c = clk_get_parent(clk->tegra_clk);
+        c = clk_get_parent(c);
+#endif
         ref = clk_get_parent(c);
-
         if (IS_ERR(ref)) {
                 gk20a_err(dev_from_gk20a(g),
                         "failed to get GPCPLL reference clock");
@@ -1163,7 +1159,9 @@ static int gm20b_init_gpc_pll(struct gk20a *g)
                 return -EINVAL;
         }
 
-        gm20b_calc_dvfs_safe_max_freq(c);
+        safe_rate = tegra_dvfs_get_fmax_at_vmin_safe_t(c);
+        safe_rate = safe_rate * (100 - DVFS_SAFE_MARGIN) / 100;
+        dvfs_safe_max_freq = rate_gpu_to_gpc2clk(safe_rate);
         clk->gpc_pll.PL = (dvfs_safe_max_freq == 0) ? 0 :
                 DIV_ROUND_UP(gpc_pll_params.min_vco, dvfs_safe_max_freq);
 
@@ -1175,51 +1173,7 @@ static int gm20b_init_gpc_pll(struct gk20a *g)
         clk->gpc_pll.freq = clk->gpc_pll.clk_in * clk->gpc_pll.N;
         clk->gpc_pll.freq /= pl_to_div(clk->gpc_pll.PL);
 
-        return 0;
-}
-
-#else /*COMMON_CLOCK_FRAMEWORK*/
-static int gm20b_init_gpc_pll(struct gk20a *g)
-{
-        struct clk_gk20a *clk = &g->clk;
-        struct clk *ref;
-
-        ref = clk_get_sys("gpu_ref", "gpu_ref");
-        if (IS_ERR(ref)) {
-                gk20a_err(dev_from_gk20a(g),
-                        "failed to get GPCPLL reference clock");
-                return -EINVAL;
-        }
-        clk->gpc_pll.id = GK20A_GPC_PLL;
-        clk->gpc_pll.clk_in = clk_get_rate(ref) / KHZ;
-
-        /* Initial freq: low enough to be safe at Vmin (default 1/3 VCO min) */
-        clk->gpc_pll.M = 1;
-        clk->gpc_pll.N = DIV_ROUND_UP(gpc_pll_params.min_vco,
-                                clk->gpc_pll.clk_in);
-        clk->gpc_pll.PL = 3;
-        clk->gpc_pll.freq = clk->gpc_pll.clk_in * clk->gpc_pll.N;
-        clk->gpc_pll.freq /= pl_to_div(clk->gpc_pll.PL);
-
-        return 0;
-}
-#endif
-
-static int gm20b_init_clk_setup_sw(struct gk20a *g)
-{
-        struct clk_gk20a *clk = &g->clk;
-
-        gk20a_dbg_fn("");
-
-        if (clk->sw_ready) {
-                gk20a_dbg_fn("skip init");
-                return 0;
-        }
-
-        if (gm20b_init_gpc_pll(g))
-                return -EINVAL;
-
-        /*
+         /*
           * All production parts should have ADC fuses burnt. Therefore, check
           * ADC fuses always, regardless of whether NA mode is selected; and if
           * NA mode is indeed selected, and part can support it, switch to NA
@@ -1237,6 +1191,7 @@ static int gm20b_init_clk_setup_sw(struct gk20a *g)
 #endif
 
         mutex_init(&clk->clk_mutex);
+
         clk->sw_ready = true;
 
         gk20a_dbg_fn("done");
@@ -1346,11 +1301,6 @@ int gm20b_register_gpcclk(struct gk20a *g) {
         struct clk_gk20a *clk = &g->clk;
         struct clk_init_data init;
         struct clk *c;
-        int err = 0;
-
-        err = gm20b_init_clk_setup_sw(g);
-        if (err)
-                return err;
 
         init.name = "gpcclk";
         init.ops = &gk20a_clk_ops;
@@ -1370,7 +1320,7 @@ int gm20b_register_gpcclk(struct gk20a *g) {
         clk->tegra_clk = c;
         clk_register_clkdev(c, "gpcclk", "gpcclk");
 
-        return err;
+        return 0;
 }
 #endif /* CONFIG_COMMON_CLK */
 
@@ -1580,11 +1530,9 @@ static int gm20b_init_clk_support(struct gk20a *g)
         if (err)
                 return err;
 
-#ifdef CONFIG_TEGRA_CLK_FRAMEWORK
         err = gm20b_init_clk_setup_sw(g);
         if (err)
                 return err;
-#endif
 
         mutex_lock(&clk->clk_mutex);
         clk->clk_hw_on = true;