Re: [PATCH 4/4] scsi: ncr53c8xx: remove ScsiResult macro

From: Bart Van Assche
Date: Thu Jul 05 2018 - 13:04:13 EST


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.