Message ID | 1239093583-12201-1-git-send-email-sr@denx.de |
---|---|
State | New, archived |
Headers | show |
On Tue, Apr 7, 2009 at 2:39 AM, Stefan Roese <sr@denx.de> wrote: > Signed-off-by: Stefan Roese <sr@denx.de> > CC: Grant Likely <grant.likely@secretlab.ca> > --- > Documentation/powerpc/dts-bindings/mtd-physmap.txt | 20 +++++++++++++++++++- > 1 files changed, 19 insertions(+), 1 deletions(-) > > diff --git a/Documentation/powerpc/dts-bindings/mtd-physmap.txt b/Documentation/powerpc/dts-bindings/mtd-physmap.txt > index ab1115e..a3fa9e0 100644 > --- a/Documentation/powerpc/dts-bindings/mtd-physmap.txt > +++ b/Documentation/powerpc/dts-bindings/mtd-physmap.txt > @@ -5,7 +5,10 @@ file systems on embedded devices. > > - compatible : should contain the specific model of flash chip(s) > used, if known, followed by either "cfi-flash" or "jedec-flash" > - - reg : Address range of the flash chip > + - reg : Address range(s) of the flash chip(s) > + It's possible to (optionally) define multiple "reg" tuples so that the > + Intel P30 48F4400 chip which consists internally of 2 non-identical > + NOR chips on one die can also be supported. This isn't a P30 48F4400 specific feature, even if it is the only current user. This description works for any arrangement of multiple NOR chips. I isn't really appropriate to mention the P30 in this way and it should be more generic. > - bank-width : Width (in bytes) of the flash bank. Equal to the > device width times the number of interleaved chips. > - #address-cells, #size-cells : Must be present if the flash has > @@ -58,3 +61,18 @@ Example: > read-only; > }; > }; > + > +Here an example with multiple "reg" tuples (e.g. Intel P30): However, it is completely appropriate to mention P30 here because it is in the context of an example. > + > + flash@f0000000,0 { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "cfi-flash"; For best practice, compatible should also specify the exact chip in this case. g.
diff --git a/Documentation/powerpc/dts-bindings/mtd-physmap.txt b/Documentation/powerpc/dts-bindings/mtd-physmap.txt index ab1115e..a3fa9e0 100644 --- a/Documentation/powerpc/dts-bindings/mtd-physmap.txt +++ b/Documentation/powerpc/dts-bindings/mtd-physmap.txt @@ -5,7 +5,10 @@ file systems on embedded devices. - compatible : should contain the specific model of flash chip(s) used, if known, followed by either "cfi-flash" or "jedec-flash" - - reg : Address range of the flash chip + - reg : Address range(s) of the flash chip(s) + It's possible to (optionally) define multiple "reg" tuples so that the + Intel P30 48F4400 chip which consists internally of 2 non-identical + NOR chips on one die can also be supported. - bank-width : Width (in bytes) of the flash bank. Equal to the device width times the number of interleaved chips. - #address-cells, #size-cells : Must be present if the flash has @@ -58,3 +61,18 @@ Example: read-only; }; }; + +Here an example with multiple "reg" tuples (e.g. Intel P30): + + flash@f0000000,0 { + #address-cells = <1>; + #size-cells = <1>; + compatible = "cfi-flash"; + reg = <0 0x00000000 0x02000000 + 0 0x02000000 0x02000000>; + bank-width = <2>; + partition@0 { + label = "test-part1"; + reg = <0 0x04000000>; + }; + };
Signed-off-by: Stefan Roese <sr@denx.de> CC: Grant Likely <grant.likely@secretlab.ca> --- Documentation/powerpc/dts-bindings/mtd-physmap.txt | 20 +++++++++++++++++++- 1 files changed, 19 insertions(+), 1 deletions(-)