Re: xilinx-pci driver and pci in general

2010-01-18 Thread Grant Likely
On Fri, Jan 15, 2010 at 2:23 AM, Michal Simek wrote: > Benjamin Herrenschmidt wrote: >> On Wed, 2010-01-13 at 16:23 +0100, Michal Simek wrote: >>> Thanks for this early discuss. I would like to hear your opinion and then >>> I will choose solution how to add our pci support to mainline. > > I will

Re: xilinx-pci driver and pci in general

2010-01-18 Thread Grant Likely
On Wed, Jan 13, 2010 at 7:07 PM, Benjamin Herrenschmidt wrote: > On Wed, 2010-01-13 at 16:23 +0100, Michal Simek wrote: > >> The main problems are: >> ppc use ppc_md struct which we don't have it on Microblaze. >> xilinx-pci driver uses exclude_device function. This function is used in >> indirect

Re: xilinx-pci driver and pci in general

2010-01-18 Thread Grant Likely
On Wed, Jan 13, 2010 at 8:23 AM, Michal Simek wrote: > Hi guys, > > We (John and partially I) did initial support for pci on Microblaze. > It is based on powerpc files and almost everything is the same. > There are some small differences which could be easily removed that's why I > think that will

Re: xilinx-pci driver and pci in general

2010-01-15 Thread Michal Simek
Benjamin Herrenschmidt wrote: On Wed, 2010-01-13 at 16:23 +0100, Michal Simek wrote: The main problems are: ppc use ppc_md struct which we don't have it on Microblaze. xilinx-pci driver uses exclude_device function. This function is used in indirect_pci.c too. There could be a way to move that

Re: xilinx-pci driver and pci in general

2010-01-13 Thread Benjamin Herrenschmidt
On Wed, 2010-01-13 at 16:23 +0100, Michal Simek wrote: > The main problems are: > ppc use ppc_md struct which we don't have it on Microblaze. > xilinx-pci driver uses exclude_device function. This function is used in > indirect_pci.c too. There could be a way to move that function directly > to

xilinx-pci driver and pci in general

2010-01-13 Thread Michal Simek
Hi guys, We (John and partially I) did initial support for pci on Microblaze. It is based on powerpc files and almost everything is the same. There are some small differences which could be easily removed that's why I think that will be good to move that file to any generic location. That's why