Re: [PATCH 2/2] soc: mediatek: Add mtk-mmdvfs driver

From: Randy Dunlap
Date: Wed Apr 15 2020 - 11:25:50 EST


Hi--


On 4/15/20 4:18 AM, Anthony Huang wrote:
> Mediatek MMDVFS driver is used to set clk for Mediatek multimedia
> hardwares. The MMDVFS registers a regulator callback and multimedia

hardware.

> hardwares set voltage by regulator API and then this callback will be

hardware

> triggered. The MMDVFS will get current opp level from opp table according
> to the voltage, and then the MMDVFS sets all the clock MUXs to the clock
> sources according to the opp level.
>
> On some platforms, both clock MUX and frequency hopping need to be used
> together. The MMDVFS supports these two clock setting methods and the
> execution sequence of them can be configured in DTS.
>
> Signed-off-by: Anthony Huang <anthony.huang@xxxxxxxxxxxx>
> ---
> drivers/soc/mediatek/Kconfig | 9 ++
> drivers/soc/mediatek/Makefile | 1 +
> drivers/soc/mediatek/mtk-mmdvfs.c | 312 +++++++++++++++++++++++++++++++++++++
> 3 files changed, 322 insertions(+)
> create mode 100644 drivers/soc/mediatek/mtk-mmdvfs.c
>
> diff --git a/drivers/soc/mediatek/Kconfig b/drivers/soc/mediatek/Kconfig
> index 2114b56..e92762b 100644
> --- a/drivers/soc/mediatek/Kconfig
> +++ b/drivers/soc/mediatek/Kconfig
> @@ -44,4 +44,13 @@ config MTK_SCPSYS
> Say yes here to add support for the MediaTek SCPSYS power domain
> driver.
>
> +config MTK_MMDVFS
> + tristate "MediaTek MMDVFS Support"
> + depends on ARCH_MEDIATEK || COMPILE_TEST
> + help
> + Say yes here to add support for the MediaTek Multimedia DVFS (MMDVFS)
> + driver. The MMDVFS is used to set clk for Mediatek multimedia hardwares

hardware,

> + , such as display, camera, mdp and video codec. Say no if your device

^drop that comma.

> + does not need to do DVFS for Multimedia hardwares.

hardware.

> +
> endmenu


thanks.
--
~Randy