Re: [PATCH 1/2] scsi:qla1280: remove unnessary same condition check

[Bart Van Assche]

On Thu, 2018-08-02 at 10:45 +-0800, zhong jiang wrote:
+AD4- doulbe test in a expression is redundant. so just remove one of
+AD4- them.
+AD4- 
+AD4- Signed-off-by: zhong jiang +ADw-zhongjiang+AEA-huawei.com+AD4-
+AD4- ---
+AD4-  drivers/scsi/qla1280.c +AHw- 1 -
+AD4-  1 file changed, 1 deletion(-)
+AD4- 
+AD4- diff --git a/drivers/scsi/qla1280.c b/drivers/scsi/qla1280.c
+AD4- index 390775d..8e6f156 100644
+AD4- --- a/drivers/scsi/qla1280.c
+AD4- +-+-+- b/drivers/scsi/qla1280.c
+AD4- +AEAAQA- -1504,7 +-1504,6 +AEAAQA- static void qla1280+AF8-mailbox+AF8-timeout(struct timer+AF8-list +ACo-t)
+AD4-  	/+ACo- Issue SCSI reset, if we can't reset twice then bus is dead +ACo-/
+AD4-  	for (bus +AD0- 0+ADs- bus +ADw- ha-+AD4-ports+ADs- bus+-+-) +AHs-
+AD4-  		if (+ACE-ha-+AD4-bus+AF8-settings+AFs-bus+AF0-.disable+AF8-scsi+AF8-reset +ACYAJg-
+AD4- -		    qla1280+AF8-bus+AF8-reset(ha, bus) +ACYAJg-
+AD4-  		    qla1280+AF8-bus+AF8-reset(ha, bus))
+AD4-  			ha-+AD4-bus+AF8-settings+AFs-bus+AF0-.scsi+AF8-bus+AF8-dead +AD0- 1+ADs-
+AD4-  	+AH0-

Same question here: does qla1280+AF8-bus+AF8-reset() have any side effects?

Bart.