On Thu, Mar 19, 2009 at 04:26:44PM +0100, Wolfgang Grandegger wrote:
> This patch adds support for the "socrates" board based on the MPC8544.
> Supported are Ethernet, serial console, I2C, I2C-based RTC and
> temperature sensors, NOR and NAND flash, PCI, USB, CAN and Lime
> display controller.
> 
> The multiplexing of FPGA interrupts onto PowerPC interrupt lines is
> supported through our own fpga_pic interrupt controller driver.
> 
> For example the SJA1000 controller is level low sensitive connected to
> fpga_pic line 2 and is routed to the second (of three) irq lines to
> the CPU:

A few minor device tree nits.

> +     soc8...@e0000000 {
> +             #address-cells = <1>;
> +             #size-cells = <1>;
> +             device_type = "soc";
> +
> +             ranges = <0x00000000 0xe0000000 0x00100000>;
> +             reg = <0xe0000000 0x00001000>;  // CCSRBAR 1M
> +             bus-frequency = <0>;            // Filled in by U-Boot
> +             compatible = "fsl,socrates-immr", "simple-bus";

This should probably refer to 8544 instead of socrates.  Unless you
really do have a board-specific version of the SoC...

[snip]
> +             m...@24520 {
> +                     #address-cells = <1>;
> +                     #size-cells = <0>;
> +                     compatible = "fsl,gianfar-mdio";
> +                     reg = <0x24520 0x20>;
> +
> +                     phy0: ethernet-...@0 {
> +                             interrupt-parent = <&mpic>;
> +                             interrupts = <0 1>;
> +                             reg = <0>;
> +                             device_type = "ethernet-phy";

No device_type here.

> +                     };
> +                     phy1: ethernet-...@1 {
> +                             interrupt-parent = <&mpic>;
> +                             interrupts = <0 1>;
> +                             reg = <1>;
> +                             device_type = "ethernet-phy";
> +                     };
> +                     tbi0: tbi-...@11 {
> +                             reg = <0x11>;
> +                             device_type = "tbi-phy";

Or here.

[snip]
> +     localbus {
> +             compatible = "fsl,socrates-localbus",
> +                          "fsl,mpc85xx-localbus",
> +                          "fsl,pq3-localbus";

Is this bus controller part of the SoC?  If so, the first compatible
string probably shouldn't be there (as for the soc/immr node itself).

[snip[
> +             disp...@2,0 {
> +                     compatible = "fujitsu,lime";

This compat string looks slightly worryingly non-specific.

-- 
David Gibson                    | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au  | minimalist, thank you.  NOT _the_ _other_
                                | _way_ _around_!
http://www.ozlabs.org/~dgibson
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to