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
[email protected]
https://ozlabs.org/mailman/listinfo/linuxppc-dev
