[lkml]   [2010]   [May]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] arch/tile: new multi-core architecture for Linux
    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.

     \ /
      Last update: 2010-05-28 18:47    [W:0.029 / U:86.036 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site