* [PATCH] dt-bindings: riscv: firmware: Add OpenSBI configuration binding @ 2026-08-19 9:04 Yaxing Guo 2026-08-19 17:03 ` Conor Dooley 0 siblings, 1 reply; 3+ messages in thread From: Yaxing Guo @ 2026-08-19 9:04 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Paul Walmsley, Palmer Dabbelt, Albert Ou Cc: Alexandre Ghiti, devicetree, inux-kernel, linux-riscv, yangyinglu, yangyinglu From: yangyinglu <yangyinglu@bosc.cn> Add a Devicetree binding for OpenSBI configuration data in the chosen node, including the cold-boot-harts property. OpenSBI supports describing certain Generic Platform configurations in the device tree passed by the previous boot stage. The OpenSBI source tree documents the `opensbi,config` node and provides the corresponding device tree parsing flow. The `/chosen/opensbi-config` node can be used to configure OpenSBI, including the set of harts permitted to perform a cold boot through the `cold-boot-harts` property. OpenSBI parses this node during the boot phase and removes it from the device tree after cold boot. Add the Devicetree binding for this OpenSBI configuration node to document the supported `compatible` and `cold-boot-harts` properties. Signed-off-by: yangyinglu <yangyinglu@bosc.cn> --- .../bindings/firmware/opensbi,config.yaml | 38 +++++++++++++++++++ 1 file changed, 38 insertions(+) create mode 100644 Documentation/devicetree/bindings/firmware/opensbi,config.yaml diff --git a/Documentation/devicetree/bindings/firmware/opensbi,config.yaml b/Documentation/devicetree/bindings/firmware/opensbi,config.yaml new file mode 100644 index 000000000000..86b1e1e673a8 --- /dev/null +++ b/Documentation/devicetree/bindings/firmware/opensbi,config.yaml @@ -0,0 +1,38 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/firmware/opensbi,config.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: OpenSBI Firmware Configuration + +maintainers: + +description: | + OpenSBI configuration data passed in the chosen node. + +properties: + compatible: + const: opensbi,config + + cold-boot-harts: + $ref: /schemas/types.yaml#/definitions/phandle-array + description: + List of hart CPU nodes selected as cold boot harts by OpenSBI. + +required: + - compatible + +additionalProperties: false + +examples: + - | + / { + chosen { + opensbi-config { + compatible = "opensbi,config"; + cold-boot-harts = <&cpu0>; + }; + }; + }; +... -- 2.43.0 _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: riscv: firmware: Add OpenSBI configuration binding 2026-08-19 9:04 [PATCH] dt-bindings: riscv: firmware: Add OpenSBI configuration binding Yaxing Guo @ 2026-08-19 17:03 ` Conor Dooley 2026-08-20 18:20 ` Conor Dooley 0 siblings, 1 reply; 3+ messages in thread From: Conor Dooley @ 2026-08-19 17:03 UTC (permalink / raw) To: Yaxing Guo Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Paul Walmsley, Palmer Dabbelt, Albert Ou, Alexandre Ghiti, devicetree, inux-kernel, linux-riscv, yangyinglu, yangyinglu [-- Attachment #1.1: Type: text/plain, Size: 2721 bytes --] On Wed, Aug 19, 2026 at 05:04:11PM +0800, Yaxing Guo wrote: > From: yangyinglu <yangyinglu@bosc.cn> > > Add a Devicetree binding for OpenSBI configuration data in the > chosen node, including the cold-boot-harts property. > > OpenSBI supports describing certain Generic Platform configurations in > the device tree passed by the previous boot stage. The OpenSBI source > tree documents the `opensbi,config` node and provides the corresponding > device tree parsing flow. > > The `/chosen/opensbi-config` node can be used to configure OpenSBI, > including the set of harts permitted to perform a cold boot through the > `cold-boot-harts` property. OpenSBI parses this node during the boot > phase and removes it from the device tree after cold boot. > > Add the Devicetree binding for this OpenSBI configuration node to > document the supported `compatible` and `cold-boot-harts` properties. > > Signed-off-by: yangyinglu <yangyinglu@bosc.cn> > --- > .../bindings/firmware/opensbi,config.yaml | 38 +++++++++++++++++++ > 1 file changed, 38 insertions(+) > create mode 100644 Documentation/devicetree/bindings/firmware/opensbi,config.yaml > > diff --git a/Documentation/devicetree/bindings/firmware/opensbi,config.yaml b/Documentation/devicetree/bindings/firmware/opensbi,config.yaml > new file mode 100644 > index 000000000000..86b1e1e673a8 > --- /dev/null > +++ b/Documentation/devicetree/bindings/firmware/opensbi,config.yaml > @@ -0,0 +1,38 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/firmware/opensbi,config.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: OpenSBI Firmware Configuration > + > +maintainers: You probably should've waited for a response to your question about who should maintain it before sending, becuase I am not entirely sure this software configuration node should be documented or if the maintainers of opensbi would want it to be. Cheers, Conor. > + > +description: | > + OpenSBI configuration data passed in the chosen node. > + > +properties: > + compatible: > + const: opensbi,config > + > + cold-boot-harts: > + $ref: /schemas/types.yaml#/definitions/phandle-array > + description: > + List of hart CPU nodes selected as cold boot harts by OpenSBI. > + > +required: > + - compatible > + > +additionalProperties: false > + > +examples: > + - | > + / { > + chosen { > + opensbi-config { > + compatible = "opensbi,config"; > + cold-boot-harts = <&cpu0>; > + }; > + }; > + }; > +... > -- > 2.43.0 > [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] [-- Attachment #2: Type: text/plain, Size: 161 bytes --] _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: riscv: firmware: Add OpenSBI configuration binding 2026-08-19 17:03 ` Conor Dooley @ 2026-08-20 18:20 ` Conor Dooley 0 siblings, 0 replies; 3+ messages in thread From: Conor Dooley @ 2026-08-20 18:20 UTC (permalink / raw) To: Yaxing Guo Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Paul Walmsley, Palmer Dabbelt, Albert Ou, Alexandre Ghiti, devicetree, inux-kernel, linux-riscv, yangyinglu, yangyinglu [-- Attachment #1.1: Type: text/plain, Size: 3367 bytes --] On Wed, Aug 19, 2026 at 06:03:12PM +0100, Conor Dooley wrote: > On Wed, Aug 19, 2026 at 05:04:11PM +0800, Yaxing Guo wrote: > > From: yangyinglu <yangyinglu@bosc.cn> > > > > Add a Devicetree binding for OpenSBI configuration data in the > > chosen node, including the cold-boot-harts property. > > > > OpenSBI supports describing certain Generic Platform configurations in > > the device tree passed by the previous boot stage. The OpenSBI source > > tree documents the `opensbi,config` node and provides the corresponding > > device tree parsing flow. > > > > The `/chosen/opensbi-config` node can be used to configure OpenSBI, > > including the set of harts permitted to perform a cold boot through the > > `cold-boot-harts` property. OpenSBI parses this node during the boot > > phase and removes it from the device tree after cold boot. > > > > Add the Devicetree binding for this OpenSBI configuration node to > > document the supported `compatible` and `cold-boot-harts` properties. > > > > Signed-off-by: yangyinglu <yangyinglu@bosc.cn> > > --- > > .../bindings/firmware/opensbi,config.yaml | 38 +++++++++++++++++++ > > 1 file changed, 38 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/firmware/opensbi,config.yaml > > > > diff --git a/Documentation/devicetree/bindings/firmware/opensbi,config.yaml b/Documentation/devicetree/bindings/firmware/opensbi,config.yaml > > new file mode 100644 > > index 000000000000..86b1e1e673a8 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/firmware/opensbi,config.yaml > > @@ -0,0 +1,38 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/firmware/opensbi,config.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: OpenSBI Firmware Configuration > > + > > +maintainers: > > You probably should've waited for a response to your question about who > should maintain it before sending, becuase I am not entirely sure this > software configuration node should be documented or if the maintainers > of opensbi would want it to be. And to quote Anup: | The opensbi configuration is all under "/chosen" DT node so I am | not sure if we should have related bindings under Linux sources. | | Other projects (such as U-boot or Xen) who use "/chosen" DT node | also don't have their bindings under Linux sources. So this is a nack from me on that basis, sorry. Did you have some user for this that could change my mind? Thanks, Conor. > > Cheers, > Conor. > > > + > > +description: | > > + OpenSBI configuration data passed in the chosen node. > > + > > +properties: > > + compatible: > > + const: opensbi,config > > + > > + cold-boot-harts: > > + $ref: /schemas/types.yaml#/definitions/phandle-array > > + description: > > + List of hart CPU nodes selected as cold boot harts by OpenSBI. > > + > > +required: > > + - compatible > > + > > +additionalProperties: false > > + > > +examples: > > + - | > > + / { > > + chosen { > > + opensbi-config { > > + compatible = "opensbi,config"; > > + cold-boot-harts = <&cpu0>; > > + }; > > + }; > > + }; > > +... > > -- > > 2.43.0 > > [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] [-- Attachment #2: Type: text/plain, Size: 161 bytes --] _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-08-20 18:21 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-08-19 9:04 [PATCH] dt-bindings: riscv: firmware: Add OpenSBI configuration binding Yaxing Guo 2026-08-19 17:03 ` Conor Dooley 2026-08-20 18:20 ` Conor Dooley
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox