Commit 5553f9e2 authored by Shawn Guo's avatar Shawn Guo Committed by Rafael J. Wysocki
Browse files

cpufreq: instantiate cpufreq-cpu0 as a platform_driver 



As multiplatform build is being adopted by more and more ARM platforms,
initcall function should be used very carefully.  For example, when
GENERIC_CPUFREQ_CPU0 is built in the kernel, cpu0_cpufreq_driver_init()
will be called on all the platforms to initialize cpufreq-cpu0 driver.

To eliminate this undesired the effect, the patch changes cpufreq-cpu0
driver to have it instantiated as a platform_driver.  Then it will only
run on platforms that create the platform_device "cpufreq-cpu0".

Along with the change, it also changes cpu_dev to be &pdev->dev,
so that managed functions can start working, and module build gets
supported too.

The highbank-cpufreq driver is also updated accordingly to adapt the
changes on cpufreq-cpu0.

Signed-off-by: default avatarShawn Guo <shawn.guo@linaro.org>
Reviewed-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Acked-by: default avatarMark Langsdorf <mark.langsdorf@calxeda.com>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 58ddcead

drivers/cpufreq/Kconfig

+1 −1
Original line number Original line Diff line number Diff line
@@ -185,7 +185,7 @@ config CPU_FREQ_GOV_CONSERVATIVE 
	  If in doubt, say N.

	  If in doubt, say N.





config GENERIC_CPUFREQ_CPU0

config GENERIC_CPUFREQ_CPU0

	bool "Generic CPU0 cpufreq driver"

	tristate "Generic CPU0 cpufreq driver"

	depends on HAVE_CLK && REGULATOR && PM_OPP && OF

	depends on HAVE_CLK && REGULATOR && PM_OPP && OF

	select CPU_FREQ_TABLE

	select CPU_FREQ_TABLE

	help

	help

drivers/cpufreq/cpufreq-cpu0.c

+23 −12
Original line number Original line Diff line number Diff line
@@ -12,12 +12,12 @@ 
#define pr_fmt(fmt)	KBUILD_MODNAME ": " fmt

#define pr_fmt(fmt)	KBUILD_MODNAME ": " fmt





#include <linux/clk.h>

#include <linux/clk.h>

#include <linux/cpu.h>

#include <linux/cpufreq.h>

#include <linux/cpufreq.h>

#include <linux/err.h>

#include <linux/err.h>

#include <linux/module.h>

#include <linux/module.h>

#include <linux/of.h>

#include <linux/of.h>

#include <linux/opp.h>

#include <linux/opp.h>

#include <linux/platform_device.h>

#include <linux/regulator/consumer.h>

#include <linux/regulator/consumer.h>

#include <linux/slab.h>

#include <linux/slab.h>
@@ -177,7 +177,7 @@ static struct cpufreq_driver cpu0_cpufreq_driver = { 
	.attr = cpu0_cpufreq_attr,

	.attr = cpu0_cpufreq_attr,

};

};





static int cpu0_cpufreq_driver_init(void)

static int cpu0_cpufreq_probe(struct platform_device *pdev)

{

{

	struct device_node *np;

	struct device_node *np;

	int ret;

	int ret;
@@ -192,23 +192,17 @@ static int cpu0_cpufreq_driver_init(void) 
		return -ENOENT;

		return -ENOENT;

	}

	}





	cpu_dev = get_cpu_device(0);

	cpu_dev = &pdev->dev;

	if (!cpu_dev) {

		pr_err("failed to get cpu0 device\n");

		ret = -ENODEV;

		goto out_put_node;

	}



	cpu_dev->of_node = np;

	cpu_dev->of_node = np;





	cpu_clk = clk_get(cpu_dev, NULL);

	cpu_clk = devm_clk_get(cpu_dev, NULL);

	if (IS_ERR(cpu_clk)) {

	if (IS_ERR(cpu_clk)) {

		ret = PTR_ERR(cpu_clk);

		ret = PTR_ERR(cpu_clk);

		pr_err("failed to get cpu0 clock: %d\n", ret);

		pr_err("failed to get cpu0 clock: %d\n", ret);

		goto out_put_node;

		goto out_put_node;

	}

	}





	cpu_reg = regulator_get(cpu_dev, "cpu0");

	cpu_reg = devm_regulator_get(cpu_dev, "cpu0");

	if (IS_ERR(cpu_reg)) {

	if (IS_ERR(cpu_reg)) {

		pr_warn("failed to get cpu0 regulator\n");

		pr_warn("failed to get cpu0 regulator\n");

		cpu_reg = NULL;

		cpu_reg = NULL;
@@ -271,7 +265,24 @@ out_put_node: 
	of_node_put(np);

	of_node_put(np);

	return ret;

	return ret;

}

}

late_initcall(cpu0_cpufreq_driver_init);



static int cpu0_cpufreq_remove(struct platform_device *pdev)

{

	cpufreq_unregister_driver(&cpu0_cpufreq_driver);

	opp_free_cpufreq_table(cpu_dev, &freq_table);



	return 0;

}



static struct platform_driver cpu0_cpufreq_platdrv = {

	.driver = {

		.name	= "cpufreq-cpu0",

		.owner	= THIS_MODULE,

	},

	.probe		= cpu0_cpufreq_probe,

	.remove		= cpu0_cpufreq_remove,

};

module_platform_driver(cpu0_cpufreq_platdrv);





MODULE_AUTHOR("Shawn Guo <shawn.guo@linaro.org>");

MODULE_AUTHOR("Shawn Guo <shawn.guo@linaro.org>");

MODULE_DESCRIPTION("Generic CPU0 cpufreq driver");

MODULE_DESCRIPTION("Generic CPU0 cpufreq driver");

drivers/cpufreq/highbank-cpufreq.c

+5 −0
Original line number Original line Diff line number Diff line
@@ -20,6 +20,7 @@ 
#include <linux/err.h>

#include <linux/err.h>

#include <linux/of.h>

#include <linux/of.h>

#include <linux/mailbox.h>

#include <linux/mailbox.h>

#include <linux/platform_device.h>





#define HB_CPUFREQ_CHANGE_NOTE	0x80000001

#define HB_CPUFREQ_CHANGE_NOTE	0x80000001

#define HB_CPUFREQ_IPC_LEN	7

#define HB_CPUFREQ_IPC_LEN	7
@@ -65,6 +66,7 @@ static struct notifier_block hb_cpufreq_clk_nb = { 




static int hb_cpufreq_driver_init(void)

static int hb_cpufreq_driver_init(void)

{

{

	struct platform_device_info devinfo = { .name = "cpufreq-cpu0", };

	struct device *cpu_dev;

	struct device *cpu_dev;

	struct clk *cpu_clk;

	struct clk *cpu_clk;

	struct device_node *np;

	struct device_node *np;
@@ -104,6 +106,9 @@ static int hb_cpufreq_driver_init(void) 
		goto out_put_node;

		goto out_put_node;

	}

	}





	/* Instantiate cpufreq-cpu0 */

	platform_device_register_full(&devinfo);



out_put_node:

out_put_node:

	of_node_put(np);

	of_node_put(np);

	return ret;

	return ret;

CRA Git | Maintained and supported by SUSTech CRA and CCSE