From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 15B2AEE14DB for ; Thu, 7 Sep 2023 10:37:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=QpZQh42UZ3Xa9WlTrSo8FYqm6ZN1Fm6FNMlO2XK+RY0=; b=CMWR4dPg7aHETY DYiTrK6PgoaRW3l3eCr2WeT8t6FhfpJiW4vyr2bvcSZVIrBRe5DLCSq7tZcOC8al4v0ThHXPFkGTo G9t0JVy2KPNT5S1jK/SF0QKIrC8IqptLhid9poAVNNtTUP+hfPQsE83iEeloBNL1CoLESk9hTM3QD 349U7B1jdDoKfKQZIzjCFnimkE8wl3owWAFEsouxIwgxceMmA3gUWsuEXhoFi8Ypftcv6kS7ITc19 Ss1R5KZSi1bKSi1GgYBPkVLOr/7FZWUPWOOvD3POMPhdX15kDtLJJcRyX/ExUdKMl2Wl96ah8QzgY FomEtKmKsCRoULp9D4sg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qeCNH-00Bn2Z-26; Thu, 07 Sep 2023 10:36:35 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qeCNE-00Bn1g-2B for linux-arm-kernel@lists.infradead.org; Thu, 07 Sep 2023 10:36:34 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B8F381576; Thu, 7 Sep 2023 03:37:01 -0700 (PDT) Received: from bogus (e103737-lin.cambridge.arm.com [10.1.197.49]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id CBA573F67D; Thu, 7 Sep 2023 03:36:21 -0700 (PDT) Date: Thu, 7 Sep 2023 11:36:19 +0100 From: Sudeep Holla To: Nikunj Kela Cc: Krzysztof Kozlowski , cristian.marussi@arm.com, Sudeep Holla , robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, agross@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 Subject: Re: [PATCH v3 0/3] Add qcom hvc/shmem transport Message-ID: <20230907103619.2kqh7tfivwdfm5rd@bogus> References: <20230718160833.36397-1-quic_nkela@quicinc.com> <20230811175719.28378-1-quic_nkela@quicinc.com> <3342d8bf-5281-c082-cb9a-7a027b413237@quicinc.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230907_033632_810169_B39664E8 X-CRM114-Status: GOOD ( 29.23 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Sep 05, 2023 at 06:37:14PM +0200, Krzysztof Kozlowski wrote: > On 05/09/2023 18:06, Nikunj Kela wrote: > > > > On 8/11/2023 10:57 AM, Nikunj Kela wrote: > >> This change introduce a new transport channel for Qualcomm virtual > >> platforms. The transport is mechanically similar to ARM_SCMI_TRANSPORT_SMC. > >> The difference between the two transports is that a parameter is passed 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. > >> > >> --- > >> 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 (3): > >> dt-bindings: arm: convert nested if-else construct to allOf > >> dt-bindings: arm: Add qcom specific hvc transport for SCMI > >> firmware: arm_scmi: Add qcom hvc/shmem transport > >> > >> .../bindings/firmware/arm,scmi.yaml | 67 ++--- > >> drivers/firmware/arm_scmi/Kconfig | 13 + > >> drivers/firmware/arm_scmi/Makefile | 2 + > >> drivers/firmware/arm_scmi/common.h | 3 + > >> drivers/firmware/arm_scmi/driver.c | 4 + > >> drivers/firmware/arm_scmi/qcom_hvc.c | 232 ++++++++++++++++++ > >> 6 files changed, 293 insertions(+), 28 deletions(-) > >> create mode 100644 drivers/firmware/arm_scmi/qcom_hvc.c > > Gentle Ping! Pong ! > > It's third ping these two weeks from Qualcomm. Folks, it is merge > window. What do you think will happen with your ping during this time? > +1 Okay, here is the deal with this patch set. As you are aware that a previous merged solution was abandoned by Qcom in a single kernel release cycle. So I decided to ignore this for one or 2 kernel release cycle to make sure Qcom makes up their mind on the design and then we can see how to proceed. Qcom must understand upstream kernel is not a playground to push their design which they might decided to drop support for in such short period. Please understand the upstream kernel supports platforms that are more than few decades old. It is not like the mobile platforms that are hardly supported for couple of years. And similarly, we push core support if and only if we know for sure it will be used on some platform. I trusted Qcom with the previous extension of SMC/HVC transport but I was proven wrong. Also, I definitely don't like the way you have copied the whole smc.c and changed it to Qcom's need and made it qcom_hvc.c. Just add the required changes in smc.c. -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel