[PATCH] PCI: 'is_enabled' flag should be set/cleared when the device is actually enabled/disabled
I think 'is_enabled' flag in pci_dev structure should be set/cleared when the device actually enabled/disabled. Especially about pci_enable_device(), it can be failed. By this change, we will also get the possibility of refering 'is_enabled' flag from the functions called through pci_enable_device()/pci_disable_device(). Signed-off-by: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
parent
8800cea620
commit
ceb43744cd
1 changed files with 3 additions and 4 deletions
|
@ -398,10 +398,10 @@ pci_enable_device(struct pci_dev *dev)
|
||||||
{
|
{
|
||||||
int err;
|
int err;
|
||||||
|
|
||||||
dev->is_enabled = 1;
|
|
||||||
if ((err = pci_enable_device_bars(dev, (1 << PCI_NUM_RESOURCES) - 1)))
|
if ((err = pci_enable_device_bars(dev, (1 << PCI_NUM_RESOURCES) - 1)))
|
||||||
return err;
|
return err;
|
||||||
pci_fixup_device(pci_fixup_enable, dev);
|
pci_fixup_device(pci_fixup_enable, dev);
|
||||||
|
dev->is_enabled = 1;
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -427,16 +427,15 @@ pci_disable_device(struct pci_dev *dev)
|
||||||
{
|
{
|
||||||
u16 pci_command;
|
u16 pci_command;
|
||||||
|
|
||||||
dev->is_enabled = 0;
|
|
||||||
dev->is_busmaster = 0;
|
|
||||||
|
|
||||||
pci_read_config_word(dev, PCI_COMMAND, &pci_command);
|
pci_read_config_word(dev, PCI_COMMAND, &pci_command);
|
||||||
if (pci_command & PCI_COMMAND_MASTER) {
|
if (pci_command & PCI_COMMAND_MASTER) {
|
||||||
pci_command &= ~PCI_COMMAND_MASTER;
|
pci_command &= ~PCI_COMMAND_MASTER;
|
||||||
pci_write_config_word(dev, PCI_COMMAND, pci_command);
|
pci_write_config_word(dev, PCI_COMMAND, pci_command);
|
||||||
}
|
}
|
||||||
|
dev->is_busmaster = 0;
|
||||||
|
|
||||||
pcibios_disable_device(dev);
|
pcibios_disable_device(dev);
|
||||||
|
dev->is_enabled = 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
Loading…
Reference in a new issue