[SCSI] hpsa: avoid unwanted promotion from unsigned to signed for raid level index
authorStephen M. Cameron <scameron@beardog.cce.hp.com>
Thu, 4 Feb 2010 14:41:38 +0000 (08:41 -0600)
committerJames Bottomley <James.Bottomley@suse.de>
Wed, 17 Feb 2010 19:17:09 +0000 (13:17 -0600)
Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/hpsa.c

index 0f4a1f3..ee9db5e 100644 (file)
@@ -431,7 +431,7 @@ static ssize_t raid_level_show(struct device *dev,
             struct device_attribute *attr, char *buf)
 {
        ssize_t l = 0;
-       int rlevel;
+       unsigned char rlevel;
        struct ctlr_info *h;
        struct scsi_device *sdev;
        struct hpsa_scsi_dev_t *hdev;
@@ -455,7 +455,7 @@ static ssize_t raid_level_show(struct device *dev,
 
        rlevel = hdev->raid_level;
        spin_unlock_irqrestore(&h->lock, flags);
-       if (rlevel < 0 || rlevel > RAID_UNKNOWN)
+       if (rlevel > RAID_UNKNOWN)
                rlevel = RAID_UNKNOWN;
        l = snprintf(buf, PAGE_SIZE, "RAID %s\n", raid_label[rlevel]);
        return l;