lkml.org 
[lkml]   [2006]   [Mar]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 0/4] PCI legacy I/O port free driver (take4)
    On Thu, Mar 02, 2006 at 03:50:57PM +0000, Russell King wrote:
    > I've been wondering whether this "no_ioport" flag is the correct approach,
    > or whether it's adding to complexity when it isn't really required.

    I think it's the simplest solution to allowing a driver
    to indicate which resources it wants to use. It solves
    the problem of I/O Port resource allocation sufficiently
    well.

    > In the non-Intel world, the kernel itself sets up the PCI bus mappings,

    mappings, yes. But many of the architectures depend on (or assume)
    firmware will assign appropriate resources. The problem is firmware
    runs out of I/O Port resources.

    > and any IO bars which it can't satisfy might also need to be gracefully
    > handled. Currently, we just go 'printk("whoops, didn't allocate
    > resource")' and leave the BAR containing whatever random junk it
    > contained before, along with the resource containing whatever random
    > junk pci_bus_alloc_resource() decided to leave in it.
    >
    > In such cases, I would suggest that the method of signalling that IO
    > should not be used is to have the IO resource structures cleared out -
    > if the IO resources aren't valid, they should not contain something
    > which could be interpreted as valid.

    You want the arch PCI support to clobber the I/O port space resources?
    How will the arch PCI support know that a particular device's driver
    only uses MMIO resources?

    > Maybe something like this should be done for the "legacy IO port" case
    > as well?

    Several people have already agreed the driver needs to indicate which
    resource it wants to work with. I don't see how the arch PCI support
    can provide that "knowledge".

    hth,
    grant

    >
    > --
    > Russell King
    > Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
    > maintainer of: 2.6 Serial core
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-03-02 18:16    [W:5.196 / U:20.712 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site