Re: [PATCH] [POWERPC] DTS cleanup

2007-09-16 Thread David Gibson
On Fri, Sep 14, 2007 at 01:11:07PM -0500, Kumar Gala wrote: > > On Sep 14, 2007, at 11:28 AM, Jon Loeliger wrote: > > > Kumar Gala wrote: > > > >> Someone really needs to add some macro/preprocessor magic into DTC > >> so this is made a lot simpler. > >> - k > > > > Kumar, > > > > I am serious

Re: [PATCH] [POWERPC] DTS cleanup

2007-09-14 Thread Kumar Gala
On Sep 14, 2007, at 11:28 AM, Jon Loeliger wrote: > Kumar Gala wrote: > >> Someone really needs to add some macro/preprocessor magic into DTC >> so this is made a lot simpler. >> - k > > Kumar, > > I am seriously contemplating this problem. > The trick is, I need to quit with the whole > Power

Re: [PATCH] [POWERPC] DTS cleanup

2007-09-14 Thread Jon Loeliger
Kumar Gala wrote: > Someone really needs to add some macro/preprocessor magic into DTC so > this is made a lot simpler. > > - k Kumar, I am seriously contemplating this problem. The trick is, I need to quit with the whole Power Point Programmer job title for a bit... jdl

Re: [PATCH] [POWERPC] DTS cleanup

2007-09-13 Thread Kumar Gala
On Sep 13, 2007, at 11:56 AM, Segher Boessenkool wrote: * 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? > > Yes. You aren't CHRP so you don't h

Re: [PATCH] [POWERPC] DTS cleanup

2007-09-13 Thread Segher Boessenkool
>>> * 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? Yes. You aren't CHRP so you don't have to follow the CHRP binding. Of course it is good to copy from t

Re: [PATCH] [POWERPC] DTS cleanup

2007-09-12 Thread Kumar Gala
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

Re: [PATCH] [POWERPC] DTS cleanup

2007-09-12 Thread Segher Boessenkool
> * 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 r

[PATCH] [POWERPC] DTS cleanup

2007-09-12 Thread Kumar Gala
Removed the following cruft from .dts files: * 32-bit in cpu node -- doesn't exist in any spec and not used by kernel * built-in for non-standard buses (ISA, PCI) * Removed #interrupt-cells in places they don't need to be set * Fixed ranges on lite5200* --- Exists in my for-2.6.24 branch arch/p