On Wednesday 10 February 2016 09:27:07 Paul Burton wrote: > On Wed, Feb 10, 2016 at 05:55:51AM +0000, Bharat Kumar Gogada wrote: > > > On Tue, Feb 09, 2016 at 04:11:56PM +0530, Bharat Kumar Gogada wrote: > > > > Modifying Xilinx AXI PCIe Host Bridge Soft IP driver to work on both > > > > Zynq and Microblaze Architectures. > > > > With these modifications drivers/pci/host/pcie-xilinx.c, will work on > > > > both Zynq and Microblaze Architectures. > > > > > > > > Signed-off-by: Bharat Kumar Gogada <bharatku@xxxxxxxxxx> > > > > Signed-off-by: Ravi Kiran Gummaluri <rgummal@xxxxxxxxxx> > > > > --- > > > > Changes: > > > > Removed unneccessary architecture dependent number of MSI's. > > > > Added #ifdef to pci_fixup_irqs which is ARM specific API. > > > > > > Hi Bharat, > > > > > > Why do you say pci_fixup_irqs is ARM-specific? It's declared in > > > include/linux/pci.h, defined in drivers/pci/setup-irq.c & used by multiple > > > architectures (alpha, arm, m68k, mips, sh, sparc, tile, > > > unicore32 from a quick grep). > > > > > > Will you not break INTX-style interrupts by removing this? > > > > > I meant to say ARM specific w.r.t Microblaze architecture, which is > > what this patch series are for. This has been already discussed in my > > previous patch by Arnd Bergmann and Lorenzo Pieralisi . > > (https://lkml.org/lkml/2016/1/12/707) > > Hi Bharat, > > Ok, so you don't need it for microblaze but do need it for zynq/ARM. We > also need it for MIPS, where my recent patches enable this driver. So if > #ifdef'ing this is the current way forwards could you please invert the > condition to #ifndef CONFIG_MICROBLAZE? I think we are getting to the point where we should try much harder to make sure nobody needs that hack and it all works out of the box. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
