lkml.org 
[lkml]   [2006]   [Feb]   [27]   [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 (take 3)
Grant Grundler wrote:
> On Mon, Feb 27, 2006 at 12:43:09PM -0500, Jeff Garzik wrote:
>
>>This series still leaves a lot to be desired, and creates unnecessary
>>driver churn.
>
>
> This is a pretty small change and is not necessary for every driver.

The latter is decidedly false. The change makes no sense at all unless
you update every conceivable driver that will be used on the target
platform. You will always be patching drivers as users stick new cards
in the target hardware.


>> The better solution is:
>>
>>1) pci_enable_device() enables what it can
>>
>>2) Drivers, as they already do, will fail if they cannot map the desired
>>memory or IO resources that are needed.
>>
>>Thus, the PCI layer needs only to do #1, and existing driver code
>>handles the rest of the situation as one currently expects.
>
>
> If in #1 pci_enable_device() assigns I/O Port resources even though
> the driver doesn't need it, PCI devices which _only_ support I/O Port
> space will get screwed (depending on config). We are trying to avoid that.
> Or do you have another way of avoiding unused resource allocation?

Fix the [firmware | device load order] to allocate I/O ports first to
the hardware that only supports IO port accesses. Problem solved with
zero kernel mods...
Jeff


-
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-02-27 23:25    [from the cache]
©2003-2011 Jasper Spaans