]> www.infradead.org Git - users/hch/misc.git/commitdiff
scsi: hpsa: use min()/min_t() to improve code
authorQianfeng Rong <rongqianfeng@vivo.com>
Fri, 15 Aug 2025 12:16:04 +0000 (20:16 +0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 20 Aug 2025 02:25:06 +0000 (22:25 -0400)
Use min()/min_t() to reduce the code in complete_scsi_command() and
hpsa_vpd_page_supported(), and improve readability.

Signed-off-by: Qianfeng Rong <rongqianfeng@vivo.com>
Link: https://lore.kernel.org/r/20250815121609.384914-3-rongqianfeng@vivo.com
Acked-by: Don Brace <don.brace@microchip.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/hpsa.c

index 3bb7a1d9af53a9a535b4bfa34c0d1d13b5918643..8738ad75723543a40896710b3cb4e1cdc35d13a0 100644 (file)
@@ -2662,10 +2662,8 @@ static void complete_scsi_command(struct CommandList *cp)
        case CMD_TARGET_STATUS:
                cmd->result |= ei->ScsiStatus;
                /* copy the sense data */
-               if (SCSI_SENSE_BUFFERSIZE < sizeof(ei->SenseInfo))
-                       sense_data_size = SCSI_SENSE_BUFFERSIZE;
-               else
-                       sense_data_size = sizeof(ei->SenseInfo);
+               sense_data_size = min_t(unsigned long, SCSI_SENSE_BUFFERSIZE,
+                                       sizeof(ei->SenseInfo));
                if (ei->SenseLen < sense_data_size)
                        sense_data_size = ei->SenseLen;
                memcpy(cmd->sense_buffer, ei->SenseInfo, sense_data_size);
@@ -3628,10 +3626,7 @@ static bool hpsa_vpd_page_supported(struct ctlr_info *h,
        if (rc != 0)
                goto exit_unsupported;
        pages = buf[3];
-       if ((pages + HPSA_VPD_HEADER_SZ) <= 255)
-               bufsize = pages + HPSA_VPD_HEADER_SZ;
-       else
-               bufsize = 255;
+       bufsize = min(pages + HPSA_VPD_HEADER_SZ, 255);
 
        /* Get the whole VPD page list */
        rc = hpsa_scsi_do_inquiry(h, scsi3addr,