On 23/06/2022 14:38, AngeloGioacchino Del Regno wrote:
This is one of the steps to enable DVFS with the Panfrost driver: since Panfrost is already enabling the (required) MFG power domains and since the mtk-pm-domains driver is already responsible for actually enabling the SRAM PDN, it makes sense to make sure that the VSRAM supply is ON when trying to reset/enable the SRAM. For this reason, the MTK_SCPD_DOMAIN_SUPPLY flag was added to one more MFG domain, ensuring that the SRAM is actually powered and also not relying on the bootloader leaving this supply on; on the other hand, this is also making possible to avoid setting a sram-supply on the GPU node, making devfreq happy about having only one supply and finally allowing DVFS to happen. If no domain-supply is declared in devicetree, mtk-pm-domains driver probe will anyway keep going, so this is not breaking old devicetrees. No side effects either when this supply is declared for both a MFG domain and Panfrost together. This series has no dependencies.
Both patches applied, thanks!
Changes in v2: - Squashed all mtXXXX-pm-domains.h changes in one patch AngeloGioacchino Del Regno (2): soc: mediatek: mtk-pm-domains: Allow probing vreg supply on two MFGs arm64: dts: mediatek: mt8183-kukui: Assign sram supply to mfg_async pd arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi | 4 ++++ arch/arm64/boot/dts/mediatek/mt8183.dtsi | 2 +- drivers/soc/mediatek/mt8183-pm-domains.h | 1 + drivers/soc/mediatek/mt8186-pm-domains.h | 2 +- drivers/soc/mediatek/mt8192-pm-domains.h | 2 ++ drivers/soc/mediatek/mt8195-pm-domains.h | 2 +- 6 files changed, 10 insertions(+), 3 deletions(-)
