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 797D2C2BD09 for ; Mon, 1 Jul 2024 09:06: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=zijBfwvK55bcBCbj8S8IT/C61ymSE+pURSfjzQCx+bM=; b=cLF6l23z6ZA5y+OL8qTwbWcDVG 96PzDQEiR8OskBII9E9R968nSSBwgMedwai0CO0DXcDgm8Nd74NXHxDVxNEh7BYwizLQGdzZoX+DY PBqF5aUyeaTABwTMPtz+tO9GwP/91OlYjoprui+GC2rYXnHq5MN5OUWUtj8xDJveajIpvDO5P9dyP BvQHri//3DSQ7U3wl+7EpiP/omaKTW9QmSDyhXjFAHH4/A1jqkwuil50K6cBjguF6z3AbAK5TBTO9 xechbCQvWHjFjl7+9qopRUmLyyZUBCGFeTDsAx24w9T5kiFYvAuJ4snHvPxnGnjGtJ7OwBksai2/5 baaGWeuQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOCzN-00000002NWM-1V3H; Mon, 01 Jul 2024 09:06:21 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOCzC-00000002NUL-3ZPB for linux-arm-kernel@lists.infradead.org; Mon, 01 Jul 2024 09:06:12 +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 8627C339; Mon, 1 Jul 2024 02:06:31 -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 D25783F766; Mon, 1 Jul 2024 02:06:04 -0700 (PDT) Date: Mon, 1 Jul 2024 10:06:00 +0100 From: Sudeep Holla To: Peng Fan Cc: Rob Herring , "Peng Fan (OSS)" , Sudeep Holla , Cristian Marussi , Krzysztof Kozlowski , Conor Dooley , "arm-scmi@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 1/2] dt-bindings: firmware: arm,scmi: introduce property mbox-rx-timeout-ms Message-ID: References: <20240621-scmi-mailbox-v1-v1-0-8ed450735f46@nxp.com> <20240621-scmi-mailbox-v1-v1-1-8ed450735f46@nxp.com> <20240627214645.GA614300-robh@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240701_020610_957074_815995EB X-CRM114-Status: GOOD ( 27.59 ) 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 Thu, Jun 27, 2024 at 11:17:49PM +0000, Peng Fan wrote: > > Subject: Re: [PATCH 1/2] dt-bindings: firmware: arm,scmi: introduce > > property mbox-rx-timeout-ms > > > > On Fri, Jun 21, 2024 at 08:46:57PM +0800, Peng Fan (OSS) wrote: > > > From: Peng Fan > > > > > > System Controller Management Interface(SCMI) firmwares might > > have > > > different designs by SCMI firmware developers. So the maximum > > receive > > > channel timeout value might also varies in the various designs. > > > > > > So introduce property mbox-rx-timeout-ms to let each platform could > > > set its own timeout value in device tree. > > > > > > Signed-off-by: Peng Fan > > > --- > > > Documentation/devicetree/bindings/firmware/arm,scmi.yaml | 6 > > ++++++ > > > 1 file changed, 6 insertions(+) > > > > > > diff --git > > a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > > b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > > index 4d823f3b1f0e..d6cc2bf4c819 100644 > > > --- a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > > +++ b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml > > > @@ -121,6 +121,12 @@ properties: > > > atomic mode of operation, even if requested. > > > default: 0 > > > > > > + max-rx-timeout-ms: > > > + description: > > > + An optional time value, expressed in milliseconds, representing, > > on this > > > + platform, the mailbox maximum timeout value for receive > > channel. > > > > "on this platform"? Doesn't every property apply to the given platform? > > Yeah, apply to all the use mailbox. > > > > > > + default: 0 > > > > 0 means no timeout or response is instant? > > I should use 30ms same as what the driver currently use. > That sounds very wrong to me. The binding is independent of current driver behaviour. How the driver handles the case of default 0 value is different from what the default value in the DT means IMO. You can't just set a default value in the DT binding based on the current driver setting. We can always say since it is optional, absence of it is what driver handles as 30ms. 0ms is impossible or incorrect value as transport involves some delay even if it is in terms of uS. So I prefer to set a value of > 0 in DT and make that a requirement. -- Regards, Sudeep