On Thu, 2018-07-05 at 13:01 +0200, Johannes Thumshirn wrote: > - cmd->result = ScsiResult(DID_OK, S_CHECK_COND); > + cmd->result = DID_OK << 16 | S_CHECK_COND; Since DID_OK == 0, do we really need "DID_OK << 16 |" here? Thanks, Bart.
On Thu, 2018-07-05 at 13:01 +0200, Johannes Thumshirn wrote: > - cmd->result = ScsiResult(DID_OK, S_CHECK_COND); > + cmd->result = DID_OK << 16 | S_CHECK_COND; Since DID_OK == 0, do we really need "DID_OK << 16 |" here? Thanks, Bart.