From 85a3c5691d5c85e32439cd5b60cbc592c0af1551 Mon Sep 17 00:00:00 2001 From: Bart Van Assche Date: Thu, 16 Apr 2015 09:15:36 +0200 Subject: [PATCH] scst_lib: Remove two comment blocks and a trace statement Remove the comment blocks from scst_block_generic_dev_done() and scst_tape_generic_dev_done() since that comment does not apply to these functions. Remove the trace statement from scst_block_generic_dev_done() since it prints values that are not modified by this function. Signed-off-by: Bart Van Assche --- scst/src/scst_lib.c | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/scst/src/scst_lib.c b/scst/src/scst_lib.c index b4f648568..9b99c3e73 100644 --- a/scst/src/scst_lib.c +++ b/scst/src/scst_lib.c @@ -8264,12 +8264,6 @@ int scst_block_generic_dev_done(struct scst_cmd *cmd, TRACE_ENTRY(); - /* - * SCST sets good defaults for cmd->is_send_status and - * cmd->resp_data_len based on cmd->status and cmd->data_direction, - * therefore change them only if necessary - */ - if (unlikely(opcode == READ_CAPACITY)) { if ((status == SAM_STAT_GOOD) || (status == SAM_STAT_CONDITION_MET)) { /* Always keep track of disk capacity */ @@ -8299,9 +8293,6 @@ int scst_block_generic_dev_done(struct scst_cmd *cmd, /* It's all good */ } - TRACE_DBG("cmd->is_send_status=%x, cmd->resp_data_len=%d, " - "res=%d", cmd->is_send_status, cmd->resp_data_len, res); - out: TRACE_EXIT_RES(res); return res; @@ -8323,12 +8314,6 @@ int scst_tape_generic_dev_done(struct scst_cmd *cmd, TRACE_ENTRY(); - /* - * SCST sets good defaults for cmd->is_send_status and - * cmd->resp_data_len based on cmd->status and cmd->data_direction, - * therefore change them only if necessary - */ - if (unlikely(cmd->status != SAM_STAT_GOOD)) goto out;