On Mon, Oct 20, 2008 at 11:14:55AM -0700, Jesse Barnes wrote: > On Monday, October 20, 2008 11:12 am Heiko Carstens wrote: > > On Mon, Oct 20, 2008 at 10:47:41AM -0700, Arjan van de Ven wrote: > > > Heiko Carstens wrote: > > >> Hi Arjan, > > >> > > >> your patch "PCI: introduce an pci_ioremap(pdev, barnr) function" breaks > > >> s390: > > >> > > >> CC fs/compat_ioctl.o > > >> In file included from fs/compat_ioctl.c:52: > > >> include/linux/pci.h: In function 'pci_ioremap_bar': > > >> include/linux/pci.h:1136: error: implicit declaration of function > > >> 'ioremap_nocache' include/linux/pci.h:1137: warning: return makes > > >> pointer from integer without a cast make[1]: *** [fs/compat_ioctl.o] > > >> Error 1 > > >> make: *** [fs] Error 2 > > > > > > hmm does s390 have ioremap ? > > > since that'd be the most useful solution, to add an ioremap_nocache(). > > > > s390 doesn't have ioremap. > > > > > if it doesn't have ioremap.. wtf is it doing using pci.h then ?? > > > > Several common code parts pull pci.h, not s390. > > fs/compat_ioctl.c is one, drivers/scsi/scsi_lib.c is another one. > > Hm, well it would probably be best to fix those (maybe add some #ifdef > CONFIG_PCI to them or somesuch), but that could get ugly. I'll go ahead and > apply your initial patch. Thanks! Patch was Signed-off-by: Heiko Carstens <heiko.carstens@xxxxxxxxxx> in case you need that. -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
