Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit aa7c352f authored by Chanwoo Choi's avatar Chanwoo Choi Committed by MyungJoo Ham
Browse files

PM / devfreq: Define the constant governor name 



Prior to that, the devfreq device uses the governor name when adding
the itself. In order to prevent the mistake used the wrong governor name,
this patch defines the governor name as a constant and then uses them
instead of using the string directly.

Signed-off-by: default avatarChanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: default avatarMyungJoo Ham <myungjoo.ham@samsung.com>
Cc: Kukjin Kim <kgene@kernel.org>
Cc: Krzysztof Kozlowski <krzk@kernel.org>
Cc: linux-samsung-soc@vger.kernel.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
parent ccc4c3bc

drivers/devfreq/exynos-bus.c

+3 −2
Original line number Diff line number Diff line
@@ -436,7 +436,8 @@ static int exynos_bus_probe(struct platform_device *pdev) 
	ondemand_data->downdifferential = 5;



	/* Add devfreq device to monitor and handle the exynos bus */

	bus->devfreq = devm_devfreq_add_device(dev, profile, "simple_ondemand",

	bus->devfreq = devm_devfreq_add_device(dev, profile,

						DEVFREQ_GOV_SIMPLE_ONDEMAND,

						ondemand_data);

	if (IS_ERR(bus->devfreq)) {

		dev_err(dev, "failed to add devfreq device\n");
@@ -488,7 +489,7 @@ static int exynos_bus_probe(struct platform_device *pdev) 
	passive_data->parent = parent_devfreq;



	/* Add devfreq device for exynos bus with passive governor */

	bus->devfreq = devm_devfreq_add_device(dev, profile, "passive",

	bus->devfreq = devm_devfreq_add_device(dev, profile, DEVFREQ_GOV_PASSIVE,

						passive_data);

	if (IS_ERR(bus->devfreq)) {

		dev_err(dev,

drivers/devfreq/governor_passive.c

+1 −1
Original line number Diff line number Diff line
@@ -183,7 +183,7 @@ static int devfreq_passive_event_handler(struct devfreq *devfreq, 
}



static struct devfreq_governor devfreq_passive = {

	.name = "passive",

	.name = DEVFREQ_GOV_PASSIVE,

	.immutable = 1,

	.get_target_freq = devfreq_passive_get_target_freq,

	.event_handler = devfreq_passive_event_handler,

drivers/devfreq/governor_performance.c

+1 −1
Original line number Diff line number Diff line
@@ -42,7 +42,7 @@ static int devfreq_performance_handler(struct devfreq *devfreq, 
}



static struct devfreq_governor devfreq_performance = {

	.name = "performance",

	.name = DEVFREQ_GOV_PERFORMANCE,

	.get_target_freq = devfreq_performance_func,

	.event_handler = devfreq_performance_handler,

};

drivers/devfreq/governor_powersave.c

+1 −1
Original line number Diff line number Diff line
@@ -39,7 +39,7 @@ static int devfreq_powersave_handler(struct devfreq *devfreq, 
}



static struct devfreq_governor devfreq_powersave = {

	.name = "powersave",

	.name = DEVFREQ_GOV_POWERSAVE,

	.get_target_freq = devfreq_powersave_func,

	.event_handler = devfreq_powersave_handler,

};

drivers/devfreq/governor_simpleondemand.c

+1 −1
Original line number Diff line number Diff line
@@ -125,7 +125,7 @@ static int devfreq_simple_ondemand_handler(struct devfreq *devfreq, 
}



static struct devfreq_governor devfreq_simple_ondemand = {

	.name = "simple_ondemand",

	.name = DEVFREQ_GOV_SIMPLE_ONDEMAND,

	.get_target_freq = devfreq_simple_ondemand_func,

	.event_handler = devfreq_simple_ondemand_handler,

};