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 X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8175DC352A2 for ; Fri, 7 Feb 2020 11:02:09 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 55E4720720 for ; Fri, 7 Feb 2020 11:02:09 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="g0yxLMZi"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="UPDjrC7Q" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 55E4720720 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:Message-ID:References:In-Reply-To:Subject:To:From: Date:MIME-Version:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Mt/Jdmc2tOcIN76JWEZuuNoJJc6nCchTAkkj4bQRJMM=; b=g0yxLMZiUBQ/lOg6PQQQXBIME FROiCtMiDEOLWT7jXFQ9POGLWBWhkPihtkJ+8LYPiVgSJ+StpBxrrNx8qOx+Pd50D58JiRBUMtJh9 bI/D6e0pTK+RGIkgtryg2bZLEz85XZTsbklE7p4qsU9XknGsSPuoYRbqcLacEM1AcimmYswepmr+4 WNc0GPTGQ+bGXbZ35xXuWcbqvfyyCXB+PrdXVwrI6SPCwTW6ToeF1220Tqb0ErxwJQroaKvctpDTn olX2cExGj13QMf454LSNnXvJHbvFJClrurS6xfa+FXQcKZ0Ljmpymq2iklhQPmmRRW/TtSZ4duQZU BjjMjHdAQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1j01Od-0003r8-Rx; Fri, 07 Feb 2020 11:02:03 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1j01Oa-0003qe-Q9 for linux-arm-kernel@lists.infradead.org; Fri, 07 Feb 2020 11:02:02 +0000 Received: from disco-boy.misterjones.org (disco-boy.misterjones.org [51.254.78.96]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 5DFA820720; Fri, 7 Feb 2020 11:02:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1581073320; bh=wzRFLxPbnSe5cu8XeZVGgnId82lJFpJljUiJjz6qjxA=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=UPDjrC7Q3WIJMCxgLStaBCHc+a/KI6Bhi1wWx61drC9s4XOF526fjXXbrYa+mCMh7 2bRx4aWuOj7TpU313SR7AkcDr8xApmExtnuarHcOK2cQ510+yfwSY1BeocAZXk3z1G rSp0y69hD9PJnaBRd8Spa/h2cr96hnJCJF8ljL0I= Received: from disco-boy.misterjones.org ([51.254.78.96] helo=www.loen.fr) by disco-boy.misterjones.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1j01OY-003Vxd-NP; Fri, 07 Feb 2020 11:01:58 +0000 MIME-Version: 1.0 Date: Fri, 07 Feb 2020 11:01:58 +0000 From: Marc Zyngier To: Peng Fan Subject: Re: [PATCH 1/2] dt-bindings: arm: arm,scmi: add smc/hvc transports In-Reply-To: References: <1580994086-17850-1-git-send-email-peng.fan@nxp.com> <1580994086-17850-2-git-send-email-peng.fan@nxp.com> <7875e2533c4ba23b8ca0a2a296699497@kernel.org> <20200207104736.GB36345@bogus> Message-ID: <455c737109271b0536617e179ffa2606@kernel.org> X-Sender: maz@kernel.org User-Agent: Roundcube Webmail/1.3.8 X-SA-Exim-Connect-IP: 51.254.78.96 X-SA-Exim-Rcpt-To: peng.fan@nxp.com, sudeep.holla@arm.com, robh+dt@kernel.org, mark.rutland@arm.com, devicetree@vger.kernel.org, f.fainelli@gmail.com, viresh.kumar@linaro.org, linux-kernel@vger.kernel.org, linux-imx@nxp.com, andre.przywara@arm.com, linux-arm-kernel@lists.infradead.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200207_030200_898480_E92ABFBF X-CRM114-Status: GOOD ( 20.13 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, f.fainelli@gmail.com, viresh.kumar@linaro.org, linux-kernel@vger.kernel.org, andre.przywara@arm.com, robh+dt@kernel.org, dl-linux-imx , Sudeep Holla , linux-arm-kernel@lists.infradead.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 2020-02-07 10:55, Peng Fan wrote: >> Subject: Re: [PATCH 1/2] dt-bindings: arm: arm,scmi: add smc/hvc >> transports >> >> On Fri, Feb 07, 2020 at 10:08:36AM +0000, Marc Zyngier wrote: >> > On 2020-02-06 13:01, peng.fan@nxp.com wrote: >> > > From: Peng Fan >> > > >> > > SCMI could use SMC/HVC as tranports, so add into devicetree binding >> > > doc. >> > > >> > > Signed-off-by: Peng Fan >> > > --- >> > > Documentation/devicetree/bindings/arm/arm,scmi.txt | 4 +++- >> > > 1 file changed, 3 insertions(+), 1 deletion(-) >> > > >> > > diff --git a/Documentation/devicetree/bindings/arm/arm,scmi.txt >> > > b/Documentation/devicetree/bindings/arm/arm,scmi.txt >> > > index f493d69e6194..03cff8b55a93 100644 >> > > --- a/Documentation/devicetree/bindings/arm/arm,scmi.txt >> > > +++ b/Documentation/devicetree/bindings/arm/arm,scmi.txt >> > > @@ -14,7 +14,7 @@ Required properties: >> > > >> > > The scmi node with the following properties shall be under the >> > > /firmware/ node. >> > > >> > > -- compatible : shall be "arm,scmi" >> > > +- compatible : shall be "arm,scmi" or "arm,scmi-smc" >> > > - mboxes: List of phandle and mailbox channel specifiers. It should >> > > contain >> > > exactly one or two mailboxes, one for transmitting messages("tx") >> > > and another optional for receiving the notifications("rx") if @@ >> > > -25,6 +25,8 @@ The scmi node with the following properties shall be >> > > under the /firmware/ node. >> > > protocol identifier for a given sub-node. >> > > - #size-cells : should be '0' as 'reg' property doesn't have any size >> > > associated with it. >> > > +- arm,smc-id : SMC id required when using smc transports >> > > +- arm,hvc-id : HVC id required when using hvc transports >> > > >> > > Optional properties: >> > >> > Not directly related to DT: Why do we need to distinguish between SMC >> > and HVC? >> >> IIUC you want just one property to get the function ID ? Does that >> align with >> what you are saying ? I wanted to ask the same question and I see no >> need for >> 2 different properties. > > The multiple protocols might use SMC or HVC. Saying > > Protocol@x { > method="smc"; > arm,func-id=<0x....> > }; > Protocol@y { > method="hvc"; > arm,func-id=<0x....> > }; > > With my propose: > > Protocol@x { > arm,smc-id=<0x....> > }; > Protocol@y { > arm,hvc-id=<0x....> > }; > > No need an extra method property to indicate it is smc or hvc. > The driver use take arm,smc-id as SMC, arm,hvc-id as HVC. You're missing the point. I do not want to see different IDs depending on Whether we use HVC and SMC. They *MUST* have the same value, and the right way to enforce this is to disallow any indication of the conduit. An even better thing would be for ARM to mandate the ID, so that vendors can be deprived of their special "value add" once and for all. Thanks, M. -- Jazz is not dead. It just smells funny... _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel