Message ID | 20230112135342.3927338-1-keguang.zhang@gmail.com |
---|---|
State | Accepted |
Commit | 9a55ab6f02c98bfca1c9c9d73507c1744406d2ba |
Headers | show |
Series | cpufreq: loongson1: Delete obsolete driver | expand |
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!
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");
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 base-commit: 837dc62247d5e50a0724af2001f070962300c87e