On Wed, Jan 14, 2015 at 07:27:48PM +0200, Michael S. Tsirkin wrote: > pci-iomap.c was (apparently, mistakenly) reintroduced as part of > commit 83c2dc15ce824450e7044b9f90cd529c25747ae0 > MN10300: Handle cacheable PCI regions in pci_iomap() > probably as side-effect of forward-porting the patch > from an old kernel. > > It's not really needed: the generic pci_iomap does the right thing here. > > The new file isn't compiled so it's safe to drop. > > Cc: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > Cc: linux-pci@xxxxxxxxxxxxxxx > Cc: trivial@xxxxxxxxxx > Cc: David Howells <dhowells@xxxxxxxxxx> > Signed-off-by: Michael S. Tsirkin <mst@xxxxxxxxxx> Acked-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > --- > > Can relevant people please ack this for merging through virtio tree? > > arch/mn10300/unit-asb2305/pci-iomap.c | 35 ----------------------------------- > 1 file changed, 35 deletions(-) > delete mode 100644 arch/mn10300/unit-asb2305/pci-iomap.c > > diff --git a/arch/mn10300/unit-asb2305/pci-iomap.c b/arch/mn10300/unit-asb2305/pci-iomap.c > deleted file mode 100644 > index bd65dae..0000000 > --- a/arch/mn10300/unit-asb2305/pci-iomap.c > +++ /dev/null > @@ -1,35 +0,0 @@ > -/* ASB2305 PCI I/O mapping handler > - * > - * Copyright (C) 2007 Red Hat, Inc. All Rights Reserved. > - * Written by David Howells (dhowells@xxxxxxxxxx) > - * > - * This program is free software; you can redistribute it and/or > - * modify it under the terms of the GNU General Public Licence > - * as published by the Free Software Foundation; either version > - * 2 of the Licence, or (at your option) any later version. > - */ > -#include <linux/pci.h> > -#include <linux/module.h> > - > -/* > - * Create a virtual mapping cookie for a PCI BAR (memory or IO) > - */ > -void __iomem *pci_iomap(struct pci_dev *dev, int bar, unsigned long maxlen) > -{ > - resource_size_t start = pci_resource_start(dev, bar); > - resource_size_t len = pci_resource_len(dev, bar); > - unsigned long flags = pci_resource_flags(dev, bar); > - > - if (!len || !start) > - return NULL; > - > - if ((flags & IORESOURCE_IO) || (flags & IORESOURCE_MEM)) { > - if (flags & IORESOURCE_CACHEABLE && !(flags & IORESOURCE_IO)) > - return ioremap(start, len); > - else > - return ioremap_nocache(start, len); > - } > - > - return NULL; > -} > -EXPORT_SYMBOL(pci_iomap); > -- > MST > _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/virtualization