On 8/13/21 6:49 AM, John Garry wrote: > - action = /*cmd->tag ? WD719X_CMD_ABORT_TAG : */WD719X_CMD_ABORT; > + action = /*tag ? WD719X_CMD_ABORT_TAG : */WD719X_CMD_ABORT; If this patch series would be reposted, please remove the commented-out code instead of modifying it. Anyway: Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>