Re: [PATCH v7 2/4] soc: qcom: Add AOSS QMP driver

From: Bjorn Andersson
Date: Sat May 25 2019 - 14:19:57 EST


On Sat 25 May 10:53 PDT 2019, Sai Prakash Ranjan wrote:

> On 5/1/2019 10:07 AM, Bjorn Andersson wrote:
> > The Always On Subsystem (AOSS) Qualcomm Messaging Protocol (QMP) driver
> > is used to communicate with the AOSS for certain side-channel requests,
> > that are not available through the RPMh interface.
> >
> > The communication is a very simple synchronous mechanism of messages
> > being written in message RAM and a doorbell in the AOSS is rung. As the
> > AOSS has processed the message length is cleared and an interrupt is
> > fired by the AOSS as acknowledgment.
> >
> > The driver exposes the QDSS clock as a clock and the low-power state
> > associated with the remoteprocs in the system as a set of power-domains.
> >
> > Signed-off-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
> > ---
> >
> > Changes since v6:
> > - Squash the pd into the same driver as the communication, to simplify
> > the interaction.
> > - Representing the QDSS clocks as a clock/power domain turns out to
> > cascade into a request to make all Coresight drivers have a secondary
> > compatible to replace the required bus clock with a required power
> > domain. So in v7 this is exposed as a clock instead.
> > - Some error checking updates, as reported by Doug.
> >
>
> Thanks for the patch Bjorn.
> Tested the QDSS functionality on SDM845 based Cheza board with this
> change and it works just fine.
>
> Tested-by: Sai Prakash Ranjan <saiprakash.ranjan@xxxxxxxxxxxxxx>

Thanks for the confirmation Sai!

Regards,
Bjorn