]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commitdiff
cpufreq: mediatek: Unregister platform device on exit
authorRex-BC Chen <rex-bc.chen@mediatek.com>
Thu, 5 May 2022 11:52:18 +0000 (19:52 +0800)
committerStefan Bader <stefan.bader@canonical.com>
Wed, 10 Aug 2022 07:25:00 +0000 (09:25 +0200)
BugLink: https://bugs.launchpad.net/bugs/1981864
[ Upstream commit f126fbadce92b92c3a7be41e4abc1fbae93ae2ef ]

We register the platform device when driver inits. However, we do not
unregister it when driver exits.

To resolve this, we declare the platform data to be a global static
variable and rename it to be "cpufreq_pdev". With this global variable,
we can do platform_device_unregister() when driver exits.

Fixes: 501c574f4e3a ("cpufreq: mediatek: Add support of cpufreq to MT2701/MT7623 SoC")
Signed-off-by: Rex-BC Chen <rex-bc.chen@mediatek.com>
[ Viresh: Commit log and Subject ]
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
drivers/cpufreq/mediatek-cpufreq.c

index 9d7d9c8dc184bc9da94a5b75614e2d2b6b9d5918..bfe240c726e346c4ce542560da5e4fa02d175d1f 100644 (file)
@@ -44,6 +44,8 @@ struct mtk_cpu_dvfs_info {
        bool need_voltage_tracking;
 };
 
+static struct platform_device *cpufreq_pdev;
+
 static LIST_HEAD(dvfs_info_list);
 
 static struct mtk_cpu_dvfs_info *mtk_cpu_dvfs_info_lookup(int cpu)
@@ -547,7 +549,6 @@ static int __init mtk_cpufreq_driver_init(void)
 {
        struct device_node *np;
        const struct of_device_id *match;
-       struct platform_device *pdev;
        int err;
 
        np = of_find_node_by_path("/");
@@ -571,11 +572,11 @@ static int __init mtk_cpufreq_driver_init(void)
         * and the device registration codes are put here to handle defer
         * probing.
         */
-       pdev = platform_device_register_simple("mtk-cpufreq", -1, NULL, 0);
-       if (IS_ERR(pdev)) {
+       cpufreq_pdev = platform_device_register_simple("mtk-cpufreq", -1, NULL, 0);
+       if (IS_ERR(cpufreq_pdev)) {
                pr_err("failed to register mtk-cpufreq platform device\n");
                platform_driver_unregister(&mtk_cpufreq_platdrv);
-               return PTR_ERR(pdev);
+               return PTR_ERR(cpufreq_pdev);
        }
 
        return 0;
@@ -584,6 +585,7 @@ module_init(mtk_cpufreq_driver_init)
 
 static void __exit mtk_cpufreq_driver_exit(void)
 {
+       platform_device_unregister(cpufreq_pdev);
        platform_driver_unregister(&mtk_cpufreq_platdrv);
 }
 module_exit(mtk_cpufreq_driver_exit)