Re: [PATCH v3 2/2] clk: qcom: Add LPASS VA CSR heartbeat pulse clock

[Konrad Dybcio]

On 7/23/26 1:02 PM, Sarath Ganapathiraju wrote:
> The HeartBeat Pulse (also known as RateGen Pulse) synchronizes the
> start of the DMAs and Codec Interfaces for the audio usecases and
> can serve as a periodic wakeup source for the DSP.
> 
> Add the LPASS VA CSR driver that models the rate generator as a clock
> provider so it is enabled and disabled automatically alongside the
> other clocks during runtime PM resume and suspend.
> 
> Signed-off-by: Sarath Ganapathiraju <sarath.ganapathiraju@xxxxxxxxxxxxxxxx>
> ---
>  drivers/clk/qcom/Kconfig        |  12 +++
>  drivers/clk/qcom/Makefile       |   1 +
>  drivers/clk/qcom/lpass-va-csr.c | 143 ++++++++++++++++++++++++++++++++
>  sound/soc/codecs/Kconfig        |   1 +

This patch spans 2 subsystems and 2 separate maintainers.. we'll need
Mark's ack for the sound/ change

>  4 files changed, 157 insertions(+)
>  create mode 100644 drivers/clk/qcom/lpass-va-csr.c
> 
> diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig
> index 63f271aff177..6ebdccfe37e0 100644
> --- a/drivers/clk/qcom/Kconfig
> +++ b/drivers/clk/qcom/Kconfig
> @@ -1998,6 +1998,18 @@ config CLK_GFM_LPASS_SM8250
>  	  Support for the Glitch Free Mux (GFM) Low power audio
>            subsystem (LPASS) clocks found on SM8250 SoCs.
>  
> +config QCOM_CLK_LPASS_VA_CSR

VA_CSR is the Control-and-Status-Register space of the VA Macro,
the name of the config should deifnitely focus on the heartbeat
pulse clock generator within it instead

Besides, that address space is huge, yet you're claiming it for
a single simple clock. Exposing anything else from within there
in the future will require bindings updates, which are usually a
tough sell. Is there anything else in there that we need to cover
from HLOS?

Konrad