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 D23F8C61DA4 for ; Mon, 6 Feb 2023 10:48: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=8NoP2hgY2Bp0XRWIlVTC6m4PjLgGwUaFUNGjZO9ayUY=; b=UYFRIQs2MGoidf orSa87QPaKKt1SZxM1MAEsxVgWmURctYxY8YqooP/YfaTNt4HB95YF8If4cX5J2uxxwMkxH3O3A32 Hz7mHESieghm5rN8PmOrilE1qdqrjC1dybjwad6eU6GB1CXNMYL/uj9IPOMX/zgPUp6oIWQNowAMj JsYrjJYfOxt257cmQo4riHKzCy1sApGzIrhhFWaNMfrT4I+qaWCxarDEIl94OnaZuiGtab5Z0eThH US+hoLUTM5TBOLGaE+PLR4lXnvsRmjOqPdJNnJ7GoZqv6x9V23VXqfkBlyvZgsQJhFFsVCEhcksc1 wGQ3SC9mjAbbRREz5++g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pOz1q-008BXP-33; Mon, 06 Feb 2023 10:47:18 +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 1pOz1j-008BUh-V4 for linux-arm-kernel@lists.infradead.org; Mon, 06 Feb 2023 10:47:13 +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 05CEF13D5; Mon, 6 Feb 2023 02:47:51 -0800 (PST) Received: from bogus (unknown [10.57.12.205]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 42A603F8C6; Mon, 6 Feb 2023 02:47:07 -0800 (PST) Date: Mon, 6 Feb 2023 10:47:04 +0000 From: Sudeep Holla To: Rob Herring Cc: Cristian Marussi , Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] dt-bindings: firmware: arm,scmi: Restrict protocol child node properties Message-ID: <20230206104704.xe72srqygepguuk2@bogus> References: <20230124222023.316089-1-robh@kernel.org> <20230125141113.kkbowopusikuogx6@bogus> <20230126144647.6q3qlu5sqz27cmyc@bogus> <20230126170412.4ytcky6a7lnll6it@bogus> 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-20230206_024712_082316_B9969EA0 X-CRM114-Status: GOOD ( 11.09 ) 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 Hi Rob, On Fri, Jan 27, 2023 at 12:52:33PM -0600, Rob Herring wrote: > > TBC, 'protocol@.*' would not allow anything but the properties defined > in the /$defs/protocol-node. So [1] would throw errors without a > schema addition. Right I clearly missed that, somehow I assumed it would allow. > We should either do that along with dropping 'protocol@18' or we keep > protocol 0x18 node and add all other providerless protocols. I don't > think we need the latter to just check unit-address vs. reg. I only argument today it to allow protocol specific transport. So we could delay addition of it until someone needs that way. So far we haven't seen anyone using it other than performance(even that is not needed with the introduction of fast channels that are auto discoverable in relatively newer versions of the spec). -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel