diff mbox

[2/8] pci/msi: Use list_first_entry()

Message ID 4A7A40A4.8090509@jp.fujitsu.com (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Hidetoshi Seto Aug. 6, 2009, 2:32 a.m. UTC 
use list_first_entry() instead of list_entry().

Signed-off-by: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
---
 drivers/pci/msi.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Matthew Wilcox Aug. 7, 2009, 5:20 p.m. UTC | #1 
On Thu, Aug 06, 2009 at 11:32:04AM +0900, Hidetoshi Seto wrote:
> use list_first_entry() instead of list_entry().
> 
> Signed-off-by: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>

Reviewed-by: Matthew Wilcox <willy@linux.intel.com>
diff mbox

Patch

diff --git a/drivers/pci/msi.c b/drivers/pci/msi.c
index a7b72f6..bdd2ca9 100644
--- a/drivers/pci/msi.c
+++ b/drivers/pci/msi.c
@@ -324,7 +324,7 @@  static void __pci_restore_msix_state(struct pci_dev *dev)
 	if (!dev->msix_enabled)
 		return;
 	BUG_ON(list_empty(&dev->msi_list));
-	entry = list_entry(dev->msi_list.next, struct msi_desc, list);
+	entry = list_first_entry(&dev->msi_list, struct msi_desc, list);
 	pos = entry->msi_attrib.pos;
 	pci_read_config_word(dev, pos + PCI_MSIX_FLAGS, &control);