diff options
author | Mike Miller <mike.miller@hp.com> | 2011-10-13 12:44:06 -0400 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2011-10-20 11:16:20 -0400 |
commit | fba63097b8614a4a158226c02eec0318f41cd24f (patch) | |
tree | 40ea0cbd92ee73231086ba74dc134a8fd650eccb /drivers/scsi/hpsa.c | |
parent | 2d63673b4d469cf2ddba309a916090b54e31cc35 (diff) |
[SCSI] hpsa: change confusing message to be more clear
The following warning message may be confusing to some users:
dev_warn(&pdev->dev, "Controller claims that "
"'Bit 2 doorbell reset' is "
"supported, but not 'bit 5 doorbell reset'. "
"Firmware update is recommended.\n");
Most users don't know or care what bit we may be hitting. Also change
"recommended" to "required."
Signed-off-by: Mike Miller <mike.miller@hp.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/hpsa.c')
-rw-r--r-- | drivers/scsi/hpsa.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index b200b736b000..9825ecf34957 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c | |||
@@ -3438,10 +3438,8 @@ static __devinit int hpsa_kdump_hard_reset_controller(struct pci_dev *pdev) | |||
3438 | } else { | 3438 | } else { |
3439 | use_doorbell = misc_fw_support & MISC_FW_DOORBELL_RESET; | 3439 | use_doorbell = misc_fw_support & MISC_FW_DOORBELL_RESET; |
3440 | if (use_doorbell) { | 3440 | if (use_doorbell) { |
3441 | dev_warn(&pdev->dev, "Controller claims that " | 3441 | dev_warn(&pdev->dev, "Soft reset not supported. " |
3442 | "'Bit 2 doorbell reset' is " | 3442 | "Firmware update is required.\n"); |
3443 | "supported, but not 'bit 5 doorbell reset'. " | ||
3444 | "Firmware update is recommended.\n"); | ||
3445 | rc = -ENOTSUPP; /* try soft reset */ | 3443 | rc = -ENOTSUPP; /* try soft reset */ |
3446 | goto unmap_cfgtable; | 3444 | goto unmap_cfgtable; |
3447 | } | 3445 | } |