Hi Sergei, >> Yes I know ;) I was just wanting to get this out quickly before you kill >> platform.c > > I'd NAK such patch (and have already done so, AFAIR). I've already surrendered myself to the fact that I'll never be able to get rid of this file in my lifetime. However I've set a timer on my mail machine to send a patch (which I'll keep rebasing to latest sources) trying that again in 80 years or so ;-) >> I will make the au1000-eth devices be registered on a per-board basis. > > Please don't. You can register them in platform.c, and yet leave actually > board specific platform data in the board files. There's no reason to > duplicate the platfrom device itself. Let's say I have 2 pieces of hardware, indentical in all things, except one has an Au1100, and the other Au1500 (different MAC mmio address and unit counts). I want to build a kernel which runs on both. This can certainly be done, but the existence of common/platform.c and your insistence on maintaining the status-quo limits me to one board per kernel (theoretical example currently, i know). I also dislike having to #ifdef around this file when a new platform is introduced which doesn't need/use all devices registered in there! (for example au1200 mmc platform data. Suppose I have a platform which doesn't use mmc; I can either add a #ifdef for my new board or provide empty platform data stubs in my board code. Both solutions suck IMO; the former because then when I (and others) submit new board code upstream common/platform.c will develop into a mess of random #ifdefs (just look at common/reset.c!) and the latter because platform data and -device registration are in different places in the source tree. Manuel Lauss
