[PATCH] [POWERPC] DTS cleanup

Segher Boessenkool segher at kernel.crashing.org
Thu Sep 13 08:10:38 EST 2007


> * 32-bit in cpu node -- doesn't exist in any spec and not used by 
> kernel

Yeah.

> * built-in for non-standard buses (ISA, PCI)

"built-in" is some weird CHRP property, so yes we don't need it
or want it.

> * Removed #interrupt-cells in places they don't need to be set

Great :-)

> * Fixed ranges on lite5200*

This has a problem still:

>  		model = "fsl,mpc5200";
>  		compatible = "mpc5200";
>  		revision = "";			// from bootloader
> -		#interrupt-cells = <3>;
>  		device_type = "soc";
> -		ranges = <0 f0000000 f0010000>;
> -		reg = <f0000000 00010000>;
> +		ranges = <0 f0000000 0000c000>;
> +		reg = <f0000000 0000c000>;

That makes "reg" and "ranges" identify an identical address range,
which means no subnode can claim any address in that range, so the
"ranges" property should go.  Alternatively, the "reg" might be
claiming too big a space.

Which is it?


Segher




More information about the Linuxppc-dev mailing list