ipq806x: drop old cpufreq patch
authorAnsuel Smith <ansuelsmth@gmail.com>
Sun, 15 Dec 2019 01:39:38 +0000 (02:39 +0100)
committerPetr Štetiar <ynezz@true.cz>
Thu, 26 Dec 2019 07:31:42 +0000 (08:31 +0100)
Drop old cpufreq as now we have new driver that
can use normal kernel opp definition

Tested-by: Stefan Lippers-Hollmann <s.l-h@gmx.de> [nbg6817/ipq8065]
Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
target/linux/ipq806x/patches-4.19/0045-cpufreq-Add-module-to-register-cpufreq-on-Krait-CPUs.patch [deleted file]
target/linux/ipq806x/patches-4.19/0046-cpufreq-qcom-independent-core-clocks.patch [deleted file]
target/linux/ipq806x/patches-4.19/0048-PM-OPP-HACK-Allow-to-set-regulator-without-opp_list.patch [deleted file]

diff --git a/target/linux/ipq806x/patches-4.19/0045-cpufreq-Add-module-to-register-cpufreq-on-Krait-CPUs.patch b/target/linux/ipq806x/patches-4.19/0045-cpufreq-Add-module-to-register-cpufreq-on-Krait-CPUs.patch
deleted file mode 100644 (file)
index 5b081fe..0000000
+++ /dev/null
@@ -1,307 +0,0 @@
-From patchwork Fri Dec  8 09:42:29 2017
-Content-Type: text/plain; charset="utf-8"
-MIME-Version: 1.0
-Content-Transfer-Encoding: 7bit
-Subject: [v4,11/12] cpufreq: Add module to register cpufreq on Krait CPUs
-From: Sricharan R <sricharan@codeaurora.org>
-X-Patchwork-Id: 10102075
-Message-Id: <1512726150-7204-12-git-send-email-sricharan@codeaurora.org>
-To: mturquette@baylibre.com, sboyd@codeaurora.org,
- devicetree@vger.kernel.org, linux-pm@vger.kernel.org,
- linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
- viresh.kumar@linaro.org, linux-arm-kernel@lists.infradead.org
-Cc: sricharan@codeaurora.org
-Date: Fri,  8 Dec 2017 15:12:29 +0530
-
-From: Stephen Boyd <sboyd@codeaurora.org>
-
-Register a cpufreq-generic device whenever we detect that a
-"qcom,krait" compatible CPU is present in DT.
-
-Cc: <devicetree@vger.kernel.org>
-Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
----
- .../devicetree/bindings/arm/msm/qcom,pvs.txt       |  38 ++++
- drivers/cpufreq/Kconfig.arm                        |   9 +
- drivers/cpufreq/Makefile                           |   1 +
- drivers/cpufreq/qcom-cpufreq.c                     | 204 +++++++++++++++++++++
- 4 files changed, 252 insertions(+)
- create mode 100644 Documentation/devicetree/bindings/arm/msm/qcom,pvs.txt
- create mode 100644 drivers/cpufreq/qcom-cpufreq.c
-
---- /dev/null
-+++ b/Documentation/devicetree/bindings/arm/msm/qcom,pvs.txt
-@@ -0,0 +1,38 @@
-+Qualcomm Process Voltage Scaling Tables
-+
-+The node name is required to be "qcom,pvs". There shall only be one
-+such node present in the root of the tree.
-+
-+PROPERTIES
-+
-+- qcom,pvs-format-a or qcom,pvs-format-b:
-+      Usage: required
-+      Value type: <empty>
-+      Definition: Indicates the format of qcom,speedX-pvsY-bin-vZ properties.
-+                  If qcom,pvs-format-a is used the table is two columns
-+                  (frequency and voltage in that order). If qcom,pvs-format-b                     is used the table is three columns (frequency, voltage,
-+                  and current in that order).
-+
-+- qcom,speedX-pvsY-bin-vZ:
-+      Usage: required
-+      Value type: <prop-encoded-array>
-+      Definition: The PVS table corresponding to the speed bin X, pvs bin Y,
-+                  and version Z.
-+Example:
-+
-+      qcom,pvs {
-+              qcom,pvs-format-a;
-+              qcom,speed0-pvs0-bin-v0 =
-+                      <  384000000  950000 >,
-+                      <  486000000  975000 >,
-+                      <  594000000 1000000 >,
-+                      <  702000000 1025000 >,
-+                      <  810000000 1075000 >,
-+                      <  918000000 1100000 >,
-+                      < 1026000000 1125000 >,
-+                      < 1134000000 1175000 >,
-+                      < 1242000000 1200000 >,
-+                      < 1350000000 1225000 >,
-+                      < 1458000000 1237500 >,
-+                      < 1512000000 1250000 >;
-+      };
---- a/drivers/cpufreq/Kconfig.arm
-+++ b/drivers/cpufreq/Kconfig.arm
-@@ -121,6 +121,15 @@ config ARM_QCOM_CPUFREQ_KRYO
-         If in doubt, say N.
-+config ARM_QCOM_CPUFREQ
-+      tristate "Qualcomm based"
-+      depends on ARCH_QCOM
-+      select PM_OPP
-+      help
-+        This adds the CPUFreq driver for Qualcomm SoC based boards.
-+
-+        If in doubt, say N.
-+
- config ARM_S3C_CPUFREQ
-       bool
-       help
---- a/drivers/cpufreq/Makefile
-+++ b/drivers/cpufreq/Makefile
-@@ -64,6 +64,7 @@ obj-$(CONFIG_MACH_MVEBU_V7)          += mvebu-cp
- obj-$(CONFIG_ARM_OMAP2PLUS_CPUFREQ)   += omap-cpufreq.o
- obj-$(CONFIG_ARM_PXA2xx_CPUFREQ)      += pxa2xx-cpufreq.o
- obj-$(CONFIG_PXA3xx)                  += pxa3xx-cpufreq.o
-+obj-$(CONFIG_ARM_QCOM_CPUFREQ)                += qcom-cpufreq.o
- obj-$(CONFIG_ARM_QCOM_CPUFREQ_KRYO)   += qcom-cpufreq-kryo.o
- obj-$(CONFIG_ARM_S3C2410_CPUFREQ)     += s3c2410-cpufreq.o
- obj-$(CONFIG_ARM_S3C2412_CPUFREQ)     += s3c2412-cpufreq.o
---- /dev/null
-+++ b/drivers/cpufreq/qcom-cpufreq.c
-@@ -0,0 +1,204 @@
-+/* Copyright (c) 2014, The Linux Foundation. All rights reserved.
-+ *
-+ * This program is free software; you can redistribute it and/or modify
-+ * it under the terms of the GNU General Public License version 2 and
-+ * only version 2 as published by the Free Software Foundation.
-+ *
-+ * This program is distributed in the hope that it will be useful,
-+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
-+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-+ * GNU General Public License for more details.
-+ */
-+
-+#include <linux/cpu.h>
-+#include <linux/err.h>
-+#include <linux/init.h>
-+#include <linux/io.h>
-+#include <linux/kernel.h>
-+#include <linux/module.h>
-+#include <linux/of.h>
-+#include <linux/platform_device.h>
-+#include <linux/pm_opp.h>
-+#include <linux/slab.h>
-+#include "cpufreq-dt.h"
-+
-+static void __init get_krait_bin_format_a(int *speed, int *pvs, int *pvs_ver)
-+{
-+      void __iomem *base;
-+      u32 pte_efuse;
-+
-+      *speed = *pvs = *pvs_ver = 0;
-+
-+      base = ioremap(0x007000c0, 4);
-+      if (!base) {
-+              pr_warn("Unable to read efuse data. Defaulting to 0!\n");
-+              return;
-+      }
-+
-+      pte_efuse = readl_relaxed(base);
-+      iounmap(base);
-+
-+      *speed = pte_efuse & 0xf;
-+      if (*speed == 0xf)
-+              *speed = (pte_efuse >> 4) & 0xf;
-+
-+      if (*speed == 0xf) {
-+              *speed = 0;
-+              pr_warn("Speed bin: Defaulting to %d\n", *speed);
-+      } else {
-+              pr_info("Speed bin: %d\n", *speed);
-+      }
-+
-+      *pvs = (pte_efuse >> 10) & 0x7;
-+      if (*pvs == 0x7)
-+              *pvs = (pte_efuse >> 13) & 0x7;
-+
-+      if (*pvs == 0x7) {
-+              *pvs = 0;
-+              pr_warn("PVS bin: Defaulting to %d\n", *pvs);
-+      } else {
-+              pr_info("PVS bin: %d\n", *pvs);
-+      }
-+}
-+
-+static void __init get_krait_bin_format_b(int *speed, int *pvs, int *pvs_ver)
-+{
-+      u32 pte_efuse, redundant_sel;
-+      void __iomem *base;
-+
-+      *speed = 0;
-+      *pvs = 0;
-+      *pvs_ver = 0;
-+
-+      base = ioremap(0xfc4b80b0, 8);
-+      if (!base) {
-+              pr_warn("Unable to read efuse data. Defaulting to 0!\n");
-+              return;
-+      }
-+
-+      pte_efuse = readl_relaxed(base);
-+      redundant_sel = (pte_efuse >> 24) & 0x7;
-+      *speed = pte_efuse & 0x7;
-+      /* 4 bits of PVS are in efuse register bits 31, 8-6. */
-+      *pvs = ((pte_efuse >> 28) & 0x8) | ((pte_efuse >> 6) & 0x7);
-+      *pvs_ver = (pte_efuse >> 4) & 0x3;
-+
-+      switch (redundant_sel) {
-+      case 1:
-+              *speed = (pte_efuse >> 27) & 0xf;
-+              break;
-+      case 2:
-+              *pvs = (pte_efuse >> 27) & 0xf;
-+              break;
-+      }
-+
-+      /* Check SPEED_BIN_BLOW_STATUS */
-+      if (pte_efuse & BIT(3)) {
-+              pr_info("Speed bin: %d\n", *speed);
-+      } else {
-+              pr_warn("Speed bin not set. Defaulting to 0!\n");
-+              *speed = 0;
-+      }
-+
-+      /* Check PVS_BLOW_STATUS */
-+      pte_efuse = readl_relaxed(base + 0x4) & BIT(21);
-+      if (pte_efuse) {
-+              pr_info("PVS bin: %d\n", *pvs);
-+      } else {
-+              pr_warn("PVS bin not set. Defaulting to 0!\n");
-+              *pvs = 0;
-+      }
-+
-+      pr_info("PVS version: %d\n", *pvs_ver);
-+      iounmap(base);
-+}
-+
-+static int __init qcom_cpufreq_populate_opps(void)
-+{
-+      int len, rows, cols, i, k, speed, pvs, pvs_ver;
-+      char table_name[] = "qcom,speedXX-pvsXX-bin-vXX";
-+      struct device_node *np;
-+      struct device *dev;
-+      int cpu = 0;
-+
-+      np = of_find_node_by_name(NULL, "qcom,pvs");
-+      if (!np)
-+              return -ENODEV;
-+
-+      if (of_property_read_bool(np, "qcom,pvs-format-a")) {
-+              get_krait_bin_format_a(&speed, &pvs, &pvs_ver);
-+              cols = 2;
-+      } else if (of_property_read_bool(np, "qcom,pvs-format-b")) {
-+              get_krait_bin_format_b(&speed, &pvs, &pvs_ver);
-+              cols = 3;
-+      } else {
-+              return -ENODEV;
-+      }
-+
-+      snprintf(table_name, sizeof(table_name),
-+               "qcom,speed%d-pvs%d-bin-v%d", speed, pvs, pvs_ver);
-+
-+      if (!of_find_property(np, table_name, &len))
-+              return -EINVAL;
-+
-+      len /= sizeof(u32);
-+      if (len % cols || len == 0)
-+              return -EINVAL;
-+
-+      rows = len / cols;
-+
-+      for (i = 0, k = 0; i < rows; i++) {
-+              u32 freq, volt;
-+
-+              of_property_read_u32_index(np, table_name, k++, &freq);
-+              of_property_read_u32_index(np, table_name, k++, &volt);
-+              while (k % cols)
-+                      k++; /* Skip uA entries if present */
-+              for (cpu = 0; cpu < num_possible_cpus(); cpu++) {
-+                      dev = get_cpu_device(cpu);
-+                      if (!dev)
-+                              return -ENODEV;
-+                      if (dev_pm_opp_add(dev, freq, volt))
-+                              pr_warn("failed to add OPP %u\n", freq);
-+              }
-+      }
-+
-+      return 0;
-+}
-+
-+static int __init qcom_cpufreq_driver_init(void)
-+{
-+      struct cpufreq_dt_platform_data pdata = { .independent_clocks = true };
-+      struct platform_device_info devinfo = {
-+              .name = "cpufreq-dt",
-+              .data = &pdata,
-+              .size_data = sizeof(pdata),
-+      };
-+      struct device *cpu_dev;
-+      struct device_node *np;
-+      int ret;
-+
-+      cpu_dev = get_cpu_device(0);
-+      if (!cpu_dev)
-+              return -ENODEV;
-+
-+      np = of_node_get(cpu_dev->of_node);
-+      if (!np)
-+              return -ENOENT;
-+
-+      if (!of_device_is_compatible(np, "qcom,krait")) {
-+              of_node_put(np);
-+              return -ENODEV;
-+      }
-+      of_node_put(np);
-+
-+      ret = qcom_cpufreq_populate_opps();
-+      if (ret)
-+              return ret;
-+
-+      return PTR_ERR_OR_ZERO(platform_device_register_full(&devinfo));
-+}
-+module_init(qcom_cpufreq_driver_init);
-+
-+MODULE_DESCRIPTION("Qualcomm CPUfreq driver");
-+MODULE_LICENSE("GPL v2");
diff --git a/target/linux/ipq806x/patches-4.19/0046-cpufreq-qcom-independent-core-clocks.patch b/target/linux/ipq806x/patches-4.19/0046-cpufreq-qcom-independent-core-clocks.patch
deleted file mode 100644 (file)
index 7743dd1..0000000
+++ /dev/null
@@ -1,66 +0,0 @@
-From patchwork Fri Dec  8 09:42:30 2017
-Content-Type: text/plain; charset="utf-8"
-MIME-Version: 1.0
-Content-Transfer-Encoding: 7bit
-Subject: [v4,12/12] cpufreq: dt: Reintroduce independent_clocks platform data
-From: Sricharan R <sricharan@codeaurora.org>
-X-Patchwork-Id: 10102073
-Message-Id: <1512726150-7204-13-git-send-email-sricharan@codeaurora.org>
-To: mturquette@baylibre.com, sboyd@codeaurora.org,
- devicetree@vger.kernel.org, linux-pm@vger.kernel.org,
- linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
- viresh.kumar@linaro.org, linux-arm-kernel@lists.infradead.org
-Cc: sricharan@codeaurora.org
-Date: Fri,  8 Dec 2017 15:12:30 +0530
-
-The Platform data was removed earlier by,
-'commit eb96924acddc ("cpufreq: dt: Kill platform-data")'
-since there were no users at that time.
-Now this is required when the each of the cpu clocks
-can be scaled independently, which is the case
-for krait cores. So reintroduce it.
-
-Signed-off-by: Sricharan R <sricharan@codeaurora.org>
----
- drivers/cpufreq/cpufreq-dt.c | 7 ++++++-
- drivers/cpufreq/cpufreq-dt.h | 6 ++++++
- 2 files changed, 12 insertions(+), 1 deletion(-)
-
---- a/drivers/cpufreq/cpufreq-dt.c
-+++ b/drivers/cpufreq/cpufreq-dt.c
-@@ -239,7 +239,10 @@ static int cpufreq_init(struct cpufreq_p
-       }
-       if (fallback) {
--              cpumask_setall(policy->cpus);
-+              struct cpufreq_dt_platform_data *pd = cpufreq_get_driver_data();
-+
-+              if (!pd || !pd->independent_clocks)
-+                      cpumask_setall(policy->cpus);
-               /*
-                * OPP tables are initialized only for policy->cpu, do it for
-@@ -359,6 +362,8 @@ static int dt_cpufreq_probe(struct platf
-                       dt_cpufreq_driver.suspend = data->suspend;
-       }
-+      dt_cpufreq_driver.driver_data = data;
-+
-       ret = cpufreq_register_driver(&dt_cpufreq_driver);
-       if (ret)
-               dev_err(&pdev->dev, "failed register driver: %d\n", ret);
---- a/drivers/cpufreq/cpufreq-dt.h
-+++ b/drivers/cpufreq/cpufreq-dt.h
-@@ -15,6 +15,12 @@
- struct cpufreq_policy;
- struct cpufreq_dt_platform_data {
-+      /*
-+       * True when each CPU has its own clock to control its
-+       * frequency, false when all CPUs are controlled by a single
-+       * clock.
-+       */
-+      bool independent_clocks;
-       bool have_governor_per_policy;
-       int (*suspend)(struct cpufreq_policy *policy);
diff --git a/target/linux/ipq806x/patches-4.19/0048-PM-OPP-HACK-Allow-to-set-regulator-without-opp_list.patch b/target/linux/ipq806x/patches-4.19/0048-PM-OPP-HACK-Allow-to-set-regulator-without-opp_list.patch
deleted file mode 100644 (file)
index d415f7b..0000000
+++ /dev/null
@@ -1,26 +0,0 @@
-From 5c294df1715d673f94f3b0a6e1ea3a426ca35e6e Mon Sep 17 00:00:00 2001
-From: Georgi Djakov <georgi.djakov@linaro.org>
-Date: Thu, 28 Apr 2016 16:20:12 +0300
-Subject: [PATCH 48/69] PM / OPP: HACK: Allow to set regulator without opp_list
-
-Signed-off-by: Georgi Djakov <georgi.djakov@linaro.org>
----
- drivers/base/power/opp/core.c | 3 ++-
- 1 file changed, 2 insertions(+), 1 deletion(-)
-
---- a/drivers/opp/core.c
-+++ b/drivers/opp/core.c
-@@ -1317,11 +1317,13 @@ struct opp_table *dev_pm_opp_set_regulat
-       if (!opp_table)
-               return ERR_PTR(-ENOMEM);
-+#if 0
-       /* This should be called before OPPs are initialized */
-       if (WARN_ON(!list_empty(&opp_table->opp_list))) {
-               ret = -EBUSY;
-               goto err;
-       }
-+#endif
-       /* Another CPU that shares the OPP table has set the regulators ? */
-       if (opp_table->regulators)