Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 29 Jun 2022 21:47:15 -0500
From:   Bjorn Andersson <bjorn.andersson@linaro.org>
To:     Sibi Sankar <quic_sibis@quicinc.com>
Cc:     agross@kernel.org, linux-arm-msm@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        krzysztof.kozlowski+dt@linaro.org, robh+dt@kernel.org,
        quic_psodagud@quicinc.com, eberman@codeaurora.org
Subject: Re: [RFC 0/3] Add interconnect support to the SCM interface
Message-ID: <Yr0Os5TOITY7f0Wk@builder.lan>
References: <1653289258-17699-1-git-send-email-quic_sibis@quicinc.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1653289258-17699-1-git-send-email-quic_sibis@quicinc.com>
Precedence: bulk

On Mon 23 May 02:00 CDT 2022, Sibi Sankar wrote:

> Some of the SMC calls required by remoteproc PAS driver on SM8450 SoCs
> get a performance benefit from having a max vote to the crypto->ddr path.
> Add support for bandwidth (bw) voting for those SMC calls when the
> interconnects property is specified. Marking this as an RFC since the path
> could either be specified in the individual remoteprocs or directly in the
> scm interface.
> 

I find it reasonable to state that the clocking needs for the CE relates
to the SCM and not the remoteproc, and it's in line with the management
of CE clocks from the SCM driver.

Regards,
Bjorn

> Sibi Sankar (3):
>   dt-bindings: firmware: qcom-scm: Add interconnects property
>   firmware: qcom_scm: Add bw voting support to the SCM interface
>   arm64: dts: qcom: sm8450: Add interconnect requirements for SCM
> 
>  .../devicetree/bindings/firmware/qcom,scm.txt      |  1 +
>  arch/arm64/boot/dts/qcom/sm8450.dtsi               |  1 +
>  drivers/firmware/qcom_scm.c                        | 69 ++++++++++++++++++++++
>  3 files changed, 71 insertions(+)
> 
> -- 
> 2.7.4
>