On 2020-09-29 10:01, Mike Christie wrote: > +static bool scsi_cmd_retry_allowed(struct scsi_cmnd *cmd) > +{ > + if (cmd->allowed == SCSI_CMD_RETRIES_NO_LIMIT) > + return true; > + > + return (++cmd->retries <= cmd->allowed); > +} Did checkpatch complain about the parentheses in the above return statement? Anyway: Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>