On Sat, May 21, 2011 at 21:33, Sam Ravnborg <sam@xxxxxxxxxxxx> wrote: > On Sat, May 21, 2011 at 09:55:28AM +0200, Daniel Hellstrom wrote: >> The LEON architecture does not have a BIOS or bootloader that >> initializes PCI for us, instead Linux generic PCI layer is used >> to set up resources and IRQ. >> >> Signed-off-by: Daniel Hellstrom <daniel@xxxxxxxxxxx> > > Hi Daniel. > > Reading through this patch it looks like LEOON pci support is bolted on top > of what we have rather than trying to integrate it a bit more. > > I would like to see something like: > > * include/asm/pcic.h intregated in pci_32.h > * leon specifics added to pci_32.h - with minimal ifdefs > > Dunno abot the rest... > > If i find time I will try to come up with a proposal later tonight. > > Sam When I looked at this earlier, I was thinking that we might want to introduce a new KConfig variable, maybe CONFIG_LEON_PCI, to hide all this stuff behind so we don't have to do any complicated ifdefery or change any existing ifdefs. Thanks, -- Julian Calaby Email: julian.calaby@xxxxxxxxx Profile: http://www.google.com/profiles/julian.calaby/ .Plan: http://sites.google.com/site/juliancalaby/ -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
