Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) client-ip=23.128.96.38;
Message-ID: <0efe305e-031b-bdf5-0268-ca1c6d562653@quicinc.com>
Date:   Mon, 18 Sep 2023 08:01:26 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.10.0
Subject: Re: [PATCH v4 0/4] Add qcom hvc/shmem transport support
To:     <sudeep.holla@arm.com>
CC:     <cristian.marussi@arm.com>, <robh+dt@kernel.org>,
        <krzysztof.kozlowski+dt@linaro.org>, <conor+dt@kernel.org>,
        <andersson@kernel.org>, <konrad.dybcio@linaro.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <linux-arm-msm@vger.kernel.org>
References: <20230718160833.36397-1-quic_nkela@quicinc.com>
 <20230911194359.27547-1-quic_nkela@quicinc.com>
Content-Language: en-US
From:   Nikunj Kela <quic_nkela@quicinc.com>
In-Reply-To: <20230911194359.27547-1-quic_nkela@quicinc.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

Gentle Ping!

On 9/11/2023 12:43 PM, Nikunj Kela wrote:
> This change augments smc transport to include support for Qualcomm virtual
> platforms by passing a parameter(capability-id) in the hypervisor call to
> identify which doorbell to assert. This parameter is dynamically generated
> at runtime on the device and insuitable to pass via the devicetree.
>
> The function ID and parameter are stored by firmware in the shmem region.
>
> This has been tested on ARM64 virtual Qualcomm platform.
>
> ---
> v4 -> port the changes into smc.c
>
> v3 -> fix the compilation error reported by the test bot,
>        add support for polling based instances
>
> v2 -> use allOf construct in dtb schema,
>        remove wrappers from mutexes,
>        use architecture independent channel layout
>
> v1 -> original patches
>
> Nikunj Kela (4):
>    firmware: arm_scmi: Add polling support for completion in smc
>    dt-bindings: arm: convert nested if-else construct to allOf
>    dt-bindings: arm: Add new compatible for smc/hvc transport for SCMI
>    firmware: arm_scmi: Add qcom hvc/shmem transport support
>
>   .../bindings/firmware/arm,scmi.yaml           | 67 +++++++++++--------
>   drivers/firmware/arm_scmi/Kconfig             | 14 ++++
>   drivers/firmware/arm_scmi/driver.c            |  1 +
>   drivers/firmware/arm_scmi/smc.c               | 62 +++++++++++++++--
>   4 files changed, 110 insertions(+), 34 deletions(-)
>