lkml.org 
[lkml]   [2017]   [Oct]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] scsi: hpsa: simplify hpsa_set_local_logical_count()
On 16/10/17 18:55:11 +0000, Don Brace wrote:
> > -----Original Message-----
> > From: Christos Gkekas [mailto:chris.gekas@gmail.com]
> > Sent: Sunday, October 15, 2017 2:35 PM
> > To: Don Brace <don.brace@microsemi.com>; James E.J. Bottomley
> > <jejb@linux.vnet.ibm.com>; Martin K. Petersen
> > <martin.petersen@oracle.com>; esc.storagedev
> > <esc.storagedev@microsemi.com>; linux-scsi@vger.kernel.org; linux-
> > kernel@vger.kernel.org
> > Cc: Christos Gkekas <chris.gekas@gmail.com>
> > Subject: [PATCH] scsi: hpsa: simplify hpsa_set_local_logical_count()
> >
> > EXTERNAL EMAIL
> >
> >
> > Variable configured_logical_drive_count is defined as u8 and thus the
> > nested if statement always evaluates to true. Remove it and simplify.
> >
> > Signed-off-by: Christos Gkekas <chris.gekas@gmail.com>
> > ---
> > drivers/scsi/hpsa.c | 9 +--------
> > 1 file changed, 1 insertion(+), 8 deletions(-)
> >
> > diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
> > index 9abe810..7d4f139 100644
> > --- a/drivers/scsi/hpsa.c
> > +++ b/drivers/scsi/hpsa.c
> > @@ -4090,14 +4090,7 @@ static int hpsa_set_local_logical_count(struct
> > ctlr_info *h,
> > }
> > memset(id_ctlr, 0, sizeof(*id_ctlr));
> > rc = hpsa_bmic_id_controller(h, id_ctlr, sizeof(*id_ctlr));
> > - if (!rc)
> > - if (id_ctlr->configured_logical_drive_count < 256)
> > - *nlocals = id_ctlr->configured_logical_drive_count;
> > - else
> > - *nlocals = le16_to_cpu(
> > - id_ctlr->extended_logical_unit_count);
> > - else
> > - *nlocals = -1;
> > + *nlocals = rc ? -1 : id_ctlr->configured_logical_drive_count;
> > return rc;
> > }
> >
>
> Thanks for your attention to the hpsa driver.
>
> The actual check should be:
> - if (id_ctlr->configured_logical_drive_count < 255)
> We need to return id_ctlr->extended_logical_unit_count otherwise.
>
> Good catch.
>
> Thanks,
> Don Brace
> ESC - Smart Storage
> Microsemi Corporation
>
>
> > --
> > 2.7.4
>

Thanks for reviewing this change, will send an updated patch today.

Thanks,
Christos

\
 
 \ /
  Last update: 2017-10-16 21:08    [W:0.235 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site