Re: [PATCH v2 18/29] clk: mediatek: Add MT8196 adsp clock support
From: Chen-Yu Tsai <wenst@chromium.org>
Date: 2025-07-15 04:37:06
Also in:
linux-arm-kernel, linux-clk, linux-devicetree, linux-mediatek, lkml
Hi, On Tue, Jun 24, 2025 at 10:33 PM Laura Nao [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Add support for the MT8196 adsp clock controller, which provides clock gate control for Audio DSP. Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> Signed-off-by: Laura Nao <redacted> --- drivers/clk/mediatek/Kconfig | 7 + drivers/clk/mediatek/Makefile | 1 + drivers/clk/mediatek/clk-mt8196-adsp.c | 193 +++++++++++++++++++++++++ 3 files changed, 201 insertions(+) create mode 100644 drivers/clk/mediatek/clk-mt8196-adsp.cdiff --git a/drivers/clk/mediatek/Kconfig b/drivers/clk/mediatek/Kconfig index cd12e7ff3e12..d4c97f64b42a 100644 --- a/drivers/clk/mediatek/Kconfig +++ b/drivers/clk/mediatek/Kconfig@@ -1068,6 +1068,13 @@ config COMMON_CLK_MT8196 help This driver supports MediaTek MT8196 basic clocks. +config COMMON_CLK_MT8196_ADSP + tristate "Clock driver for MediaTek MT8196 adsp" + depends on COMMON_CLK_MT8196 + default m + help + This driver supports MediaTek MT8196 adsp clocks +
This is part of the AFE block, and really should not be a separate device node nor driver. The AFE driver should internalize all these controls and ideally model some of them as ASoC widgets so that DAPM can handle sequencing altogether. So please drop this patch, and also drop the entry from the binding. I've mentioned this to MediaTek's audio and clock owners, though I assume this request was somehow lost in the process. ChenYu