diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2007-08-28 03:30:42 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.localdomain> | 2007-10-12 14:46:18 -0400 |
commit | cc16cebad086090891a3f39957ec771a2292359b (patch) | |
tree | c39169817529e9349c49a30e14483cc3b60cf4de /drivers | |
parent | 2282f658914e316ca32fd120fded130d1c0e26e4 (diff) |
[SCSI] zfcp: avoid if (whatever) ; constructs.
Avoid "if (whatever) ;" constructs since they seem to confuse people,
even if there is a comment.
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Swen Schillig <swen@vnet.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/s390/scsi/zfcp_erp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/s390/scsi/zfcp_erp.c b/drivers/s390/scsi/zfcp_erp.c index d8cd75ce2d9a..40fa056e2575 100644 --- a/drivers/s390/scsi/zfcp_erp.c +++ b/drivers/s390/scsi/zfcp_erp.c | |||
@@ -1687,8 +1687,8 @@ zfcp_erp_strategy_followup_actions(int action, | |||
1687 | break; | 1687 | break; |
1688 | 1688 | ||
1689 | case ZFCP_ERP_ACTION_REOPEN_UNIT: | 1689 | case ZFCP_ERP_ACTION_REOPEN_UNIT: |
1690 | if (status == ZFCP_ERP_SUCCEEDED) ; /* no further action */ | 1690 | /* Nothing to do if status == ZFCP_ERP_SUCCEEDED */ |
1691 | else | 1691 | if (status != ZFCP_ERP_SUCCEEDED) |
1692 | zfcp_erp_port_reopen_internal(unit->port, 0); | 1692 | zfcp_erp_port_reopen_internal(unit->port, 0); |
1693 | break; | 1693 | break; |
1694 | } | 1694 | } |