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 96365C27C76 for ; Wed, 25 Jan 2023 14:12:17 +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=3FBbkp5Clxc6n9rgCg2l7RZxI0O1MWiycUVLqG/Wfes=; b=zni6bKXZrZTZNT EQCb+WFZas0vF4vu7W8Q61ZwbUBxhdLkb7U4dgWgI9iJObehFBmKL1bFFAZkl4jYctJke4bLAUQpv II/MLPpZoutqB0s6CBANAIEKT/Aa9ErTlv5uZSQlH9DhZeeRUt0fIoLm7lFSlMbdrvWzKAXNoSs3U YR2rwH+3x4uEub/hvYrv8wJqsrt+s/SJvXA+/U9IeLl5WjJeXjFCOZTKc6t5EIDJJcVWwaGJTrkG/ n8PCi/7nUS2vtvLzSW+a+KHTjONULLuPo9uAuBaaCv1yvQIWjF+lkG9IFzRyzFVC52uwy7KCOuYPX WdKy83DIzWvScanXRKEQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pKgUm-007TA7-Br; Wed, 25 Jan 2023 14:11:24 +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 1pKgUj-007T8m-KX for linux-arm-kernel@lists.infradead.org; Wed, 25 Jan 2023 14:11:23 +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 36F384B3; Wed, 25 Jan 2023 06:11:59 -0800 (PST) Received: from bogus (unknown [10.57.77.84]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 4BEFD3F71E; Wed, 25 Jan 2023 06:11:16 -0800 (PST) Date: Wed, 25 Jan 2023 14:11:13 +0000 From: Sudeep Holla To: Cristian Marussi Cc: Rob Herring , Sudeep Holla , 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: <20230125141113.kkbowopusikuogx6@bogus> References: <20230124222023.316089-1-robh@kernel.org> 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-20230125_061121_754900_3638DBAF X-CRM114-Status: GOOD ( 14.17 ) 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 Wed, Jan 25, 2023 at 01:43:48PM +0000, Cristian Marussi wrote: > so now that the catch-all protocol@ patternProperty is gone in favour > of the 'protocol-node' definition and $refs, does that mean that any > current and future SCMI officially published protocol has to be > added to the above explicit protocol list, even though it does not > have any special additional required property beside reg ? > (like protocol@18 above...) > If there are no consumers, should we just not add and deal with it entirely within the kernel. I know we rely today on presence of node before we initialise, but hey we have exception for system power protocol for other reasons, why not add this one too. In short we shouldn't have to add a node if there are no consumers. It was one of the topic of discussion initially when SCMI binding was added and they exist only for the consumers otherwise we don't need it as everything is discoverable from the interface. -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel