2017-12-09 15:34:06

by J. Neuschäfer

[permalink] [raw]
Subject: [PATCH] dt-bindings: chosen: Document linux,initrd-{start,end}

These properties have been in use for a very long time (at least since
2005), but were never documented in chosen.txt.

Signed-off-by: Jonathan Neuschäfer <[email protected]>
---
Documentation/devicetree/bindings/chosen.txt | 15 +++++++++++++++
1 file changed, 15 insertions(+)

diff --git a/Documentation/devicetree/bindings/chosen.txt b/Documentation/devicetree/bindings/chosen.txt
index e3b13ea7d2ae..45e79172a646 100644
--- a/Documentation/devicetree/bindings/chosen.txt
+++ b/Documentation/devicetree/bindings/chosen.txt
@@ -120,3 +120,18 @@ e.g.
While this property does not represent a real hardware, the address
and the size are expressed in #address-cells and #size-cells,
respectively, of the root node.
+
+linux,initrd-start and linux,initrd-end
+---------------------------------------
+
+These properties hold the physical start and end address of an initrd that's
+loaded by the bootloader. Note that linux,initrd-start is inclusive, but
+linux,initrd-end is exclusive.
+e.g.
+
+/ {
+ chosen {
+ linux,initrd-start = <0x82000000>;
+ linux,initrd-end = <0x82800000>;
+ };
+};
--
2.15.0


2017-12-15 21:01:54

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: chosen: Document linux,initrd-{start,end}

On Sat, Dec 09, 2017 at 04:33:02PM +0100, Jonathan Neusch?fer wrote:
> These properties have been in use for a very long time (at least since
> 2005), but were never documented in chosen.txt.
>
> Signed-off-by: Jonathan Neusch?fer <[email protected]>
> ---
> Documentation/devicetree/bindings/chosen.txt | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)

Applied.

I'm inclined to say to document these in the DT spec, but I'm assuming
there was some reason why they weren't put into the spec (ePAPR at the
time) originally.

Rob

2017-12-16 05:41:54

by J. Neuschäfer

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: chosen: Document linux,initrd-{start,end}

On Fri, Dec 15, 2017 at 03:01:47PM -0600, Rob Herring wrote:
> On Sat, Dec 09, 2017 at 04:33:02PM +0100, Jonathan Neuschäfer wrote:
> > These properties have been in use for a very long time (at least since
> > 2005), but were never documented in chosen.txt.
> >
> > Signed-off-by: Jonathan Neuschäfer <[email protected]>
> > ---
> > Documentation/devicetree/bindings/chosen.txt | 15 +++++++++++++++
> > 1 file changed, 15 insertions(+)
>
> Applied.

Thanks.

> I'm inclined to say to document these in the DT spec, but I'm assuming
> there was some reason why they weren't put into the spec (ePAPR at the
> time) originally.

I don't know about the history of this, but I think if and when these
properties were specified in DTSpec, they should get a non-linux-specific
name, such as initrd-start/initrd-end, and a compatibility fallback to
linux,initrd-* (similar to stdout-path and phandle).


Jonathan Neuschäfer


Attachments:
(No filename) (930.00 B)
signature.asc (819.00 B)
Download all attachments