On Sep 12, 2007, at 5:10 PM, Segher Boessenkool wrote: >> * 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.
Do you suggest we get ride of it from ISA nodes as well? > >> * 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? Yeah, I think it should be 0x100 for the 'soc' regs on 52xx so I'll set regs to that. - k _______________________________________________ Linuxppc-dev mailing list [email protected] https://ozlabs.org/mailman/listinfo/linuxppc-dev
