On Sun, Jun 8, 2008 at 1:08 PM, Jon Smirl <[EMAIL PROTECTED]> wrote:
> Why not a compatible field in the top of the tree? Then you wouldn't
> need to list the boards in mpc5200_simple.c.
> compatible = "phytec,pcm030","simple-mpc5200";
Here's the problem; what does compatible really mean at the board
level? Does it mean the board has 100% of the same capabilities?
Does it mean that it uses the same chip? Does it mean that the chip
is configured in a particular way? It is really hard to define what a
compatible value means at the board level. The meaning of compatible
at the device level is very well defined, but that meaning does not
extend well to the board level.
Therefore, it is best to be conservative here and require a specific
list of supported boards in platform code.
Besides, it is really a Linux specific thing that is trying to be
described. In Linux, we've decided to support as many 5200 boards as
possible using the same platform code, but that may not be true, or it
may be a different set of boards, when a different OS is used. To
attempt to encode those decisions is overreaching the intent of using
the device tree.
>
> Device tree has an entry for AC97 on PSC1. I don't think the Phytec
> module or carrier board has AC97 hardware.
Might be a bug
> The RTC chip says pcf8563, phytec doc says it is a pcf8564.
Sounds like a bug
>
> There should be an i2c entry for the eeprom but I don't know the part
> number for it.
Yes, i2c devices should be described.
> What about the flash on the local bus? Could we use something like
> this, or the same without the partition data?
Yes, I believe there is code in place to support this.
>
> [EMAIL PROTECTED] {
> compatible = "fsl,lpb";
> ranges = <0 ff000000 01000000>;
>
> [EMAIL PROTECTED] {
> compatible = "cfi-flash";
> reg = <00000000 01000000>;
> bank-width = <2>;
> #size-cells = <1>;
> #address-cells = <1>;
> [EMAIL PROTECTED] {
> label = "ubootl";
> reg = <00000000 00040000>;
> };
> [EMAIL PROTECTED] {
> label = "kernel";
> reg = <00040000 001c0000>;
> };
> [EMAIL PROTECTED] {
> label = "jffs2";
> reg = <00200000 00D00000>;
> };
> [EMAIL PROTECTED] {
> label = "uboot";
> reg = <00f00000 00040000>;
> };
> [EMAIL PROTECTED] {
> label = "oftree";
> reg = <00f40000 00040000>;
> };
> [EMAIL PROTECTED] {
> label = "space";
> reg = <00f80000 00080000>;
> };
> };
> };
Cheers,
g.
--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
