> +   j) CFI or JEDEC memory-mapped NOR flash
>
>      Flash chips (Memory Technology Devices) are often used for solid 
> state
>      file systems on embedded devices.

Well, almost everything has a NOR flash on it, not just
embedded boards ;-)

> +     - bank-width : Width (in bytes) of the flash bank.  Equal to the
> +       device width times the number of interleaved chips.
> +     - device-width : (optional) Width of a single flash chip.  If
> +       omitted, assumed to be equal to 'bank-width'.

Let's have bank-width optional instead, it's more natural
that way for the common case of just one chip.  Or, you can
say that either is optional.

> +     [EMAIL PROTECTED] {
> +             compatible = "amd,am29lv128ml", "cfi-flash";
> +             reg = <ff000000 01000000>;
> +             bank-width = <4>;

This is an 8/16-bit part, you need a device-width ;-)

Need #address-cells here for the child nodes:

> +             [EMAIL PROTECTED] {
> +                     reg = <0 f80000>;
> +             };

[big snip]

> +                                     [EMAIL PROTECTED] {

This show immediately why node name = partition name won't
work out.  You're not supposed to start a node name with a
capital like this.


Segher

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to