Commit ec26985b authored by Naga Chumbalkar's avatar Naga Chumbalkar Committed by Wim Van Sebroeck

[WATCHDOG] iTCO_wdt: clean up probe(), modify err msg

It's possible that the platform is not allowing reboot via TCO timer
expiration.

Also, differentiate between not finding a chipset that has TCO, and the case
where TCO is present but the driver fails to initialize for some reason.
Signed-off-by: default avatarNaga Chumbalkar <nagananda.chumbalkar@hp.com>
Signed-off-by: default avatarWim Van Sebroeck <wim@iguana.be>
parent f538ed9e
...@@ -698,7 +698,7 @@ static int __devinit iTCO_wdt_init(struct pci_dev *pdev, ...@@ -698,7 +698,7 @@ static int __devinit iTCO_wdt_init(struct pci_dev *pdev,
if (iTCO_wdt_private.iTCO_version == 2) { if (iTCO_wdt_private.iTCO_version == 2) {
pci_read_config_dword(pdev, 0xf0, &base_address); pci_read_config_dword(pdev, 0xf0, &base_address);
if ((base_address & 1) == 0) { if ((base_address & 1) == 0) {
printk(KERN_ERR PFX "RCBA is disabled by harddware\n"); printk(KERN_ERR PFX "RCBA is disabled by hardware\n");
ret = -ENODEV; ret = -ENODEV;
goto out; goto out;
} }
...@@ -708,8 +708,8 @@ static int __devinit iTCO_wdt_init(struct pci_dev *pdev, ...@@ -708,8 +708,8 @@ static int __devinit iTCO_wdt_init(struct pci_dev *pdev,
/* Check chipset's NO_REBOOT bit */ /* Check chipset's NO_REBOOT bit */
if (iTCO_wdt_unset_NO_REBOOT_bit() && iTCO_vendor_check_noreboot_on()) { if (iTCO_wdt_unset_NO_REBOOT_bit() && iTCO_vendor_check_noreboot_on()) {
printk(KERN_ERR PFX "failed to reset NO_REBOOT flag, " printk(KERN_INFO PFX "unable to reset NO_REBOOT flag, "
"reboot disabled by hardware\n"); "platform may have disabled it\n");
ret = -ENODEV; /* Cannot reset NO_REBOOT bit */ ret = -ENODEV; /* Cannot reset NO_REBOOT bit */
goto out_unmap; goto out_unmap;
} }
...@@ -805,6 +805,7 @@ static void __devexit iTCO_wdt_cleanup(void) ...@@ -805,6 +805,7 @@ static void __devexit iTCO_wdt_cleanup(void)
static int __devinit iTCO_wdt_probe(struct platform_device *dev) static int __devinit iTCO_wdt_probe(struct platform_device *dev)
{ {
int ret = -ENODEV;
int found = 0; int found = 0;
struct pci_dev *pdev = NULL; struct pci_dev *pdev = NULL;
const struct pci_device_id *ent; const struct pci_device_id *ent;
...@@ -814,19 +815,17 @@ static int __devinit iTCO_wdt_probe(struct platform_device *dev) ...@@ -814,19 +815,17 @@ static int __devinit iTCO_wdt_probe(struct platform_device *dev)
for_each_pci_dev(pdev) { for_each_pci_dev(pdev) {
ent = pci_match_id(iTCO_wdt_pci_tbl, pdev); ent = pci_match_id(iTCO_wdt_pci_tbl, pdev);
if (ent) { if (ent) {
if (!(iTCO_wdt_init(pdev, ent, dev))) { found++;
found++; ret = iTCO_wdt_init(pdev, ent, dev);
if (!ret)
break; break;
}
} }
} }
if (!found) { if (!found)
printk(KERN_INFO PFX "No card detected\n"); printk(KERN_INFO PFX "No card detected\n");
return -ENODEV;
}
return 0; return ret;
} }
static int __devexit iTCO_wdt_remove(struct platform_device *dev) static int __devexit iTCO_wdt_remove(struct platform_device *dev)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment