ide-cd: fix intendation in cdrom_decode_status()
authorBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Wed, 8 Apr 2009 12:12:54 +0000 (14:12 +0200)
committerBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Wed, 8 Apr 2009 12:12:54 +0000 (14:12 +0200)
Based on earlier work by Borislav Petkov.

Fix intendation in cdrom_decode_status(), no real code changes.

While at it:
- beautify comments

There should be no functional changes caused by this patch.

Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Signed-off-by: Borislav Petkov <petkovbb@gmail.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
drivers/ide/ide-cd.c

index ffbaa6d578ff6ca162cba0ebe91ecfea9ba80013..3ce1eafef5e46d405edba7ea0fe0ff3c384ceaa6 100644 (file)
@@ -311,7 +311,7 @@ static int cdrom_decode_status(ide_drive_t *drive, u8 stat)
 {
        ide_hwif_t *hwif = drive->hwif;
        struct request *rq = hwif->rq;
-       int err, sense_key;
+       int err, sense_key, do_end_request = 0;
        u8 quiet = rq->cmd_flags & REQ_QUIET;
 
        /* get the IDE error register */
@@ -330,117 +330,108 @@ static int cdrom_decode_status(ide_drive_t *drive, u8 stat)
                 */
                rq->cmd_flags |= REQ_FAILED;
                return 2;
-       } else {
-               int do_end_request = 0;
-
-               /*
-                * if we have an error, pass back CHECK_CONDITION as the
-                * scsi status byte
-                */
-               if (blk_pc_request(rq) && !rq->errors)
-                       rq->errors = SAM_STAT_CHECK_CONDITION;
+       }
 
-               if (blk_noretry_request(rq))
-                       do_end_request = 1;
+       /* if we have an error, pass CHECK_CONDITION as the SCSI status byte */
+       if (blk_pc_request(rq) && !rq->errors)
+               rq->errors = SAM_STAT_CHECK_CONDITION;
 
-               switch (sense_key) {
-               case NOT_READY:
-                       if (blk_fs_request(rq) == 0 ||
-                           rq_data_dir(rq) == READ) {
-                               cdrom_saw_media_change(drive);
+       if (blk_noretry_request(rq))
+               do_end_request = 1;
 
-                               if (blk_fs_request(rq) && !quiet)
-                                       printk(KERN_ERR PFX "%s: tray open\n",
-                                               drive->name);
-                       } else {
-                               if (ide_cd_breathe(drive, rq))
-                                       return 1;
-                       }
-                       do_end_request = 1;
-                       break;
-               case UNIT_ATTENTION:
+       switch (sense_key) {
+       case NOT_READY:
+               if (blk_fs_request(rq) == 0 || rq_data_dir(rq) == READ) {
                        cdrom_saw_media_change(drive);
 
-                       if (blk_fs_request(rq) == 0)
-                               return 0;
-                       /*
-                        * Arrange to retry the request but be sure to give up
-                        * if we've retried too many times.
-                        */
-                       if (++rq->errors > ERROR_MAX)
-                               do_end_request = 1;
-                       break;
-               case ILLEGAL_REQUEST:
-                       /*
-                        * Don't print error message for this condition--
-                        * SFF8090i indicates that 5/24/00 is the correct
-                        * response to a request to close the tray if the
-                        * drive doesn't have that capability.
-                        * cdrom_log_sense() knows this!
-                        */
-                       if (rq->cmd[0] == GPCMD_START_STOP_UNIT)
-                               break;
-                       /* fall-through */
-               case DATA_PROTECT:
-                       /*
-                        * No point in retrying after an illegal request or data
-                        * protect error.
-                        */
-                       if (!quiet)
-                               ide_dump_status(drive, "command error", stat);
-                       do_end_request = 1;
-                       break;
-               case MEDIUM_ERROR:
-                       /*
-                        * No point in re-trying a zillion times on a bad
-                        * sector. If we got here the error is not correctable.
-                        */
-                       if (!quiet)
-                               ide_dump_status(drive, "media error "
-                                               "(bad sector)", stat);
-                       do_end_request = 1;
-                       break;
-               case BLANK_CHECK:
-                       /* disk appears blank ?? */
-                       if (!quiet)
-                               ide_dump_status(drive, "media error (blank)",
-                                               stat);
-                       do_end_request = 1;
-                       break;
-               default:
-                       if (blk_fs_request(rq) == 0)
-                               break;
-                       if (err & ~ATA_ABORTED) {
-                               /* go to the default handler for other errors */
-                               ide_error(drive, "cdrom_decode_status", stat);
+                       if (blk_fs_request(rq) && !quiet)
+                               printk(KERN_ERR PFX "%s: tray open\n",
+                                       drive->name);
+               } else {
+                       if (ide_cd_breathe(drive, rq))
                                return 1;
-                       } else if (++rq->errors > ERROR_MAX)
-                               /* we've racked up too many retries, abort */
-                               do_end_request = 1;
                }
+               do_end_request = 1;
+               break;
+       case UNIT_ATTENTION:
+               cdrom_saw_media_change(drive);
 
-               if (blk_fs_request(rq) == 0) {
-                       rq->cmd_flags |= REQ_FAILED;
-                       do_end_request = 1;
-               }
+               if (blk_fs_request(rq) == 0)
+                       return 0;
 
                /*
-                * End a request through request sense analysis when we have
-                * sense data. We need this in order to perform end of media
-                * processing.
+                * Arrange to retry the request but be sure to give up if we've
+                * retried too many times.
                 */
-               if (do_end_request)
-                       goto end_request;
-
+               if (++rq->errors > ERROR_MAX)
+                       do_end_request = 1;
+               break;
+       case ILLEGAL_REQUEST:
+               /*
+                * Don't print error message for this condition -- SFF8090i
+                * indicates that 5/24/00 is the correct response to a request
+                * to close the tray if the drive doesn't have that capability.
+                *
+                * cdrom_log_sense() knows this!
+                */
+               if (rq->cmd[0] == GPCMD_START_STOP_UNIT)
+                       break;
+               /* fall-through */
+       case DATA_PROTECT:
                /*
-                * If we got a CHECK_CONDITION status, queue
-                * a request sense command.
+                * No point in retrying after an illegal request or data
+                * protect error.
                 */
-               if (stat & ATA_ERR)
-                       cdrom_queue_request_sense(drive, NULL, NULL);
-               return 1;
+               if (!quiet)
+                       ide_dump_status(drive, "command error", stat);
+               do_end_request = 1;
+               break;
+       case MEDIUM_ERROR:
+               /*
+                * No point in re-trying a zillion times on a bad sector.
+                * If we got here the error is not correctable.
+                */
+               if (!quiet)
+                       ide_dump_status(drive, "media error "
+                                       "(bad sector)", stat);
+               do_end_request = 1;
+               break;
+       case BLANK_CHECK:
+               /* disk appears blank? */
+               if (!quiet)
+                       ide_dump_status(drive, "media error (blank)",
+                                       stat);
+               do_end_request = 1;
+               break;
+       default:
+               if (blk_fs_request(rq) == 0)
+                       break;
+               if (err & ~ATA_ABORTED) {
+                       /* go to the default handler for other errors */
+                       ide_error(drive, "cdrom_decode_status", stat);
+                       return 1;
+               } else if (++rq->errors > ERROR_MAX)
+                       /* we've racked up too many retries, abort */
+                       do_end_request = 1;
        }
 
+       if (blk_fs_request(rq) == 0) {
+               rq->cmd_flags |= REQ_FAILED;
+               do_end_request = 1;
+       }
+
+       /*
+        * End a request through request sense analysis when we have sense data.
+        * We need this in order to perform end of media processing.
+        */
+       if (do_end_request)
+               goto end_request;
+
+       /* if we got a CHECK_CONDITION status, queue a request sense command */
+       if (stat & ATA_ERR)
+               cdrom_queue_request_sense(drive, NULL, NULL);
+       return 1;
+
 end_request:
        if (stat & ATA_ERR) {
                struct request_queue *q = drive->queue;