Skip to content

Commit

Permalink
PCI: cleanup MSI code
Browse files Browse the repository at this point in the history
Cleanup MSI code as follows:

 - fix some types
 - fix strange local variable definition
 - delete unnecessary blank line
 - add comment to #endif which is far from corresponding #ifdef

Signed-off-by: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Satoru Takeuchi authored and Greg Kroah-Hartman committed Feb 7, 2007
1 parent 8255cf3 commit c54c187
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
9 changes: 5 additions & 4 deletions drivers/pci/msi.c
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,8 @@ static void msi_set_mask_bit(unsigned int irq, int flag)
switch (entry->msi_attrib.type) {
case PCI_CAP_ID_MSI:
if (entry->msi_attrib.maskbit) {
int pos;
u32 mask_bits;
int pos;
u32 mask_bits;

pos = (long)entry->mask_base;
pci_read_config_dword(entry->dev, pos, &mask_bits);
Expand Down Expand Up @@ -162,6 +162,7 @@ void unmask_msi_irq(unsigned int irq)
}

static int msi_free_irq(struct pci_dev* dev, int irq);

static int msi_init(void)
{
static int status = -ENOMEM;
Expand Down Expand Up @@ -291,7 +292,7 @@ static int msi_lookup_irq(struct pci_dev *dev, int type)
continue;
spin_unlock_irqrestore(&msi_lock, flags);
/* This pre-assigned MSI irq for this device
already exits. Override dev->irq with this irq */
already exists. Override dev->irq with this irq */
dev->irq = irq;
return 0;
}
Expand Down Expand Up @@ -458,7 +459,7 @@ void pci_restore_msix_state(struct pci_dev *dev)
pci_write_config_word(dev, msi_control_reg(pos), save);
enable_msi_mode(dev, pos, PCI_CAP_ID_MSIX);
}
#endif
#endif /* CONFIG_PM */

/**
* msi_capability_init - configure device's MSI capability structure
Expand Down
3 changes: 1 addition & 2 deletions include/linux/msi.h
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@ struct msi_msg {
u32 data; /* 16 bits of msi message data */
};

/* Heper functions */
/* Helper functions */
extern void mask_msi_irq(unsigned int irq);
extern void unmask_msi_irq(unsigned int irq);
extern void read_msi_msg(unsigned int irq, struct msi_msg *msg);

extern void write_msi_msg(unsigned int irq, struct msi_msg *msg);

struct msi_desc {
Expand Down

0 comments on commit c54c187

Please sign in to comment.