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 A8130C433EF for ; Tue, 8 Feb 2022 17:02:14 +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=I0N6gCfYVD6GatcognSyskjuyYYiT4YtccV0bmLx1lY=; b=kdcPFxG3E1LDnW Ly1t3dXXGt0NTRmXZY3GnVze3iGRGoTUPmXSHvPWuzVt9AByE5NiVkaIkAUSHcP366qkq7sM/5dj9 NrEmekcbwGyxkTZCC3GiQ7DOdW1XT2yZRsy3lA+gLiNj6Pgty2nhv3irzvFlOVj0zYMjXxnD5eLD1 PHx6Ts4ARzD91SDpsHX7ca+b16fRNATnDX2IA7Qt4R9ZXb8Hqw8euSt5qDYCwwpIFVtsc94oP8E2j CXxHxXBnkhoS6HO6wayYjXNNiilsVic86uv8xExyDWO7+PfxCOHOzlFB0JfSDjUqEtYWf0yrj7oEd pzjUu0CglYKrt+EuN+iw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nHTr5-00EyXg-Vp; Tue, 08 Feb 2022 17:00:40 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nHTr1-00EyUK-16 for linux-arm-kernel@lists.infradead.org; Tue, 08 Feb 2022 17:00:36 +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 36B411FB; Tue, 8 Feb 2022 09:00:33 -0800 (PST) Received: from e120937-lin (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 792533F70D; Tue, 8 Feb 2022 09:00:31 -0800 (PST) Date: Tue, 8 Feb 2022 17:00:21 +0000 From: Cristian Marussi To: Florian Fainelli Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, sudeep.holla@arm.com, james.quinlan@broadcom.com, Jonathan.Cameron@Huawei.com, etienne.carriere@linaro.org, vincent.guittot@linaro.org, souvik.chakravarty@arm.com, peter.hilber@opensynergy.com, igor.skalkin@opensynergy.com, Rob Herring , devicetree@vger.kernel.org Subject: Re: [PATCH v3 4/8] dt-bindings: firmware: arm,scmi: Add atomic_threshold optional property Message-ID: <20220208170021.GE2535@e120937-lin> References: <20220208154452.39428-1-cristian.marussi@arm.com> <20220208154452.39428-5-cristian.marussi@arm.com> <681e31c1-54d8-8a3e-ff0f-683cd5156685@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <681e31c1-54d8-8a3e-ff0f-683cd5156685@gmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220208_090035_149493_25007CA4 X-CRM114-Status: GOOD ( 23.46 ) 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, Feb 08, 2022 at 08:12:39AM -0800, Florian Fainelli wrote: > > > On 2/8/2022 7:44 AM, Cristian Marussi wrote: > > SCMI protocols in the platform can optionally signal to the OSPM agent > > the expected execution latency for a specific resource/operation pair. > > > > Introduce an SCMI system wide optional property to describe a global time > > threshold which can be configured on a per-platform base to determine the > > opportunity, or not, for an SCMI command advertised to have a higher > > latency than the threshold, to be considered for atomic operations: > > high-latency SCMI synchronous commands should be preferably issued in the > > usual non-atomic mode. > > Hi Florian, thanks for the feedback first of all. > > Cc: Rob Herring > > Cc: devicetree@vger.kernel.org > > Signed-off-by: Cristian Marussi > > --- > > v1 --> v2 > > - rephrased the property description > > --- > > .../devicetree/bindings/firmware/arm,scmi.yaml | 11 +++++++++++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > index eae15df36eef..646bdf2873b5 100644 > > --- a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > +++ b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > @@ -81,6 +81,15 @@ properties: > > '#size-cells': > > const: 0 > > + atomic_threshold: > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: > > + An optional time value, expressed in microseconds, representing, on this > > + platform, the threshold above which any SCMI command, advertised to have > > + an higher-than-threshold execution latency, should not be considered for > > + atomic mode of operation, even if requested. > > + If left unconfigured defaults to zero. > > Underscores in properties is not usually something that is desired, and it > might be a good idea to put the unit in the property name, how about: > atomic-threshold-us? > I was indeed in doubt if it was desirable or not to have the units embedded in the name ... thanks for the hint about the underscores too, I'll fix nextV following your advice. Thanks, Cristian _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel