From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8A2931AAC4; Mon, 20 May 2024 19:44:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716234288; cv=none; b=qh5fIyzZUeHQCzNV4snCl/yeCmdfdqsxpi++8SReVBiz9ETOe4SFNlGLGnK6y+SMKxFdhZBBwARq2O6saPRmaYDxx5KyNxE1RvA8IBDEQp012KwbHGTCOA6oOFeyfNrVnCRdJ+hKgx3QWjYP+NP6EyaPwSSCpNEFlACA56ZFl+o= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716234288; c=relaxed/simple; bh=10RuVPJyv0cb0NDS7drH+ekYsGtckzk+dNuzrvyX0L4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=OBqq1wZyQZ6VzR1VS+EKMgHxG+xINSXjr1m1V/QIMrq5M3sF8NoMdrrpN4Nu4/SoKny9PNBfqlBeJKmn2jpqZYrjVF5AqWHrnkO7HziUZaSiwseU9vI7OVc7s3h/L6MW7OqgjCp+/039axn7OneBcJqbWVodxRDzD/JiHeIK38A= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=H0MZ8AYf; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="H0MZ8AYf" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D7A28C2BD10; Mon, 20 May 2024 19:44:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1716234288; bh=10RuVPJyv0cb0NDS7drH+ekYsGtckzk+dNuzrvyX0L4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=H0MZ8AYfLzpSg33qbrY/GlyIG7TurBlDzvbW4IM97qpgkf2laz8xA37tYK5x9I7/F Fb63qN2NBIaLGXlyEAAjtRRtn3zDT1DBUUY3S50NdbEXmtWHKw13QML7rTjyzAfVWK N4sSl5dfqO/xO4Ln0xZZglAyYbVxRaDTxCTfYkv/4ntiAJspw/E2IsAfvtghiKGfbV 8hoM2kLaBTA9imNTw9ufAODojkpgsbbeTuE3cSPcsM02ftgk3V9LhzFM6hiEIsvaHx dXRvOmkHy/vf1NQ4agWEWHmfiPd0KaM0lGyoGFoFFW6sf4JV8qTKC/uskgsn74yo81 axIBT7WkDWwRA== Date: Mon, 20 May 2024 14:44:46 -0500 From: Rob Herring To: Elliot Berman Cc: Bjorn Andersson , Konrad Dybcio , Sebastian Reichel , Krzysztof Kozlowski , Conor Dooley , Vinod Koul , Andy Yan , Lorenzo Pieralisi , Mark Rutland , Bartosz Golaszewski , Satya Durga Srinivasu Prabhala , Melody Olvera , Shivendra Pratap , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Florian Fainelli , linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org Subject: Re: [PATCH v3 2/4] dt-bindings: arm: Document reboot mode magic Message-ID: <20240520194446.GA1368012-robh@kernel.org> References: <20240515-arm-psci-system_reset2-vendor-reboots-v3-0-16dd4f9c0ab4@quicinc.com> <20240515-arm-psci-system_reset2-vendor-reboots-v3-2-16dd4f9c0ab4@quicinc.com> Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240515-arm-psci-system_reset2-vendor-reboots-v3-2-16dd4f9c0ab4@quicinc.com> On Wed, May 15, 2024 at 04:09:45PM -0700, Elliot Berman wrote: > Add bindings to describe vendor-specific reboot modes. Values here > correspond to valid parameters to vendor-specific reset types in PSCI > SYSTEM_RESET2 call. > > Signed-off-by: Elliot Berman > --- > Documentation/devicetree/bindings/arm/psci.yaml | 43 +++++++++++++++++++++++-- > 1 file changed, 41 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/psci.yaml b/Documentation/devicetree/bindings/arm/psci.yaml > index cbb012e217ab..47b5bbe540ce 100644 > --- a/Documentation/devicetree/bindings/arm/psci.yaml > +++ b/Documentation/devicetree/bindings/arm/psci.yaml > @@ -137,8 +137,34 @@ allOf: > required: > - cpu_off > - cpu_on > - > -additionalProperties: false > + - if: > + properties: > + compatible: > + contains: > + const: arm,psci-1.0 > + then: > + properties: > + reset-types: The normal structure is declare all properties and nodes at the top level (outside of if/then schemas) and then add restrictions with if/then schemas. > + type: object > + $ref: /schemas/power/reset/reboot-mode.yaml# additionalProperties: false and a blank line > + properties: > + # "mode-normal" is just SYSTEM_RESET > + mode-normal: false > + patternProperties: > + "^mode-.*$": > + items: > + maxItems: 2 > + description: | > + Describes a vendor-specific reset type. The string after "mode-" > + maps a reboot mode to the parameters in the PSCI SYSTEM_RESET2 call. > + > + Parameters are named mode-xxx = , where xxx > + is the name of the magic reboot mode, type is the lower 31 bits > + of the reset_type, and, optionally, the cookie value. If the cookie > + is not provided, it is defaulted to zero. > + The 31st bit (vendor-resets) will be implicitly set by the driver. > + > +unevaluatedProperties: false > > examples: > - |+ > @@ -261,4 +287,17 @@ examples: > domain-idle-states = <&CLUSTER_RET>, <&CLUSTER_PWRDN>; > }; > }; > + > + - |+ > + > + // Case 5: SYSTEM_RESET2 vendor resets > + psci { > + compatible = "arm,psci-1.0"; > + method = "smc"; > + > + reset-types { > + mode-edl = <0>; > + mode-bootloader = <1 2>; > + }; > + }; > ... > > -- > 2.34.1 > 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 C0997C25B77 for ; Mon, 20 May 2024 19:45:08 +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=Ypu0fwwe9ryU12fFWFat4cQ9tCXI3jV87sNq7dfuGtM=; b=QnQDbvQMFt2aL1 +skVs+WeponpK3GjnxclpTVXuLsEEAKJdiwjUw1OYgoXFlWrl/0lhmsr+LAa3/FFJf4A7boxp91hu 6VJDP20ZP+k47OH3PRDuc335UJLYoEcpLb1WD1JeBUDC5K3b4RZRnTMdww9GoCK9PgAypLqMmBWXB NMeCZSFHLZJeQi6s0PgshpctETqw2idbW9k13dQyoj7O1MVYiNi4jBQCO2sPfBD7mB37G8NNZBy00 0mExHrJGiv78BFVO4QAZsSyJaPVzyZBpvZmGkwAu331AJM8pDgP9r9bopNCCN1GwrZj3KAj0dliwj h5+Nxufc/OoVFPsQLdsQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1s98wJ-0000000FOPZ-3EGP; Mon, 20 May 2024 19:44:55 +0000 Received: from sin.source.kernel.org ([145.40.73.55]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1s98wG-0000000FOP1-25WQ for linux-arm-kernel@lists.infradead.org; Mon, 20 May 2024 19:44:54 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id CFB59CE0D88; Mon, 20 May 2024 19:44:48 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D7A28C2BD10; Mon, 20 May 2024 19:44:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1716234288; bh=10RuVPJyv0cb0NDS7drH+ekYsGtckzk+dNuzrvyX0L4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=H0MZ8AYfLzpSg33qbrY/GlyIG7TurBlDzvbW4IM97qpgkf2laz8xA37tYK5x9I7/F Fb63qN2NBIaLGXlyEAAjtRRtn3zDT1DBUUY3S50NdbEXmtWHKw13QML7rTjyzAfVWK N4sSl5dfqO/xO4Ln0xZZglAyYbVxRaDTxCTfYkv/4ntiAJspw/E2IsAfvtghiKGfbV 8hoM2kLaBTA9imNTw9ufAODojkpgsbbeTuE3cSPcsM02ftgk3V9LhzFM6hiEIsvaHx dXRvOmkHy/vf1NQ4agWEWHmfiPd0KaM0lGyoGFoFFW6sf4JV8qTKC/uskgsn74yo81 axIBT7WkDWwRA== Date: Mon, 20 May 2024 14:44:46 -0500 From: Rob Herring To: Elliot Berman Cc: Bjorn Andersson , Konrad Dybcio , Sebastian Reichel , Krzysztof Kozlowski , Conor Dooley , Vinod Koul , Andy Yan , Lorenzo Pieralisi , Mark Rutland , Bartosz Golaszewski , Satya Durga Srinivasu Prabhala , Melody Olvera , Shivendra Pratap , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Florian Fainelli , linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org Subject: Re: [PATCH v3 2/4] dt-bindings: arm: Document reboot mode magic Message-ID: <20240520194446.GA1368012-robh@kernel.org> References: <20240515-arm-psci-system_reset2-vendor-reboots-v3-0-16dd4f9c0ab4@quicinc.com> <20240515-arm-psci-system_reset2-vendor-reboots-v3-2-16dd4f9c0ab4@quicinc.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240515-arm-psci-system_reset2-vendor-reboots-v3-2-16dd4f9c0ab4@quicinc.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240520_124453_049434_C79BE95B X-CRM114-Status: GOOD ( 20.78 ) 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, May 15, 2024 at 04:09:45PM -0700, Elliot Berman wrote: > Add bindings to describe vendor-specific reboot modes. Values here > correspond to valid parameters to vendor-specific reset types in PSCI > SYSTEM_RESET2 call. > > Signed-off-by: Elliot Berman > --- > Documentation/devicetree/bindings/arm/psci.yaml | 43 +++++++++++++++++++++++-- > 1 file changed, 41 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/psci.yaml b/Documentation/devicetree/bindings/arm/psci.yaml > index cbb012e217ab..47b5bbe540ce 100644 > --- a/Documentation/devicetree/bindings/arm/psci.yaml > +++ b/Documentation/devicetree/bindings/arm/psci.yaml > @@ -137,8 +137,34 @@ allOf: > required: > - cpu_off > - cpu_on > - > -additionalProperties: false > + - if: > + properties: > + compatible: > + contains: > + const: arm,psci-1.0 > + then: > + properties: > + reset-types: The normal structure is declare all properties and nodes at the top level (outside of if/then schemas) and then add restrictions with if/then schemas. > + type: object > + $ref: /schemas/power/reset/reboot-mode.yaml# additionalProperties: false and a blank line > + properties: > + # "mode-normal" is just SYSTEM_RESET > + mode-normal: false > + patternProperties: > + "^mode-.*$": > + items: > + maxItems: 2 > + description: | > + Describes a vendor-specific reset type. The string after "mode-" > + maps a reboot mode to the parameters in the PSCI SYSTEM_RESET2 call. > + > + Parameters are named mode-xxx = , where xxx > + is the name of the magic reboot mode, type is the lower 31 bits > + of the reset_type, and, optionally, the cookie value. If the cookie > + is not provided, it is defaulted to zero. > + The 31st bit (vendor-resets) will be implicitly set by the driver. > + > +unevaluatedProperties: false > > examples: > - |+ > @@ -261,4 +287,17 @@ examples: > domain-idle-states = <&CLUSTER_RET>, <&CLUSTER_PWRDN>; > }; > }; > + > + - |+ > + > + // Case 5: SYSTEM_RESET2 vendor resets > + psci { > + compatible = "arm,psci-1.0"; > + method = "smc"; > + > + reset-types { > + mode-edl = <0>; > + mode-bootloader = <1 2>; > + }; > + }; > ... > > -- > 2.34.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel