staging: rts5208: remove unnecessary else
authorRoxana Blaj <roxanagabriela10@gmail.com>
Thu, 2 Oct 2014 15:20:35 +0000 (18:20 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 2 Oct 2014 17:07:04 +0000 (10:07 -0700)
This fixes the checkpatch.pl warning:
WARNING: else is not generally useful after a break or return

Signed-off-by: Roxana Blaj <roxanagabriela10@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rts5208/rtsx_scsi.c

index bd033d02df398f6e2ea1b99bbef049b2451d978b..bbbf7968a0b62c0bdf242ec58ceb874284af9506 100644 (file)
@@ -586,10 +586,9 @@ static int start_stop_unit(struct scsi_cmnd *srb, struct rtsx_chip *chip)
        case LOAD_MEDIUM:
                if (check_card_ready(chip, lun)) {
                        return TRANSPORT_GOOD;
-               } else {
-                       set_sense_type(chip, lun, SENSE_TYPE_MEDIA_NOT_PRESENT);
-                       TRACE_RET(chip, TRANSPORT_FAILED);
                }
+               set_sense_type(chip, lun, SENSE_TYPE_MEDIA_NOT_PRESENT);
+               TRACE_RET(chip, TRANSPORT_FAILED);
 
                break;
        }