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 8C9D4D30011 for ; Fri, 18 Oct 2024 14:12:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=VbdfspHEJETTX5L39+8vuCZNBapbH/JeJL3JH6AALJo=; b=akmVq0MkEZmje335aoTwS0RiiX F15nTNey2ZrQcavR2uLiWjcYsCms22qCqJNk6E5RhdM8K7sMpdTR97IN+T2yK8ewr8ATFmeY+QEmE 2IfWYt8W1lyk3MlJXYaN4ZOMqHsQcaU2kVgrZ8HOGnttyTij8DhPtjzK2SJSu6GK9E5tiAkXtFg5f NQN7Xyfbo+9ytCX6RMTba21P4uswPZmyzEGbxTCS/zkAgsw7DtQwHM994ZeCHJ+lPjWornUNGRNS1 ruxTUqbGOttFQqk1C2p8QM8vICoOzhZ2qDquxPazKNI3j6ru+d7M9cPplMZleUB3rulOIlC3OLMhG waLf4fvA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t1niL-000000010PN-0ObZ; Fri, 18 Oct 2024 14:12:25 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t1ngV-0000000105W-0wtW for linux-arm-kernel@lists.infradead.org; Fri, 18 Oct 2024 14:10:33 +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 0E4C5497; Fri, 18 Oct 2024 07:10:59 -0700 (PDT) Received: from pluto (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id ED8593F58B; Fri, 18 Oct 2024 07:10:26 -0700 (PDT) Date: Fri, 18 Oct 2024 15:10:18 +0100 From: Cristian Marussi To: Rob Herring Cc: Cristian Marussi , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arm-scmi@vger.kernel.org, sudeep.holla@arm.com, james.quinlan@broadcom.com, f.fainelli@gmail.com, vincent.guittot@linaro.org, etienne.carriere@st.com, peng.fan@oss.nxp.com, michal.simek@amd.com, quic_sibis@quicinc.com, quic_nkela@quicinc.com, dan.carpenter@linaro.org, devicetree@vger.kernel.org, Krzysztof Kozlowski Subject: Re: [PATCH 3/5] dt-bindings: firmware: arm,scmi: Introduce more transport properties Message-ID: References: <20241018080602.3952869-1-cristian.marussi@arm.com> <20241018080602.3952869-4-cristian.marussi@arm.com> <20241018133340.GA72220-robh@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241018133340.GA72220-robh@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241018_071031_378170_E0FDF778 X-CRM114-Status: GOOD ( 23.53 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Oct 18, 2024 at 08:33:40AM -0500, Rob Herring wrote: > On Fri, Oct 18, 2024 at 09:06:00AM +0100, Cristian Marussi wrote: > > Depending on specific hardware and firmware design choices, it may be > > possible for different platforms to end up having different requirements > > regarding the same transport characteristics. > > Hi Rob, thanks for having a look. > > Introduce max-msg-size and max-msg properties to describe such platform > > specific transport constraints, since they cannot be discovered otherwise. > > > > Cc: devicetree@vger.kernel.org > > Cc: Rob Herring (Arm) > > Cc: Krzysztof Kozlowski > > Signed-off-by: Cristian Marussi > > --- > > .../devicetree/bindings/firmware/arm,scmi.yaml | 16 ++++++++++++++++ > > 1 file changed, 16 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > index 54d7d11bfed4..42852ed887f2 100644 > > --- a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > +++ b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > @@ -131,6 +131,22 @@ properties: > > be a non-zero value if set. > > minimum: 1 > > > > + max-msg-size: > > Vendor prefix needed. Ok, but these are common properties like max-rx-timeout-ms and others in this bindings so I did not expect to have to explicit set a vendor...well I really dont know what's the rule here, so I went for similarity. > > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: > > + An optional value, expressed in bytes, representing the maximum size > > + allowed for the payload of messages transmitted on this transport. > > + If set it is recommended to be greater or equal than the minimum size > > + required to support all the messages defined by the set of protocols > > + implemented on this platform. > > Sounds kind of broken if less than the minimum... > Well, absolutely but the minimum payload really depends on how many protocols the SCMI server in firmware supports on this platform...the set of implemented messages (some are optional) AND the spec version number...so I cannot state really a fixed safe minimum here...it was just a polite warning for the user not to shot himself in the foot... ..but whoever missteps over this will see a lot of comms errors indeed...I will drop the warning in V2 > > + > > + max-msg: > > Vendor prefix and could be a bit more specific what this is. Ok I will explain more in v2. Thanks, Cristian