Re: Oops with TQM5200 on TQM5200

[Wolfgang Grandegger <wg@grandegger.com>]

Hi Anatolij,

Anatolij Gustschin wrote:
> Hello Wolfgang,
> 
> Wolfgang Grandegger wrote:
> 
>> I just tried Linux 2.6.25-rc6 on my TQM5200 module and got the attached
>> oops. Are there any known patches fixing the problems?
> 
> try the patch below for tqm5200.dts, rebuild dtb and boot
> again. Not sure if it works for Linux 2.6.25-rc6, but for
> 2.6.25-rc3 it does.
> 
> Anatolij
> -- 
> diff --git a/arch/powerpc/boot/dts/tqm5200.dts
> b/arch/powerpc/boot/dts/tqm5200.dts
> index c86464f..7c23bb3 100644
> --- a/arch/powerpc/boot/dts/tqm5200.dts
> +++ b/arch/powerpc/boot/dts/tqm5200.dts
> @@ -83,6 +83,7 @@
>         };
> 
>         dma-controller@1200 {
> +            device_type = "dma-controller";

This one actually fixes the Oops. I already had the following fixes.

>             compatible = "fsl,mpc5200-bestcomm";
>             reg = <1200 80>;
>             interrupts = <3 0 0  3 1 0  3 2 0  3 3 0
> @@ -127,10 +128,25 @@
>         ethernet@3000 {
>             device_type = "network";
>             compatible = "fsl,mpc5200-fec";
> -            reg = <3000 800>;
> +            reg = <3000 400>;

This has already been fixed a long time ago, IIRC.

>             local-mac-address = [ 00 00 00 00 00 00 ];
>             interrupts = <2 5 0>;
>             interrupt-parent = <&mpc5200_pic>;
> +            phy-handle = <&phy0>;
> +        };
> +
> +        mdio@3000 {
> +            #address-cells = <1>;
> +            #size-cells = <0>;
> +            compatible = "fsl,mpc5200b-mdio";
> +            reg = <3000 400>;       // fec range, since we need to
> setup fec interrupts
> +            interrupts = <2 5 0>;   // these are for "mii command
> finished", not link changes & co.
> +            interrupt-parent = <&mpc5200_pic>;
> +
> +            phy0:ethernet-phy@0 {
> +                device_type = "ethernet-phy";
> +                reg = <0>;
> +            };

Another question. Is it possible to configure Linux without explicit
PHY-handling.

Thanks,

Wolfgang.