Re: [PATCH 3/3] remoteproc: qcom_q6v5_pas: Add per-PD proxy performance states for Hawi CDSP

[Bjorn Andersson <andersson@kernel.org>]

On Tue, Sep 01, 2026 at 12:46:21AM +0530, Mukesh Ojha wrote:
> On Mon, Aug 31, 2026 at 10:56:55AM -0500, Bjorn Andersson wrote:
> > On Fri, Aug 28, 2026 at 11:43:11PM +0530, Mukesh Ojha wrote:
> > > The proxy power domain enable path currently requests INT_MAX performance
> > > state for every proxy PD.  While this serves as a "take highest available"
> > > hint, some SoCs require specific per-domain RPMH levels for correct
> > > operation during firmware load rather than a blanket maximum.
> > > 
> > > Introduce a proxy_pd_performance_states array in qcom_pas_data to allow
> > > each proxy PD to declare its required RPMH performance level explicitly.
> > > Platforms that do not populate this field retain the existing INT_MAX
> > > behaviour.
> > > 
> > 
> > Is it possible to encode this using an optional opp-table instead of
> > filling the driver with such details? (This is a question, not a direct
> > suggestion)
> 
> Technically yes, via required-opps, each proxy PD would need its own
> opp-table and the remoteproc node would reference specific OPP entries via
> phandles. That is heavyweight machinery for three scalar values used only
> during the transient firmware-load window. But let me know if you see the
> benefit.
> 

I don't like sprinkling these details throughout drivers, but if that's
the alternative I guess your proposed solution is the lesser of two
evils.

Thank you,
Bjorn