diff options
| author | Stephen M. Cameron <scameron@beardog.cce.hp.com> | 2010-02-04 09:41:44 -0500 |
|---|---|---|
| committer | James Bottomley <James.Bottomley@suse.de> | 2010-02-17 14:17:22 -0500 |
| commit | b2ed4f79194e06766327ae581b063f8747d94ea9 (patch) | |
| tree | 742113d677aad7a01482bd753d3bd6c66f9d7798 /drivers/scsi/hpsa.c | |
| parent | 82a72c0a12326a56a323093297e2bad29fe6c29d (diff) | |
[SCSI] hpsa: Use BUG_ON instead of an if statement.
Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/hpsa.c')
| -rw-r--r-- | drivers/scsi/hpsa.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index ee9db5ee3cb1..2c256e31b107 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c | |||
| @@ -628,8 +628,7 @@ static void hpsa_scsi_remove_entry(struct ctlr_info *h, int hostno, int entry, | |||
| 628 | int i; | 628 | int i; |
| 629 | struct hpsa_scsi_dev_t *sd; | 629 | struct hpsa_scsi_dev_t *sd; |
| 630 | 630 | ||
| 631 | if (entry < 0 || entry >= HPSA_MAX_SCSI_DEVS_PER_HBA) | 631 | BUG_ON(entry < 0 || entry >= HPSA_MAX_SCSI_DEVS_PER_HBA); |
| 632 | BUG(); | ||
| 633 | 632 | ||
| 634 | sd = h->dev[entry]; | 633 | sd = h->dev[entry]; |
| 635 | removed[*nremoved] = h->dev[entry]; | 634 | removed[*nremoved] = h->dev[entry]; |