Skip to content

Commit

Permalink
EISA: Use dev_printk() when possible
Browse files Browse the repository at this point in the history
Use dev_printk() when possible to make messages more useful.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
  • Loading branch information
bjorn-helgaas committed Apr 15, 2013
1 parent f6161aa commit 4cf9f24
Show file tree
Hide file tree
Showing 2 changed files with 30 additions and 38 deletions.
63 changes: 28 additions & 35 deletions drivers/eisa/eisa-bus.c
Original file line number Diff line number Diff line change
Expand Up @@ -314,22 +314,22 @@ static int __init eisa_probe(struct eisa_root_device *root)
{
int i, c;
struct eisa_device *edev;
char *enabled_str;

printk(KERN_INFO "EISA: Probing bus %d at %s\n",
root->bus_nr, dev_name(root->dev));
dev_info(root->dev, "Probing EISA bus %d\n", root->bus_nr);

/* First try to get hold of slot 0. If there is no device
* here, simply fail, unless root->force_probe is set. */

edev = kzalloc(sizeof(*edev), GFP_KERNEL);
if (!edev) {
printk(KERN_ERR "EISA: Couldn't allocate mainboard slot\n");
dev_err(root->dev, "EISA: Couldn't allocate mainboard slot\n");
return -ENOMEM;
}

if (eisa_request_resources(root, edev, 0)) {
printk(KERN_WARNING \
"EISA: Cannot allocate resource for mainboard\n");
dev_warn(root->dev,
"EISA: Cannot allocate resource for mainboard\n");
kfree(edev);
if (!root->force_probe)
return -EBUSY;
Expand All @@ -344,11 +344,11 @@ static int __init eisa_probe(struct eisa_root_device *root)
goto force_probe;
}

printk(KERN_INFO "EISA: Mainboard %s detected.\n", edev->id.sig);
dev_info(&edev->dev, "EISA: Mainboard %s detected\n", edev->id.sig);

if (eisa_register_device(edev)) {
printk(KERN_ERR "EISA: Failed to register %s\n",
edev->id.sig);
dev_err(&edev->dev, "EISA: Failed to register %s\n",
edev->id.sig);
eisa_release_resources(edev);
kfree(edev);
}
Expand All @@ -358,14 +358,15 @@ static int __init eisa_probe(struct eisa_root_device *root)
for (c = 0, i = 1; i <= root->slots; i++) {
edev = kzalloc(sizeof(*edev), GFP_KERNEL);
if (!edev) {
printk(KERN_ERR "EISA: Out of memory for slot %d\n", i);
dev_err(root->dev, "EISA: Out of memory for slot %d\n",
i);
continue;
}

if (eisa_request_resources(root, edev, i)) {
printk(KERN_WARNING \
"Cannot allocate resource for EISA slot %d\n",
i);
dev_warn(root->dev,
"Cannot allocate resource for EISA slot %d\n",
i);
kfree(edev);
continue;
}
Expand All @@ -375,38 +376,30 @@ static int __init eisa_probe(struct eisa_root_device *root)
kfree(edev);
continue;
}

printk(KERN_INFO "EISA: slot %d : %s detected",
i, edev->id.sig);

switch (edev->state) {
case EISA_CONFIG_ENABLED | EISA_CONFIG_FORCED:
printk(" (forced enabled)");
break;

case EISA_CONFIG_FORCED:
printk(" (forced disabled)");
break;

case 0:
printk(" (disabled)");
break;
}

printk (".\n");

if (edev->state == (EISA_CONFIG_ENABLED | EISA_CONFIG_FORCED))
enabled_str = " (forced enabled)";
else if (edev->state == EISA_CONFIG_FORCED)
enabled_str = " (forced disabled)";
else if (edev->state == 0)
enabled_str = " (disabled)";
else
enabled_str = "";

dev_info(&edev->dev, "EISA: slot %d: %s detected%s\n", i,
edev->id.sig, enabled_str);

c++;

if (eisa_register_device(edev)) {
printk(KERN_ERR "EISA: Failed to register %s\n",
edev->id.sig);
dev_err(&edev->dev, "EISA: Failed to register %s\n",
edev->id.sig);
eisa_release_resources(edev);
kfree(edev);
}
}

printk(KERN_INFO "EISA: Detected %d card%s.\n", c, c == 1 ? "" : "s");

dev_info(root->dev, "EISA: Detected %d card%s\n", c, c == 1 ? "" : "s");
return 0;
}

Expand Down
5 changes: 2 additions & 3 deletions drivers/eisa/pci_eisa.c
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,7 @@ static int __init pci_eisa_init(struct pci_dev *pdev,
int rc;

if ((rc = pci_enable_device (pdev))) {
printk (KERN_ERR "pci_eisa : Could not enable device %s\n",
pci_name(pdev));
dev_err(&pdev->dev, "Could not enable device\n");
return rc;
}

Expand All @@ -38,7 +37,7 @@ static int __init pci_eisa_init(struct pci_dev *pdev,
dev_set_drvdata(pci_eisa_root.dev, &pci_eisa_root);

if (eisa_root_register (&pci_eisa_root)) {
printk (KERN_ERR "pci_eisa : Could not register EISA root\n");
dev_err(&pdev->dev, "Could not register EISA root\n");
return -1;
}

Expand Down

0 comments on commit 4cf9f24

Please sign in to comment.