lkml.org 
[lkml]   [2009]   [Jan]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: libata, devm_*, and MSI ?
Hello, Mark.

Sorry about slacking off. Having way too much fun mucking around
assembly code lately.

Mark Lord wrote:
> I believe that other MSI users might want something similar,
> or perhaps Tejun could extend devres to include a pair
> of suitable functions, devm_enable_msi() and devm_disable_msi().
> Then it would be just automatic for drivers, without any fuss.

devres does handle MSI. Please take a look at pcim_release() in
drivers/pci/pci.c. msi/msix enabledness is tracked by
pdev->msi[x]_enabled and if either one was enabled on a managed PCI
device, it would be disabled on driver detach, so there's no reason to
worry about it.

Also, there is no reason for low level driver to dingle with intx. In
fact, it shouldn't as PCI quirk is supposed to handle all the pretty
quirkiness. The intx code was there before the quirk code was in
place and no one went after them. They should go and probably won't
cause any problem in the process.

Thanks.

--
tejun


\
 
 \ /
  Last update: 2009-01-21 13:01    [W:0.038 / U:0.108 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site