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, 7 Sep 2022 13:01:44 +0300
From:   Abel Vesa <abel.vesa@linaro.org>
To:     Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Cc:     Amol Maheshwari <amahesh@qti.qualcomm.com>,
        Andy Gross <agross@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@somainline.org>,
        Ekansh Gupta <quic_ekagupt@quicinc.com>,
        Rob Herring <robh@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Arnd Bergmann <arnd@arndb.de>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-arm-msm@vger.kernel.org,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        devicetree@vger.kernel.org
Subject: Re: [PATCH v2 00/10] misc: fastrpc: Add audiopd support
Message-ID: <YxhsCKEWFmlFgKWU@linaro.org>
References: <20220902154900.3404524-1-abel.vesa@linaro.org>
 <87f59c6d-c2ad-25c2-a0cf-972b5df42bce@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <87f59c6d-c2ad-25c2-a0cf-972b5df42bce@linaro.org>
Precedence: bulk

On 22-09-06 15:12:39, Srinivas Kandagatla wrote:
> Hi Abel,
> Thanks for picking up these patches and reworking.
> 
> On 02/09/2022 16:48, Abel Vesa wrote:
> > This patchset adds audiopd support to fastrpc.
> > 
> > The first version is here:
> > https://lore.kernel.org/all/20220902131344.3029826-1-abel.vesa@linaro.org/
> > 
> 
> I have tested this on sm8450 with audiopd and loading Single MIC ECNS module
> to adsp.
> 
> Which platforms did you test these patches on?

I have tested on sm8250, with memory-region property and vmids added
locally to the adsp fastrpc devicetree node.

> 
> 
> Tested-by: Srinivas Kandagatla  <srinivas.kandagatla@linaro.org>

Thanks for testing it on sm8450 too.

> 
> 
> --srini
> 
> 
> > Changes since v1:
> >   * dropped the patch 13:
> >     "misc: fastrpc: Remove unnecessary if braces in fastrpc_internal_invoke"
> >   * sent patches 1, 2 and 3 as a separate patchset
> > 
> > Abel Vesa (10):
> >    misc: fastrpc: Rename audio protection domain to root
> >    misc: fastrpc: Add reserved mem support
> >    dt-bindings: misc: fastrpc: Document memory-region property
> >    misc: fastrpc: Add fastrpc_remote_heap_alloc
> >    misc: fastrpc: Use fastrpc_map_put in fastrpc_map_create on fail
> >    misc: fastrpc: Rework fastrpc_req_munmap
> >    misc: fastrpc: Add support for audiopd
> >    misc: fastrpc: Safekeep mmaps on interrupted invoke
> >    misc: fastrpc: Add mmap request assigning for static PD pool
> >    misc: fastrpc: Add dma_mask to fastrpc_channel_ctx
> > 
> >   .../devicetree/bindings/misc/qcom,fastrpc.txt |   5 +
> >   drivers/misc/fastrpc.c                        | 267 +++++++++++++++---
> >   include/uapi/misc/fastrpc.h                   |   7 +
> >   3 files changed, 247 insertions(+), 32 deletions(-)
> >