diff options
| author | Bart Van Assche <[email protected]> | 2023-05-18 12:31:57 -0700 |
|---|---|---|
| committer | Martin K. Petersen <[email protected]> | 2023-05-31 11:05:34 -0400 |
| commit | 416dace649c45fdbf7a9d62dfc060fdbee9c8bea (patch) | |
| tree | 1d19b7dd8b0045486187a2ff854cc4e55910c55c | |
| parent | f047d1e38bdfda2e9b9bf82ffa761711acacbd69 (diff) | |
scsi: core: Use min() instead of open-coding it
Use min() instead of open-coding it in scsi_normalize_sense().
Reviewed-by: Christoph Hellwig <[email protected]>
Reviewed-by: Benjamin Block <[email protected]>
Cc: Douglas Gilbert <[email protected]>
Signed-off-by: Bart Van Assche <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Martin K. Petersen <[email protected]>
| -rw-r--r-- | drivers/scsi/scsi_common.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/scsi_common.c b/drivers/scsi/scsi_common.c index b7a7a2eea887..9c14fdf61037 100644 --- a/drivers/scsi/scsi_common.c +++ b/drivers/scsi/scsi_common.c @@ -219,8 +219,7 @@ bool scsi_normalize_sense(const u8 *sense_buffer, int sb_len, if (sb_len > 2) sshdr->sense_key = (sense_buffer[2] & 0xf); if (sb_len > 7) { - sb_len = (sb_len < (sense_buffer[7] + 8)) ? - sb_len : (sense_buffer[7] + 8); + sb_len = min(sb_len, sense_buffer[7] + 8); if (sb_len > 12) sshdr->asc = sense_buffer[12]; if (sb_len > 13) |