]> www.infradead.org Git - users/hch/dma-mapping.git/commitdiff
soc/tegra: regulators: Support core domain state syncing
authorDmitry Osipenko <digetx@gmail.com>
Tue, 1 Jun 2021 02:31:19 +0000 (05:31 +0300)
committerThierry Reding <treding@nvidia.com>
Wed, 2 Jun 2021 08:58:55 +0000 (10:58 +0200)
The core voltage shall not drop until state of core domain is synced,
i.e. all device drivers that use core domain are loaded and ready.

Support core domain state syncing. The core domain driver invokes the
core-regulator voltage syncing once the state of domain is synced, at
this point the core voltage is allowed to go lower than the level left
after bootloader.

Tested-by: Peter Geis <pgwipeout@gmail.com> # Ouya T30
Tested-by: Paul Fertser <fercerpav@gmail.com> # PAZ00 T20
Tested-by: Nicolas Chauvet <kwizart@gmail.com> # PAZ00 T20 and TK1 T124
Tested-by: Matt Merhar <mattmerhar@protonmail.com> # Ouya T30
Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/soc/tegra/regulators-tegra20.c
drivers/soc/tegra/regulators-tegra30.c

index 3479be5ee494a5e83c15989a239fa19cfc810c37..b8ce9fd0650ddc223134e29b31606cf9a899feb0 100644 (file)
@@ -17,6 +17,8 @@
 #include <linux/regulator/driver.h>
 #include <linux/regulator/machine.h>
 
+#include <soc/tegra/pmc.h>
+
 struct tegra_regulator_coupler {
        struct regulator_coupler coupler;
        struct regulator_dev *core_rdev;
@@ -42,6 +44,21 @@ static int tegra20_core_limit(struct tegra_regulator_coupler *tegra,
        int core_cur_uV;
        int err;
 
+       /*
+        * Tegra20 SoC has critical DVFS-capable devices that are
+        * permanently-active or active at a boot time, like EMC
+        * (DRAM controller) or Display controller for example.
+        *
+        * The voltage of a CORE SoC power domain shall not be dropped below
+        * a minimum level, which is determined by device's clock rate.
+        * This means that we can't fully allow CORE voltage scaling until
+        * the state of all DVFS-critical CORE devices is synced.
+        */
+       if (tegra_pmc_core_domain_state_synced() && !tegra->sys_reboot_mode) {
+               pr_info_once("voltage state synced\n");
+               return 0;
+       }
+
        if (tegra->core_min_uV > 0)
                return tegra->core_min_uV;
 
@@ -62,7 +79,7 @@ static int tegra20_core_limit(struct tegra_regulator_coupler *tegra,
         */
        tegra->core_min_uV = core_max_uV;
 
-       pr_info("core minimum voltage limited to %duV\n", tegra->core_min_uV);
+       pr_info("core voltage initialized to %duV\n", tegra->core_min_uV);
 
        return tegra->core_min_uV;
 }
index 18fe53d0a870dfa3b85209573a15ce34cfd37344..e74bbc9c78597f3425820b1760ada7efc37d6f91 100644 (file)
@@ -18,6 +18,7 @@
 #include <linux/regulator/machine.h>
 
 #include <soc/tegra/fuse.h>
+#include <soc/tegra/pmc.h>
 
 struct tegra_regulator_coupler {
        struct regulator_coupler coupler;
@@ -43,6 +44,21 @@ static int tegra30_core_limit(struct tegra_regulator_coupler *tegra,
        int core_cur_uV;
        int err;
 
+       /*
+        * Tegra30 SoC has critical DVFS-capable devices that are
+        * permanently-active or active at a boot time, like EMC
+        * (DRAM controller) or Display controller for example.
+        *
+        * The voltage of a CORE SoC power domain shall not be dropped below
+        * a minimum level, which is determined by device's clock rate.
+        * This means that we can't fully allow CORE voltage scaling until
+        * the state of all DVFS-critical CORE devices is synced.
+        */
+       if (tegra_pmc_core_domain_state_synced() && !tegra->sys_reboot_mode) {
+               pr_info_once("voltage state synced\n");
+               return 0;
+       }
+
        if (tegra->core_min_uV > 0)
                return tegra->core_min_uV;
 
@@ -63,7 +79,7 @@ static int tegra30_core_limit(struct tegra_regulator_coupler *tegra,
         */
        tegra->core_min_uV = core_max_uV;
 
-       pr_info("core minimum voltage limited to %duV\n", tegra->core_min_uV);
+       pr_info("core voltage initialized to %duV\n", tegra->core_min_uV);
 
        return tegra->core_min_uV;
 }