| Message ID | ZH3umLU883eGbYBx@makrotopia.org |
|---|---|
| State | Accepted |
| Commit | f85534113f5ae90a52521cdb9e9977a43ee42626 |
| Headers | show |
| Series | cpufreq: mediatek: correct voltages for MT7622 and MT7623 | expand |
Il 05/06/23 16:18, Daniel Golle ha scritto: > The MT6380 regulator typically used together with MT7622 does not > support the current maximum processor and SRAM voltage in the cpufreq > driver (1360000uV). > For MT7622 limit processor and SRAM supply voltages to 1350000uV to > avoid having the tracking algorithm request unsupported voltages from > the regulator. > > On MT7623 there is no separate SRAM supply and the maximum voltage used > is 1300000uV. Create dedicated platform data for MT7623 to cover that > case as well. > > Fixes: 0883426fd07e3 ("cpufreq: mediatek: Raise proc and sram max voltage for MT7622/7623") > Suggested-by: Jia-wei Chang <Jia-wei.Chang@mediatek.com> > Signed-off-by: Daniel Golle <daniel@makrotopia.org> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
On 06-06-23, 09:41, AngeloGioacchino Del Regno wrote: > Il 05/06/23 16:18, Daniel Golle ha scritto: > > The MT6380 regulator typically used together with MT7622 does not > > support the current maximum processor and SRAM voltage in the cpufreq > > driver (1360000uV). > > For MT7622 limit processor and SRAM supply voltages to 1350000uV to > > avoid having the tracking algorithm request unsupported voltages from > > the regulator. > > > > On MT7623 there is no separate SRAM supply and the maximum voltage used > > is 1300000uV. Create dedicated platform data for MT7623 to cover that > > case as well. > > > > Fixes: 0883426fd07e3 ("cpufreq: mediatek: Raise proc and sram max voltage for MT7622/7623") > > Suggested-by: Jia-wei Chang <Jia-wei.Chang@mediatek.com> > > Signed-off-by: Daniel Golle <daniel@makrotopia.org> > > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> Applied. Thanks.
diff --git a/drivers/cpufreq/mediatek-cpufreq.c b/drivers/cpufreq/mediatek-cpufreq.c index 9a39a7ccfae96..fef68cb2b38f7 100644 --- a/drivers/cpufreq/mediatek-cpufreq.c +++ b/drivers/cpufreq/mediatek-cpufreq.c @@ -696,9 +696,16 @@ static const struct mtk_cpufreq_platform_data mt2701_platform_data = { static const struct mtk_cpufreq_platform_data mt7622_platform_data = { .min_volt_shift = 100000, .max_volt_shift = 200000, - .proc_max_volt = 1360000, + .proc_max_volt = 1350000, .sram_min_volt = 0, - .sram_max_volt = 1360000, + .sram_max_volt = 1350000, + .ccifreq_supported = false, +}; + +static const struct mtk_cpufreq_platform_data mt7623_platform_data = { + .min_volt_shift = 100000, + .max_volt_shift = 200000, + .proc_max_volt = 1300000, .ccifreq_supported = false, }; @@ -734,7 +741,7 @@ static const struct of_device_id mtk_cpufreq_machines[] __initconst = { { .compatible = "mediatek,mt2701", .data = &mt2701_platform_data }, { .compatible = "mediatek,mt2712", .data = &mt2701_platform_data }, { .compatible = "mediatek,mt7622", .data = &mt7622_platform_data }, - { .compatible = "mediatek,mt7623", .data = &mt7622_platform_data }, + { .compatible = "mediatek,mt7623", .data = &mt7623_platform_data }, { .compatible = "mediatek,mt8167", .data = &mt8516_platform_data }, { .compatible = "mediatek,mt817x", .data = &mt2701_platform_data }, { .compatible = "mediatek,mt8173", .data = &mt2701_platform_data },
The MT6380 regulator typically used together with MT7622 does not support the current maximum processor and SRAM voltage in the cpufreq driver (1360000uV). For MT7622 limit processor and SRAM supply voltages to 1350000uV to avoid having the tracking algorithm request unsupported voltages from the regulator. On MT7623 there is no separate SRAM supply and the maximum voltage used is 1300000uV. Create dedicated platform data for MT7623 to cover that case as well. Fixes: 0883426fd07e3 ("cpufreq: mediatek: Raise proc and sram max voltage for MT7622/7623") Suggested-by: Jia-wei Chang <Jia-wei.Chang@mediatek.com> Signed-off-by: Daniel Golle <daniel@makrotopia.org> --- drivers/cpufreq/mediatek-cpufreq.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-)