cpufreq: mediatek: Make sram regulator optional
authorJia-Wei Chang <jia-wei.chang@mediatek.com>
Fri, 22 Apr 2022 07:52:33 +0000 (15:52 +0800)
committerViresh Kumar <viresh.kumar@linaro.org>
Tue, 26 Apr 2022 06:39:10 +0000 (12:09 +0530)
For some MediaTek SoCs, like MT8186, it's possible that the sram regulator
is shared between CPU and CCI.
We hope regulator framework can return error for error handling rather
than a dummy handler from regulator_get api.
Therefore, we choose to use regulator_get_optional.

Signed-off-by: Jia-Wei Chang <jia-wei.chang@mediatek.com>
Signed-off-by: Rex-BC Chen <rex-bc.chen@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
drivers/cpufreq/mediatek-cpufreq.c

index 1688cf6..bcabb37 100644 (file)
@@ -352,7 +352,7 @@ static int mtk_cpu_dvfs_info_init(struct mtk_cpu_dvfs_info *info, int cpu)
        }
 
        /* Both presence and absence of sram regulator are valid cases. */
-       info->sram_reg = regulator_get_exclusive(cpu_dev, "sram");
+       info->sram_reg = regulator_get_optional(cpu_dev, "sram");
        if (IS_ERR(info->sram_reg))
                info->sram_reg = NULL;
        else {