Commit 811e843d authored by David Matlack's avatar David Matlack Committed by Greg Kroah-Hartman

staging: slicoss: clean up use of dev_err

First, don't print pci device information or driver prefixes, this
is already printed by dev_err. Next, don't report error messages
via dev_err when the failing function already reports all errors
via dev_err.
Signed-off-by: default avatarDavid Matlack <matlackdavid@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 17dd4338
...@@ -395,7 +395,7 @@ static int slic_card_download_gbrcv(struct adapter *adapter) ...@@ -395,7 +395,7 @@ static int slic_card_download_gbrcv(struct adapter *adapter)
ret = request_firmware(&fw, file, &adapter->pcidev->dev); ret = request_firmware(&fw, file, &adapter->pcidev->dev);
if (ret) { if (ret) {
dev_err(&adapter->pcidev->dev, dev_err(&adapter->pcidev->dev,
"SLICOSS: Failed to load firmware %s\n", file); "Failed to load firmware %s\n", file);
return ret; return ret;
} }
...@@ -473,7 +473,7 @@ static int slic_card_download(struct adapter *adapter) ...@@ -473,7 +473,7 @@ static int slic_card_download(struct adapter *adapter)
ret = request_firmware(&fw, file, &adapter->pcidev->dev); ret = request_firmware(&fw, file, &adapter->pcidev->dev);
if (ret) { if (ret) {
dev_err(&adapter->pcidev->dev, dev_err(&adapter->pcidev->dev,
"SLICOSS: Failed to load firmware %s\n", file); "Failed to load firmware %s\n", file);
return ret; return ret;
} }
numsects = *(u32 *)(fw->data + index); numsects = *(u32 *)(fw->data + index);
...@@ -2699,13 +2699,8 @@ static int slic_card_init(struct sliccard *card, struct adapter *adapter) ...@@ -2699,13 +2699,8 @@ static int slic_card_init(struct sliccard *card, struct adapter *adapter)
/* Download the microcode */ /* Download the microcode */
status = slic_card_download(adapter); status = slic_card_download(adapter);
if (status)
if (status != 0) {
dev_err(&adapter->pcidev->dev,
"download failed bus %d slot %d\n",
adapter->busnumber, adapter->slotnumber);
return status; return status;
}
if (!card->config_set) { if (!card->config_set) {
peeprom = pci_alloc_consistent(adapter->pcidev, peeprom = pci_alloc_consistent(adapter->pcidev,
...@@ -2717,8 +2712,7 @@ static int slic_card_init(struct sliccard *card, struct adapter *adapter) ...@@ -2717,8 +2712,7 @@ static int slic_card_init(struct sliccard *card, struct adapter *adapter)
if (!peeprom) { if (!peeprom) {
dev_err(&adapter->pcidev->dev, dev_err(&adapter->pcidev->dev,
"eeprom read failed to get memory bus %d slot %d\n", adapter->busnumber, "Failed to allocate DMA memory for EEPROM.\n");
adapter->slotnumber);
return -ENOMEM; return -ENOMEM;
} else { } else {
memset(peeprom, 0, sizeof(struct slic_eeprom)); memset(peeprom, 0, sizeof(struct slic_eeprom));
...@@ -2767,8 +2761,7 @@ static int slic_card_init(struct sliccard *card, struct adapter *adapter) ...@@ -2767,8 +2761,7 @@ static int slic_card_init(struct sliccard *card, struct adapter *adapter)
i++; i++;
if (i > 5000) { if (i > 5000) {
dev_err(&adapter->pcidev->dev, dev_err(&adapter->pcidev->dev,
"%d config data fetch timed out!\n", "Fetch of config data timed out.\n");
adapter->port);
slic_reg64_write(adapter, slic_reg64_write(adapter,
&slic_regs->slic_isp, 0, &slic_regs->slic_isp, 0,
&slic_regs->slic_addr_upper, &slic_regs->slic_addr_upper,
...@@ -2851,19 +2844,16 @@ static int slic_card_init(struct sliccard *card, struct adapter *adapter) ...@@ -2851,19 +2844,16 @@ static int slic_card_init(struct sliccard *card, struct adapter *adapter)
slic_reg64_write(adapter, &slic_regs->slic_isp, 0, slic_reg64_write(adapter, &slic_regs->slic_isp, 0,
&slic_regs->slic_addr_upper, &slic_regs->slic_addr_upper,
0, FLUSH); 0, FLUSH);
dev_err(&adapter->pcidev->dev, dev_err(&adapter->pcidev->dev, "EEPROM invalid.\n");
"unsupported CONFIGURATION EEPROM invalid\n");
return -EINVAL; return -EINVAL;
} }
card->config_set = 1; card->config_set = 1;
} }
if (slic_card_download_gbrcv(adapter)) { status = slic_card_download_gbrcv(adapter);
dev_err(&adapter->pcidev->dev, if (status)
"unable to download GB receive microcode\n"); return status;
return -EINVAL;
}
if (slic_global.dynamic_intagg) if (slic_global.dynamic_intagg)
slic_intagg_set(adapter, 0); slic_intagg_set(adapter, 0);
......
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