* [PATCH] cpufreq: loongson1: Delete obsolete driver
@ 2023-01-12 13:53 Keguang Zhang
2023-01-20 16:49 ` Rafael J. Wysocki
0 siblings, 1 reply; 2+ messages in thread
From: Keguang Zhang @ 2023-01-12 13:53 UTC (permalink / raw)
To: linux-pm, linux-mips, linux-kernel
Cc: Rafael J . Wysocki, Viresh Kumar, Keguang Zhang
The generic DT based cpufreq driver works for Loongson-1,
so delete the old custom driver.
Signed-off-by: Keguang Zhang <keguang.zhang@gmail.com>
---
drivers/cpufreq/Kconfig | 9 --
drivers/cpufreq/Makefile | 1 -
drivers/cpufreq/loongson1-cpufreq.c | 222 ----------------------------
3 files changed, 232 deletions(-)
delete mode 100644 drivers/cpufreq/loongson1-cpufreq.c
diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig
index 2a84fc63371e..448b8ffb4ebd 100644
--- a/drivers/cpufreq/Kconfig
+++ b/drivers/cpufreq/Kconfig
@@ -270,15 +270,6 @@ config LOONGSON2_CPUFREQ
Loongson2F and its successors support this feature.
- If in doubt, say N.
-
-config LOONGSON1_CPUFREQ
- tristate "Loongson1 CPUFreq Driver"
- depends on LOONGSON1_LS1B
- help
- This option adds a CPUFreq driver for loongson1 processors which
- support software configurable cpu frequency.
-
If in doubt, say N.
endif
diff --git a/drivers/cpufreq/Makefile b/drivers/cpufreq/Makefile
index 32a7029e25ed..4a806cc5265b 100644
--- a/drivers/cpufreq/Makefile
+++ b/drivers/cpufreq/Makefile
@@ -111,7 +111,6 @@ obj-$(CONFIG_POWERNV_CPUFREQ) += powernv-cpufreq.o
obj-$(CONFIG_BMIPS_CPUFREQ) += bmips-cpufreq.o
obj-$(CONFIG_IA64_ACPI_CPUFREQ) += ia64-acpi-cpufreq.o
obj-$(CONFIG_LOONGSON2_CPUFREQ) += loongson2_cpufreq.o
-obj-$(CONFIG_LOONGSON1_CPUFREQ) += loongson1-cpufreq.o
obj-$(CONFIG_SH_CPU_FREQ) += sh-cpufreq.o
obj-$(CONFIG_SPARC_US2E_CPUFREQ) += sparc-us2e-cpufreq.o
obj-$(CONFIG_SPARC_US3_CPUFREQ) += sparc-us3-cpufreq.o
diff --git a/drivers/cpufreq/loongson1-cpufreq.c b/drivers/cpufreq/loongson1-cpufreq.c
deleted file mode 100644
index fb72d709db56..000000000000
--- a/drivers/cpufreq/loongson1-cpufreq.c
+++ /dev/null
@@ -1,222 +0,0 @@
-/*
- * CPU Frequency Scaling for Loongson 1 SoC
- *
- * Copyright (C) 2014-2016 Zhang, Keguang <keguang.zhang@gmail.com>
- *
- * This file is licensed under the terms of the GNU General Public
- * License version 2. This program is licensed "as is" without any
- * warranty of any kind, whether express or implied.
- */
-
-#include <linux/clk.h>
-#include <linux/clk-provider.h>
-#include <linux/cpu.h>
-#include <linux/cpufreq.h>
-#include <linux/delay.h>
-#include <linux/io.h>
-#include <linux/module.h>
-#include <linux/platform_device.h>
-#include <linux/slab.h>
-
-#include <cpufreq.h>
-#include <loongson1.h>
-
-struct ls1x_cpufreq {
- struct device *dev;
- struct clk *clk; /* CPU clk */
- struct clk *mux_clk; /* MUX of CPU clk */
- struct clk *pll_clk; /* PLL clk */
- struct clk *osc_clk; /* OSC clk */
- unsigned int max_freq;
- unsigned int min_freq;
-};
-
-static struct ls1x_cpufreq *cpufreq;
-
-static int ls1x_cpufreq_notifier(struct notifier_block *nb,
- unsigned long val, void *data)
-{
- if (val == CPUFREQ_POSTCHANGE)
- current_cpu_data.udelay_val = loops_per_jiffy;
-
- return NOTIFY_OK;
-}
-
-static struct notifier_block ls1x_cpufreq_notifier_block = {
- .notifier_call = ls1x_cpufreq_notifier
-};
-
-static int ls1x_cpufreq_target(struct cpufreq_policy *policy,
- unsigned int index)
-{
- struct device *cpu_dev = get_cpu_device(policy->cpu);
- unsigned int old_freq, new_freq;
-
- old_freq = policy->cur;
- new_freq = policy->freq_table[index].frequency;
-
- /*
- * The procedure of reconfiguring CPU clk is as below.
- *
- * - Reparent CPU clk to OSC clk
- * - Reset CPU clock (very important)
- * - Reconfigure CPU DIV
- * - Reparent CPU clk back to CPU DIV clk
- */
-
- clk_set_parent(policy->clk, cpufreq->osc_clk);
- __raw_writel(__raw_readl(LS1X_CLK_PLL_DIV) | RST_CPU_EN | RST_CPU,
- LS1X_CLK_PLL_DIV);
- __raw_writel(__raw_readl(LS1X_CLK_PLL_DIV) & ~(RST_CPU_EN | RST_CPU),
- LS1X_CLK_PLL_DIV);
- clk_set_rate(cpufreq->mux_clk, new_freq * 1000);
- clk_set_parent(policy->clk, cpufreq->mux_clk);
- dev_dbg(cpu_dev, "%u KHz --> %u KHz\n", old_freq, new_freq);
-
- return 0;
-}
-
-static int ls1x_cpufreq_init(struct cpufreq_policy *policy)
-{
- struct device *cpu_dev = get_cpu_device(policy->cpu);
- struct cpufreq_frequency_table *freq_tbl;
- unsigned int pll_freq, freq;
- int steps, i;
-
- pll_freq = clk_get_rate(cpufreq->pll_clk) / 1000;
-
- steps = 1 << DIV_CPU_WIDTH;
- freq_tbl = kcalloc(steps, sizeof(*freq_tbl), GFP_KERNEL);
- if (!freq_tbl)
- return -ENOMEM;
-
- for (i = 0; i < (steps - 1); i++) {
- freq = pll_freq / (i + 1);
- if ((freq < cpufreq->min_freq) || (freq > cpufreq->max_freq))
- freq_tbl[i].frequency = CPUFREQ_ENTRY_INVALID;
- else
- freq_tbl[i].frequency = freq;
- dev_dbg(cpu_dev,
- "cpufreq table: index %d: frequency %d\n", i,
- freq_tbl[i].frequency);
- }
- freq_tbl[i].frequency = CPUFREQ_TABLE_END;
-
- policy->clk = cpufreq->clk;
- cpufreq_generic_init(policy, freq_tbl, 0);
-
- return 0;
-}
-
-static int ls1x_cpufreq_exit(struct cpufreq_policy *policy)
-{
- kfree(policy->freq_table);
- return 0;
-}
-
-static struct cpufreq_driver ls1x_cpufreq_driver = {
- .name = "cpufreq-ls1x",
- .flags = CPUFREQ_NEED_INITIAL_FREQ_CHECK,
- .verify = cpufreq_generic_frequency_table_verify,
- .target_index = ls1x_cpufreq_target,
- .get = cpufreq_generic_get,
- .init = ls1x_cpufreq_init,
- .exit = ls1x_cpufreq_exit,
- .attr = cpufreq_generic_attr,
-};
-
-static int ls1x_cpufreq_remove(struct platform_device *pdev)
-{
- cpufreq_unregister_notifier(&ls1x_cpufreq_notifier_block,
- CPUFREQ_TRANSITION_NOTIFIER);
- cpufreq_unregister_driver(&ls1x_cpufreq_driver);
-
- return 0;
-}
-
-static int ls1x_cpufreq_probe(struct platform_device *pdev)
-{
- struct plat_ls1x_cpufreq *pdata = dev_get_platdata(&pdev->dev);
- struct clk *clk;
- int ret;
-
- if (!pdata || !pdata->clk_name || !pdata->osc_clk_name) {
- dev_err(&pdev->dev, "platform data missing\n");
- return -EINVAL;
- }
-
- cpufreq =
- devm_kzalloc(&pdev->dev, sizeof(struct ls1x_cpufreq), GFP_KERNEL);
- if (!cpufreq)
- return -ENOMEM;
-
- cpufreq->dev = &pdev->dev;
-
- clk = devm_clk_get(&pdev->dev, pdata->clk_name);
- if (IS_ERR(clk)) {
- dev_err(&pdev->dev, "unable to get %s clock\n",
- pdata->clk_name);
- return PTR_ERR(clk);
- }
- cpufreq->clk = clk;
-
- clk = clk_get_parent(clk);
- if (IS_ERR(clk)) {
- dev_err(&pdev->dev, "unable to get parent of %s clock\n",
- __clk_get_name(cpufreq->clk));
- return PTR_ERR(clk);
- }
- cpufreq->mux_clk = clk;
-
- clk = clk_get_parent(clk);
- if (IS_ERR(clk)) {
- dev_err(&pdev->dev, "unable to get parent of %s clock\n",
- __clk_get_name(cpufreq->mux_clk));
- return PTR_ERR(clk);
- }
- cpufreq->pll_clk = clk;
-
- clk = devm_clk_get(&pdev->dev, pdata->osc_clk_name);
- if (IS_ERR(clk)) {
- dev_err(&pdev->dev, "unable to get %s clock\n",
- pdata->osc_clk_name);
- return PTR_ERR(clk);
- }
- cpufreq->osc_clk = clk;
-
- cpufreq->max_freq = pdata->max_freq;
- cpufreq->min_freq = pdata->min_freq;
-
- ret = cpufreq_register_driver(&ls1x_cpufreq_driver);
- if (ret) {
- dev_err(&pdev->dev,
- "failed to register CPUFreq driver: %d\n", ret);
- return ret;
- }
-
- ret = cpufreq_register_notifier(&ls1x_cpufreq_notifier_block,
- CPUFREQ_TRANSITION_NOTIFIER);
-
- if (ret) {
- dev_err(&pdev->dev,
- "failed to register CPUFreq notifier: %d\n",ret);
- cpufreq_unregister_driver(&ls1x_cpufreq_driver);
- }
-
- return ret;
-}
-
-static struct platform_driver ls1x_cpufreq_platdrv = {
- .probe = ls1x_cpufreq_probe,
- .remove = ls1x_cpufreq_remove,
- .driver = {
- .name = "ls1x-cpufreq",
- },
-};
-
-module_platform_driver(ls1x_cpufreq_platdrv);
-
-MODULE_ALIAS("platform:ls1x-cpufreq");
-MODULE_AUTHOR("Kelvin Cheung <keguang.zhang@gmail.com>");
-MODULE_DESCRIPTION("Loongson1 CPUFreq driver");
-MODULE_LICENSE("GPL");
base-commit: 837dc62247d5e50a0724af2001f070962300c87e
--
2.34.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] cpufreq: loongson1: Delete obsolete driver
2023-01-12 13:53 [PATCH] cpufreq: loongson1: Delete obsolete driver Keguang Zhang
@ 2023-01-20 16:49 ` Rafael J. Wysocki
0 siblings, 0 replies; 2+ messages in thread
From: Rafael J. Wysocki @ 2023-01-20 16:49 UTC (permalink / raw)
To: Keguang Zhang
Cc: linux-pm, linux-mips, linux-kernel, Rafael J . Wysocki, Viresh Kumar
On Thu, Jan 12, 2023 at 2:54 PM Keguang Zhang <keguang.zhang@gmail.com> wrote:
>
> The generic DT based cpufreq driver works for Loongson-1,
> so delete the old custom driver.
>
> Signed-off-by: Keguang Zhang <keguang.zhang@gmail.com>
> ---
> drivers/cpufreq/Kconfig | 9 --
> drivers/cpufreq/Makefile | 1 -
> drivers/cpufreq/loongson1-cpufreq.c | 222 ----------------------------
> 3 files changed, 232 deletions(-)
> delete mode 100644 drivers/cpufreq/loongson1-cpufreq.c
>
> diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig
> index 2a84fc63371e..448b8ffb4ebd 100644
> --- a/drivers/cpufreq/Kconfig
> +++ b/drivers/cpufreq/Kconfig
> @@ -270,15 +270,6 @@ config LOONGSON2_CPUFREQ
>
> Loongson2F and its successors support this feature.
>
> - If in doubt, say N.
> -
> -config LOONGSON1_CPUFREQ
> - tristate "Loongson1 CPUFreq Driver"
> - depends on LOONGSON1_LS1B
> - help
> - This option adds a CPUFreq driver for loongson1 processors which
> - support software configurable cpu frequency.
> -
> If in doubt, say N.
> endif
>
> diff --git a/drivers/cpufreq/Makefile b/drivers/cpufreq/Makefile
> index 32a7029e25ed..4a806cc5265b 100644
> --- a/drivers/cpufreq/Makefile
> +++ b/drivers/cpufreq/Makefile
> @@ -111,7 +111,6 @@ obj-$(CONFIG_POWERNV_CPUFREQ) += powernv-cpufreq.o
> obj-$(CONFIG_BMIPS_CPUFREQ) += bmips-cpufreq.o
> obj-$(CONFIG_IA64_ACPI_CPUFREQ) += ia64-acpi-cpufreq.o
> obj-$(CONFIG_LOONGSON2_CPUFREQ) += loongson2_cpufreq.o
> -obj-$(CONFIG_LOONGSON1_CPUFREQ) += loongson1-cpufreq.o
> obj-$(CONFIG_SH_CPU_FREQ) += sh-cpufreq.o
> obj-$(CONFIG_SPARC_US2E_CPUFREQ) += sparc-us2e-cpufreq.o
> obj-$(CONFIG_SPARC_US3_CPUFREQ) += sparc-us3-cpufreq.o
> diff --git a/drivers/cpufreq/loongson1-cpufreq.c b/drivers/cpufreq/loongson1-cpufreq.c
> deleted file mode 100644
> index fb72d709db56..000000000000
> --- a/drivers/cpufreq/loongson1-cpufreq.c
> +++ /dev/null
> @@ -1,222 +0,0 @@
> -/*
> - * CPU Frequency Scaling for Loongson 1 SoC
> - *
> - * Copyright (C) 2014-2016 Zhang, Keguang <keguang.zhang@gmail.com>
> - *
> - * This file is licensed under the terms of the GNU General Public
> - * License version 2. This program is licensed "as is" without any
> - * warranty of any kind, whether express or implied.
> - */
> -
> -#include <linux/clk.h>
> -#include <linux/clk-provider.h>
> -#include <linux/cpu.h>
> -#include <linux/cpufreq.h>
> -#include <linux/delay.h>
> -#include <linux/io.h>
> -#include <linux/module.h>
> -#include <linux/platform_device.h>
> -#include <linux/slab.h>
> -
> -#include <cpufreq.h>
> -#include <loongson1.h>
> -
> -struct ls1x_cpufreq {
> - struct device *dev;
> - struct clk *clk; /* CPU clk */
> - struct clk *mux_clk; /* MUX of CPU clk */
> - struct clk *pll_clk; /* PLL clk */
> - struct clk *osc_clk; /* OSC clk */
> - unsigned int max_freq;
> - unsigned int min_freq;
> -};
> -
> -static struct ls1x_cpufreq *cpufreq;
> -
> -static int ls1x_cpufreq_notifier(struct notifier_block *nb,
> - unsigned long val, void *data)
> -{
> - if (val == CPUFREQ_POSTCHANGE)
> - current_cpu_data.udelay_val = loops_per_jiffy;
> -
> - return NOTIFY_OK;
> -}
> -
> -static struct notifier_block ls1x_cpufreq_notifier_block = {
> - .notifier_call = ls1x_cpufreq_notifier
> -};
> -
> -static int ls1x_cpufreq_target(struct cpufreq_policy *policy,
> - unsigned int index)
> -{
> - struct device *cpu_dev = get_cpu_device(policy->cpu);
> - unsigned int old_freq, new_freq;
> -
> - old_freq = policy->cur;
> - new_freq = policy->freq_table[index].frequency;
> -
> - /*
> - * The procedure of reconfiguring CPU clk is as below.
> - *
> - * - Reparent CPU clk to OSC clk
> - * - Reset CPU clock (very important)
> - * - Reconfigure CPU DIV
> - * - Reparent CPU clk back to CPU DIV clk
> - */
> -
> - clk_set_parent(policy->clk, cpufreq->osc_clk);
> - __raw_writel(__raw_readl(LS1X_CLK_PLL_DIV) | RST_CPU_EN | RST_CPU,
> - LS1X_CLK_PLL_DIV);
> - __raw_writel(__raw_readl(LS1X_CLK_PLL_DIV) & ~(RST_CPU_EN | RST_CPU),
> - LS1X_CLK_PLL_DIV);
> - clk_set_rate(cpufreq->mux_clk, new_freq * 1000);
> - clk_set_parent(policy->clk, cpufreq->mux_clk);
> - dev_dbg(cpu_dev, "%u KHz --> %u KHz\n", old_freq, new_freq);
> -
> - return 0;
> -}
> -
> -static int ls1x_cpufreq_init(struct cpufreq_policy *policy)
> -{
> - struct device *cpu_dev = get_cpu_device(policy->cpu);
> - struct cpufreq_frequency_table *freq_tbl;
> - unsigned int pll_freq, freq;
> - int steps, i;
> -
> - pll_freq = clk_get_rate(cpufreq->pll_clk) / 1000;
> -
> - steps = 1 << DIV_CPU_WIDTH;
> - freq_tbl = kcalloc(steps, sizeof(*freq_tbl), GFP_KERNEL);
> - if (!freq_tbl)
> - return -ENOMEM;
> -
> - for (i = 0; i < (steps - 1); i++) {
> - freq = pll_freq / (i + 1);
> - if ((freq < cpufreq->min_freq) || (freq > cpufreq->max_freq))
> - freq_tbl[i].frequency = CPUFREQ_ENTRY_INVALID;
> - else
> - freq_tbl[i].frequency = freq;
> - dev_dbg(cpu_dev,
> - "cpufreq table: index %d: frequency %d\n", i,
> - freq_tbl[i].frequency);
> - }
> - freq_tbl[i].frequency = CPUFREQ_TABLE_END;
> -
> - policy->clk = cpufreq->clk;
> - cpufreq_generic_init(policy, freq_tbl, 0);
> -
> - return 0;
> -}
> -
> -static int ls1x_cpufreq_exit(struct cpufreq_policy *policy)
> -{
> - kfree(policy->freq_table);
> - return 0;
> -}
> -
> -static struct cpufreq_driver ls1x_cpufreq_driver = {
> - .name = "cpufreq-ls1x",
> - .flags = CPUFREQ_NEED_INITIAL_FREQ_CHECK,
> - .verify = cpufreq_generic_frequency_table_verify,
> - .target_index = ls1x_cpufreq_target,
> - .get = cpufreq_generic_get,
> - .init = ls1x_cpufreq_init,
> - .exit = ls1x_cpufreq_exit,
> - .attr = cpufreq_generic_attr,
> -};
> -
> -static int ls1x_cpufreq_remove(struct platform_device *pdev)
> -{
> - cpufreq_unregister_notifier(&ls1x_cpufreq_notifier_block,
> - CPUFREQ_TRANSITION_NOTIFIER);
> - cpufreq_unregister_driver(&ls1x_cpufreq_driver);
> -
> - return 0;
> -}
> -
> -static int ls1x_cpufreq_probe(struct platform_device *pdev)
> -{
> - struct plat_ls1x_cpufreq *pdata = dev_get_platdata(&pdev->dev);
> - struct clk *clk;
> - int ret;
> -
> - if (!pdata || !pdata->clk_name || !pdata->osc_clk_name) {
> - dev_err(&pdev->dev, "platform data missing\n");
> - return -EINVAL;
> - }
> -
> - cpufreq =
> - devm_kzalloc(&pdev->dev, sizeof(struct ls1x_cpufreq), GFP_KERNEL);
> - if (!cpufreq)
> - return -ENOMEM;
> -
> - cpufreq->dev = &pdev->dev;
> -
> - clk = devm_clk_get(&pdev->dev, pdata->clk_name);
> - if (IS_ERR(clk)) {
> - dev_err(&pdev->dev, "unable to get %s clock\n",
> - pdata->clk_name);
> - return PTR_ERR(clk);
> - }
> - cpufreq->clk = clk;
> -
> - clk = clk_get_parent(clk);
> - if (IS_ERR(clk)) {
> - dev_err(&pdev->dev, "unable to get parent of %s clock\n",
> - __clk_get_name(cpufreq->clk));
> - return PTR_ERR(clk);
> - }
> - cpufreq->mux_clk = clk;
> -
> - clk = clk_get_parent(clk);
> - if (IS_ERR(clk)) {
> - dev_err(&pdev->dev, "unable to get parent of %s clock\n",
> - __clk_get_name(cpufreq->mux_clk));
> - return PTR_ERR(clk);
> - }
> - cpufreq->pll_clk = clk;
> -
> - clk = devm_clk_get(&pdev->dev, pdata->osc_clk_name);
> - if (IS_ERR(clk)) {
> - dev_err(&pdev->dev, "unable to get %s clock\n",
> - pdata->osc_clk_name);
> - return PTR_ERR(clk);
> - }
> - cpufreq->osc_clk = clk;
> -
> - cpufreq->max_freq = pdata->max_freq;
> - cpufreq->min_freq = pdata->min_freq;
> -
> - ret = cpufreq_register_driver(&ls1x_cpufreq_driver);
> - if (ret) {
> - dev_err(&pdev->dev,
> - "failed to register CPUFreq driver: %d\n", ret);
> - return ret;
> - }
> -
> - ret = cpufreq_register_notifier(&ls1x_cpufreq_notifier_block,
> - CPUFREQ_TRANSITION_NOTIFIER);
> -
> - if (ret) {
> - dev_err(&pdev->dev,
> - "failed to register CPUFreq notifier: %d\n",ret);
> - cpufreq_unregister_driver(&ls1x_cpufreq_driver);
> - }
> -
> - return ret;
> -}
> -
> -static struct platform_driver ls1x_cpufreq_platdrv = {
> - .probe = ls1x_cpufreq_probe,
> - .remove = ls1x_cpufreq_remove,
> - .driver = {
> - .name = "ls1x-cpufreq",
> - },
> -};
> -
> -module_platform_driver(ls1x_cpufreq_platdrv);
> -
> -MODULE_ALIAS("platform:ls1x-cpufreq");
> -MODULE_AUTHOR("Kelvin Cheung <keguang.zhang@gmail.com>");
> -MODULE_DESCRIPTION("Loongson1 CPUFreq driver");
> -MODULE_LICENSE("GPL");
>
> base-commit: 837dc62247d5e50a0724af2001f070962300c87e
> --
Applied as 6.3 material, thanks!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-01-20 16:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-12 13:53 [PATCH] cpufreq: loongson1: Delete obsolete driver Keguang Zhang
2023-01-20 16:49 ` Rafael J. Wysocki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).