diff options
| author | Prarit Bhargava <prarit@redhat.com> | 2010-10-27 15:50:22 +0200 |
|---|---|---|
| committer | Wim Van Sebroeck <wim@iguana.be> | 2010-10-29 00:34:37 +0200 |
| commit | ad1d3a26cdb9a0eaa0bf8351a000df0f256b0baa (patch) | |
| tree | 5100819a67970274c4d3f2b928f05c32d51811b2 /drivers/watchdog/mv64x60_wdt.c | |
| parent | watchdog: add support for Broadcom BCM63xx built-in watchdog (diff) | |
| download | linux-ad1d3a26cdb9a0eaa0bf8351a000df0f256b0baa.tar.xz linux-ad1d3a26cdb9a0eaa0bf8351a000df0f256b0baa.zip | |
watchdog: iTCO_wdt.c: remove extra pci_dev_put()'s from init code
The iTCO_wdt driver erroneously releases the pci_dev, and causes PCI hotremove
to fail because of an incorrect usage count.
The probe for this driver does a for_each_pci_dev() which gets a reference for
a pci_dev when iTCO_wdt_init() is successful. The for_each_pci_dev() loop
puts a reference for a pci_dev when iTCO_wdt_init() fails, so the
iTCO_wdt_init() does not need to do any pci_dev_put()'s.
The only pci_dev_put() that is required is in the iTCO_wdt_cleanup() function.
Signed-off-by: Prarit Bhargava <prarit@redhat.com>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Diffstat (limited to 'drivers/watchdog/mv64x60_wdt.c')
0 files changed, 0 insertions, 0 deletions