Re: [PATCH v3 2/2] soc: qcom: Add Qualcomm Ramp Controller driver

[AngeloGioacchino Del Regno]

Il 16/11/22 14:46, Dmitry Baryshkov ha scritto:
> On 15/11/2022 18:45, AngeloGioacchino Del Regno wrote:
> > The Ramp Controller is used to program the sequence ID for pulse
> > swallowing, enable sequence and linking sequence IDs for the CPU
> > cores on some Qualcomm SoCs.
> >
> > Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>
> > ---
> >   drivers/soc/qcom/Kconfig           |   9 +
> >   drivers/soc/qcom/Makefile          |   1 +
> >   drivers/soc/qcom/ramp_controller.c | 331 +++++++++++++++++++++++++++++
> >   3 files changed, 341 insertions(+)
> >   create mode 100644 drivers/soc/qcom/ramp_controller.c
>
> I'd allow myself to re-ask the same questions here:
>
> Generic question. regarding this controller. If it is supposed to work
> close to DVCS, etc. Does ramp controller need any programming when
> changing speed and/or APC voltage?
> Is it necessary to turn ramp on and off during the runtime?

No, it's a set-and-forget type of programming. Turning that on/off during
runtime is self-managed and, after the initial setting that this driver,
no further action is required - or at least not from Linux.

Cheers!
Angelo