diff options
| author | Bjorn Helgaas <bhelgaas@google.com> | 2013-04-15 14:33:42 -0600 | 
|---|---|---|
| committer | Bjorn Helgaas <bhelgaas@google.com> | 2013-04-15 14:33:42 -0600 | 
| commit | 4cf9f24db3954ac5802de24c826b7ebed27760c4 (patch) | |
| tree | 4b5c10c284a548c53d5ab6d646fed951fdc3469d /drivers/eisa | |
| parent | f6161aa153581da4a3867a2d1a7caf4be19b6ec9 (diff) | |
| download | linux-4cf9f24db3954ac5802de24c826b7ebed27760c4.tar.bz2 | |
EISA: Use dev_printk() when possible
Use dev_printk() when possible to make messages more useful.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'drivers/eisa')
| -rw-r--r-- | drivers/eisa/eisa-bus.c | 63 | ||||
| -rw-r--r-- | drivers/eisa/pci_eisa.c | 5 | 
2 files changed, 30 insertions, 38 deletions
| diff --git a/drivers/eisa/eisa-bus.c b/drivers/eisa/eisa-bus.c index 806c77bfd434..0b06df44ed54 100644 --- a/drivers/eisa/eisa-bus.c +++ b/drivers/eisa/eisa-bus.c @@ -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; @@ -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);  	} @@ -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;  		} @@ -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;  } diff --git a/drivers/eisa/pci_eisa.c b/drivers/eisa/pci_eisa.c index cdae207028a7..0d42ea791060 100644 --- a/drivers/eisa/pci_eisa.c +++ b/drivers/eisa/pci_eisa.c @@ -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;  	} @@ -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;  	} |