Re: [PATCH] scsi: hisi_sas: add missing break in switch statement

2017-03-29 Thread Martin K. Petersen
Colin King writes: Colin, > It appears that a break in the TRANS_TX_OPEN_CNX_ERR_NO_DESTINATION > case got accidentally removed in an earlier commit, as it stands, the > ts->stat and ts->open_rej_reason are being updated twice for this case > which looks incorrect. Fix this by adding in the mis

Re: [PATCH] scsi: hisi_sas: add missing break in switch statement

2017-03-28 Thread John Garry
On 28/03/2017 12:12, Colin King wrote: From: Colin Ian King It appears that a break in the TRANS_TX_OPEN_CNX_ERR_NO_DESTINATION case got accidentally removed in an earlier commit, as it stands, the ts->stat and ts->open_rej_reason are being updated twice for this case which looks incorrect. Fi

[PATCH] scsi: hisi_sas: add missing break in switch statement

2017-03-28 Thread Colin King
From: Colin Ian King It appears that a break in the TRANS_TX_OPEN_CNX_ERR_NO_DESTINATION case got accidentally removed in an earlier commit, as it stands, the ts->stat and ts->open_rej_reason are being updated twice for this case which looks incorrect. Fix this by adding in the missing break sta