On 5/25/2010 5:45 PM, Arnd Bergmann wrote: >> +u32 inb(u32 addr) >> > +{ >> > + panic("inb not implemented"); >> > +} >> > +EXPORT_SYMBOL(inb); >> > If you just remove these definitions, you get a link error for any > driver that tries to use these, which is probably more helpful than > the panic. > > OTOH, are you sure that you can't just map the PIO calls to mmio functions > like readb plus some fixed offset? On most non-x86 architectures, the PIO > area of the PCI bus is just mapped to a memory range somewhere. > I looked at where inb() and friends are used, and although we can get close to not requiring them for tile, it's not quite there, and will need some further discussions on the various lists to clean up enough for our purposes. There are three problems that surface when I try to remove the inb family: 1. The ide-io-std.c file sets up the default_tp_ops, which we override in our IDE driver, but we still have to build this file, and it contains calls to inb, etc. 2. The usb pci quirks file contains some quirks that use inb and outb, and since we support PCI, we have to try to compile this file. 3. CONFIG_DEVPORT defaults to yes, unless you're M68K. None of this seems to depend on CONFIG_HAS_IOPORT. Our PCI driver supports IOMEM read/write, but not IOPORT. Perhaps something like CONFIG_ARCH_PCI_HAS_NO_IOPORT or some such, and then we can disable all of the above things if that config option is present (and CONFIG_HAS_IOPORT is false also?). For now, I'll just leave the inb/outb implementation as panic() calls. -- Chris Metcalf, Tilera Corp. http://www.tilera.com -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html