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 4F6FCC4828D for ; Thu, 1 Feb 2024 23:50:24 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:CC:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=LVtYJ37EIy/8IK9TDE/Y/N/A/rR20SgOdMFkmVipRjI=; b=JdWRefRUEk+Pmf RO+MMVeRsLz/fTSYXmYabOSJG5zeL3I0bhVL/J3TKQoqkyxNaORj+wYAPsjWljZplwLbfccgDgNOW WHprAZI8yd7u7CJwj1yT1WCdDtMh331cSJV/Nzn/zzYOmYcQ3a7JHNYzClFqpH2zzf1A/BhTwxSMv N9glLHDKJ1eLojvFEnwwvtYQlZxN2j2Ro3Xmd6ZrluDrLxGmin9rfwSx79nO7vRy7T2D9gw23Kzkd 05ZjXeKSVuXFOkdFFfFo7XnSqOAcIEUiGI/0g2KtIGTJqqQ8LRP/14AiGnlGEUhGCq7VjfyC9ICdJ l3qOedr/bHAjhjogwkUA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rVgot-00000009gfd-2d99; Thu, 01 Feb 2024 23:50:11 +0000 Received: from fllv0016.ext.ti.com ([198.47.19.142]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rVgor-00000009gfE-1OVG for linux-arm-kernel@lists.infradead.org; Thu, 01 Feb 2024 23:50:10 +0000 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 411No1hc022509; Thu, 1 Feb 2024 17:50:01 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1706831401; bh=pA3/cTZ7i/yJAMqa6Cw/IrsmxoD/vopbCB/efVaHv1A=; h=Date:Subject:To:CC:References:From:In-Reply-To; b=mCmxhQktzfBdzUR++y9QJ2gjcZ79F6I+u+Pf0YitUGlLEHDivkEWT4ni+98VkTROB CbhrQQ3E97J8mAEx5RwCn6zqB6/ZXfOy/02cj74UtdL9/zks8lbpw1N7CX+JSH05A9 wVnRJ67k/awJrKvCbZBV5/1UIsup2/FztgswVVvM= Received: from DLEE111.ent.ti.com (dlee111.ent.ti.com [157.170.170.22]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 411No1h7022823 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Thu, 1 Feb 2024 17:50:01 -0600 Received: from DLEE104.ent.ti.com (157.170.170.34) by DLEE111.ent.ti.com (157.170.170.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Thu, 1 Feb 2024 17:50:01 -0600 Received: from lelvsmtp6.itg.ti.com (10.180.75.249) by DLEE104.ent.ti.com (157.170.170.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Thu, 1 Feb 2024 17:50:01 -0600 Received: from [10.249.42.149] ([10.249.42.149]) by lelvsmtp6.itg.ti.com (8.15.2/8.15.2) with ESMTP id 411No0Ns130682; Thu, 1 Feb 2024 17:50:01 -0600 Message-ID: <6ba87a3f-e2f4-4caf-8d23-aa78caaba45a@ti.com> Date: Thu, 1 Feb 2024 17:50:00 -0600 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 02/12] dt-bindings: arm: keystone: ti-sci: Add reboot-controller child node Content-Language: en-US To: Rob Herring CC: Nishanth Menon , Tero Kristo , Santosh Shilimkar , Krzysztof Kozlowski , Conor Dooley , Sebastian Reichel , Vignesh Raghavendra , , , , References: <20240131221957.213717-1-afd@ti.com> <20240131221957.213717-3-afd@ti.com> <20240201230351.GA1900918-robh@kernel.org> From: Andrew Davis In-Reply-To: <20240201230351.GA1900918-robh@kernel.org> X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240201_155009_540796_C0517A5B X-CRM114-Status: GOOD ( 16.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: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 2/1/24 5:03 PM, Rob Herring wrote: > On Wed, Jan 31, 2024 at 04:19:47PM -0600, Andrew Davis wrote: >> The TI-SCI firmware supports rebooting the system in addition to the >> functions already listed here, document child node for the same. >> >> Signed-off-by: Andrew Davis >> --- >> .../devicetree/bindings/arm/keystone/ti,sci.yaml | 8 ++++++++ >> 1 file changed, 8 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml >> index c24ad0968f3ef..e392175b33c74 100644 >> --- a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml >> +++ b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml >> @@ -83,6 +83,10 @@ properties: >> type: object >> $ref: /schemas/reset/ti,sci-reset.yaml# >> >> + reboot-controller: >> + type: object >> + $ref: /schemas/power/reset/ti,sci-reboot.yaml# > > Don't need a ref just for a single property. > > But then why do we need a node here at all? Can't you assume reboot > support for TI-SCI firmware (i.e. based on the parent node). Then you > don't need a DT update to add the feature. > We could yes, but then again we could do the same for all the child nodes of this system-controller parent node. Might even have been better that way, for now I'm trying to be consistent with what is already here (child node per service provided, even though the services are always the same). Andrew > Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel