Received-SPF: pass (google.com: domain of linux-kernel+bounces-206861-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <933f7759-3561-4ad3-8d4e-a05e5fb0ddf3@quicinc.com>
Date: Fri, 7 Jun 2024 17:46:29 -0700
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v8 5/5] remoteproc: qcom: enable in-kernel PD mapper
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
        Bjorn Andersson
	<andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Sibi Sankar
	<quic_sibis@quicinc.com>,
        Mathieu Poirier <mathieu.poirier@linaro.org>
CC: <linux-arm-msm@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <linux-remoteproc@vger.kernel.org>,
        Johan Hovold <johan+linaro@kernel.org>,
        Xilin Wu <wuxilin123@gmail.com>,
        Bryan O'Donoghue
	<bryan.odonoghue@linaro.org>,
        Steev Klimaszewski <steev@kali.org>,
        "Alexey
 Minnekhanov" <alexeymin@postmarketos.org>
References: <20240512-qcom-pd-mapper-v8-0-5ecbb276fcc0@linaro.org>
 <20240512-qcom-pd-mapper-v8-5-5ecbb276fcc0@linaro.org>
Content-Language: en-US
From: Chris Lew <quic_clew@quicinc.com>
In-Reply-To: <20240512-qcom-pd-mapper-v8-5-5ecbb276fcc0@linaro.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit


On 5/11/2024 2:56 PM, Dmitry Baryshkov wrote:
> Request in-kernel protection domain mapper to be started before starting
> Qualcomm DSP and release it once DSP is stopped. Once all DSPs are
> stopped, the PD mapper will be stopped too.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>   drivers/remoteproc/qcom_common.c    | 87 +++++++++++++++++++++++++++++++++++++
>   drivers/remoteproc/qcom_common.h    | 10 +++++
>   drivers/remoteproc/qcom_q6v5_adsp.c |  3 ++
>   drivers/remoteproc/qcom_q6v5_mss.c  |  3 ++
>   drivers/remoteproc/qcom_q6v5_pas.c  |  3 ++
>   drivers/remoteproc/qcom_q6v5_wcss.c |  3 ++
>   6 files changed, 109 insertions(+)
> 

Thanks for looking into whether this could be implemented as a 
remoteproc subdevice.

Reviewed-by: Chris Lew <quic_clew@quicinc.com>