ide-cd: coding style fixes for VERBOSE_IDE_CD_ERRORS code

* Coding style fixes for VERBOSE_IDE_CD_ERRORS code.

* Add KERN_{ERR,CONT} printk() levels where needed.

This is a preparation for moving this code out of ide-cd.[c,h].
Signed-off-by: default avatarBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
parent 1b1215d5
...@@ -205,28 +205,30 @@ void cdrom_analyze_sense_data(ide_drive_t *drive, ...@@ -205,28 +205,30 @@ void cdrom_analyze_sense_data(ide_drive_t *drive,
const char *s = "bad sense key!"; const char *s = "bad sense key!";
char buf[80]; char buf[80];
printk ("ATAPI device %s:\n", drive->name); printk(KERN_ERR "ATAPI device %s:\n", drive->name);
if (sense->error_code==0x70) if (sense->error_code == 0x70)
printk(" Error: "); printk(KERN_CONT " Error: ");
else if (sense->error_code==0x71) else if (sense->error_code == 0x71)
printk(" Deferred Error: "); printk(" Deferred Error: ");
else if (sense->error_code == 0x7f) else if (sense->error_code == 0x7f)
printk(" Vendor-specific Error: "); printk(KERN_CONT " Vendor-specific Error: ");
else else
printk(" Unknown Error Type: "); printk(KERN_CONT " Unknown Error Type: ");
if (sense->sense_key < ARRAY_SIZE(sense_key_texts)) if (sense->sense_key < ARRAY_SIZE(sense_key_texts))
s = sense_key_texts[sense->sense_key]; s = sense_key_texts[sense->sense_key];
printk("%s -- (Sense key=0x%02x)\n", s, sense->sense_key); printk(KERN_CONT "%s -- (Sense key=0x%02x)\n",
s, sense->sense_key);
if (sense->asc == 0x40) { if (sense->asc == 0x40) {
sprintf(buf, "Diagnostic failure on component 0x%02x", sprintf(buf, "Diagnostic failure on component 0x%02x",
sense->ascq); sense->ascq);
s = buf; s = buf;
} else { } else {
int lo = 0, mid, hi = ARRAY_SIZE(sense_data_texts); int lo = 0, mid, hi = ARRAY_SIZE(sense_data_texts);
unsigned long key = (sense->sense_key << 16); unsigned long key = (sense->sense_key << 16);
key |= (sense->asc << 8); key |= (sense->asc << 8);
if (!(sense->ascq >= 0x80 && sense->ascq <= 0xdd)) if (!(sense->ascq >= 0x80 && sense->ascq <= 0xdd))
key |= sense->ascq; key |= sense->ascq;
...@@ -238,11 +240,10 @@ void cdrom_analyze_sense_data(ide_drive_t *drive, ...@@ -238,11 +240,10 @@ void cdrom_analyze_sense_data(ide_drive_t *drive,
sense_data_texts[mid].asc_ascq == (0xff0000|key)) { sense_data_texts[mid].asc_ascq == (0xff0000|key)) {
s = sense_data_texts[mid].text; s = sense_data_texts[mid].text;
break; break;
} } else if (sense_data_texts[mid].asc_ascq > key)
else if (sense_data_texts[mid].asc_ascq > key)
hi = mid; hi = mid;
else else
lo = mid+1; lo = mid + 1;
} }
} }
...@@ -254,11 +255,10 @@ void cdrom_analyze_sense_data(ide_drive_t *drive, ...@@ -254,11 +255,10 @@ void cdrom_analyze_sense_data(ide_drive_t *drive,
} }
printk(KERN_ERR " %s -- (asc=0x%02x, ascq=0x%02x)\n", printk(KERN_ERR " %s -- (asc=0x%02x, ascq=0x%02x)\n",
s, sense->asc, sense->ascq); s, sense->asc, sense->ascq);
if (failed_command != NULL) { if (failed_command != NULL) {
int lo = 0, mid, hi = ARRAY_SIZE(packet_command_texts);
int lo=0, mid, hi= ARRAY_SIZE(packet_command_texts);
s = NULL; s = NULL;
while (hi > lo) { while (hi > lo) {
...@@ -272,13 +272,15 @@ void cdrom_analyze_sense_data(ide_drive_t *drive, ...@@ -272,13 +272,15 @@ void cdrom_analyze_sense_data(ide_drive_t *drive,
failed_command->cmd[0]) failed_command->cmd[0])
hi = mid; hi = mid;
else else
lo = mid+1; lo = mid + 1;
} }
printk (KERN_ERR " The failed \"%s\" packet command was: \n \"", s); printk(KERN_ERR " The failed \"%s\" packet command "
for (i=0; i<sizeof (failed_command->cmd); i++) "was: \n \"", s);
printk ("%02x ", failed_command->cmd[i]); for (i = 0; i < sizeof(failed_command->cmd); i++)
printk ("\"\n"); printk(KERN_CONT "%02x ",
failed_command->cmd[i]);
printk(KERN_CONT "\"\n");
} }
/* The SKSV bit specifies validity of the sense_key_specific /* The SKSV bit specifies validity of the sense_key_specific
...@@ -288,38 +290,37 @@ void cdrom_analyze_sense_data(ide_drive_t *drive, ...@@ -288,38 +290,37 @@ void cdrom_analyze_sense_data(ide_drive_t *drive,
*/ */
if (sense->sense_key == NOT_READY && (sense->sks[0] & 0x80)) { if (sense->sense_key == NOT_READY && (sense->sks[0] & 0x80)) {
int progress = (sense->sks[1] << 8 | sense->sks[2]) * 100; int progress = (sense->sks[1] << 8 | sense->sks[2]) * 100;
printk(KERN_ERR " Command is %02d%% complete\n", progress / 0xffff);
printk(KERN_ERR " Command is %02d%% complete\n",
progress / 0xffff);
} }
if (sense->sense_key == ILLEGAL_REQUEST && if (sense->sense_key == ILLEGAL_REQUEST &&
(sense->sks[0] & 0x80) != 0) { (sense->sks[0] & 0x80) != 0) {
printk(KERN_ERR " Error in %s byte %d", printk(KERN_ERR " Error in %s byte %d",
(sense->sks[0] & 0x40) != 0 ? (sense->sks[0] & 0x40) != 0 ?
"command packet" : "command data", "command packet" : "command data",
(sense->sks[1] << 8) + sense->sks[2]); (sense->sks[1] << 8) + sense->sks[2]);
if ((sense->sks[0] & 0x40) != 0) if ((sense->sks[0] & 0x40) != 0)
printk (" bit %d", sense->sks[0] & 0x07); printk(KERN_CONT " bit %d",
sense->sks[0] & 0x07);
printk ("\n"); printk(KERN_CONT "\n");
} }
} }
#else /* not VERBOSE_IDE_CD_ERRORS */ #else /* not VERBOSE_IDE_CD_ERRORS */
/* Suppress printing unit attention and `in progress of becoming ready' /* Suppress printing unit attention and `in progress of becoming ready'
errors when we're not being verbose. */ errors when we're not being verbose. */
if (sense->sense_key == UNIT_ATTENTION || if (sense->sense_key == UNIT_ATTENTION ||
(sense->sense_key == NOT_READY && (sense->asc == 4 || (sense->sense_key == NOT_READY && (sense->asc == 4 ||
sense->asc == 0x3a))) sense->asc == 0x3a)))
return; return;
printk(KERN_ERR "%s: error code: 0x%02x sense_key: 0x%02x asc: 0x%02x ascq: 0x%02x\n", printk(KERN_ERR "%s: error code: 0x%02x sense_key: 0x%02x "
drive->name, "asc: 0x%02x ascq: 0x%02x\n",
sense->error_code, sense->sense_key, drive->name, sense->error_code, sense->sense_key,
sense->asc, sense->ascq); sense->asc, sense->ascq);
#endif /* not VERBOSE_IDE_CD_ERRORS */ #endif /* not VERBOSE_IDE_CD_ERRORS */
} }
......
...@@ -159,9 +159,8 @@ struct cdrom_info { ...@@ -159,9 +159,8 @@ struct cdrom_info {
/* This stuff should be in cdrom.h, since it is now generic... */ /* This stuff should be in cdrom.h, since it is now generic... */
#if VERBOSE_IDE_CD_ERRORS #if VERBOSE_IDE_CD_ERRORS
/* The generic packet command opcodes for CD/DVD Logical Units,
/* The generic packet command opcodes for CD/DVD Logical Units, * From Table 57 of the SFF8090 Ver. 3 (Mt. Fuji) draft standard. */
* From Table 57 of the SFF8090 Ver. 3 (Mt. Fuji) draft standard. */
static const struct { static const struct {
unsigned short packet_command; unsigned short packet_command;
const char * const text; const char * const text;
...@@ -187,7 +186,8 @@ static const struct { ...@@ -187,7 +186,8 @@ static const struct {
{ GPCMD_GET_CONFIGURATION, "Get Configuration" }, { GPCMD_GET_CONFIGURATION, "Get Configuration" },
{ GPCMD_PLAY_AUDIO_MSF, "Play Audio MSF" }, { GPCMD_PLAY_AUDIO_MSF, "Play Audio MSF" },
{ GPCMD_PLAYAUDIO_TI, "Play Audio TrackIndex" }, { GPCMD_PLAYAUDIO_TI, "Play Audio TrackIndex" },
{ GPCMD_GET_EVENT_STATUS_NOTIFICATION, "Get Event Status Notification" }, { GPCMD_GET_EVENT_STATUS_NOTIFICATION,
"Get Event Status Notification" },
{ GPCMD_PAUSE_RESUME, "Pause/Resume" }, { GPCMD_PAUSE_RESUME, "Pause/Resume" },
{ GPCMD_STOP_PLAY_SCAN, "Stop Play/Scan" }, { GPCMD_STOP_PLAY_SCAN, "Stop Play/Scan" },
{ GPCMD_READ_DISC_INFO, "Read Disc Info" }, { GPCMD_READ_DISC_INFO, "Read Disc Info" },
...@@ -217,8 +217,6 @@ static const struct { ...@@ -217,8 +217,6 @@ static const struct {
{ GPCMD_READ_CD, "Read CD" }, { GPCMD_READ_CD, "Read CD" },
}; };
/* From Table 303 of the SFF8090 Ver. 3 (Mt. Fuji) draft standard. */ /* From Table 303 of the SFF8090 Ver. 3 (Mt. Fuji) draft standard. */
static const char * const sense_key_texts[16] = { static const char * const sense_key_texts[16] = {
"No sense data", "No sense data",
...@@ -262,16 +260,16 @@ static const struct { ...@@ -262,16 +260,16 @@ static const struct {
{ 0x011802, "Recovered data - the data was auto-reallocated" }, { 0x011802, "Recovered data - the data was auto-reallocated" },
{ 0x011803, "Recovered data with CIRC" }, { 0x011803, "Recovered data with CIRC" },
{ 0x011804, "Recovered data with L-EC" }, { 0x011804, "Recovered data with L-EC" },
{ 0x015d00, { 0x015d00, "Failure prediction threshold exceeded"
"Failure prediction threshold exceeded - Predicted logical unit failure" }, " - Predicted logical unit failure" },
{ 0x015d01, { 0x015d01, "Failure prediction threshold exceeded"
"Failure prediction threshold exceeded - Predicted media failure" }, " - Predicted media failure" },
{ 0x015dff, "Failure prediction threshold exceeded - False" }, { 0x015dff, "Failure prediction threshold exceeded - False" },
{ 0x017301, "Power calibration area almost full" }, { 0x017301, "Power calibration area almost full" },
{ 0x020400, "Logical unit not ready - cause not reportable" }, { 0x020400, "Logical unit not ready - cause not reportable" },
/* Following is misspelled in ATAPI 2.6, _and_ in Mt. Fuji */ /* Following is misspelled in ATAPI 2.6, _and_ in Mt. Fuji */
{ 0x020401, { 0x020401, "Logical unit not ready"
"Logical unit not ready - in progress [sic] of becoming ready" }, " - in progress [sic] of becoming ready" },
{ 0x020402, "Logical unit not ready - initializing command required" }, { 0x020402, "Logical unit not ready - initializing command required" },
{ 0x020403, "Logical unit not ready - manual intervention required" }, { 0x020403, "Logical unit not ready - manual intervention required" },
{ 0x020404, "Logical unit not ready - format in progress" }, { 0x020404, "Logical unit not ready - format in progress" },
...@@ -310,7 +308,6 @@ static const struct { ...@@ -310,7 +308,6 @@ static const struct {
{ 0x037304, "Program memory area / RMA update failure" }, { 0x037304, "Program memory area / RMA update failure" },
{ 0x037305, "Program memory area / RMA is full" }, { 0x037305, "Program memory area / RMA is full" },
{ 0x037306, "Program memory area / RMA is (almost) full" }, { 0x037306, "Program memory area / RMA is (almost) full" },
{ 0x040200, "No seek complete" }, { 0x040200, "No seek complete" },
{ 0x040300, "Write fault" }, { 0x040300, "Write fault" },
{ 0x040900, "Track following error" }, { 0x040900, "Track following error" },
...@@ -344,12 +341,15 @@ static const struct { ...@@ -344,12 +341,15 @@ static const struct {
{ 0x055500, "System resource failure" }, { 0x055500, "System resource failure" },
{ 0x056300, "End of user area encountered on this track" }, { 0x056300, "End of user area encountered on this track" },
{ 0x056400, "Illegal mode for this track or incompatible medium" }, { 0x056400, "Illegal mode for this track or incompatible medium" },
{ 0x056f00, "Copy protection key exchange failure - Authentication failure" }, { 0x056f00, "Copy protection key exchange failure"
" - Authentication failure" },
{ 0x056f01, "Copy protection key exchange failure - Key not present" }, { 0x056f01, "Copy protection key exchange failure - Key not present" },
{ 0x056f02, "Copy protection key exchange failure - Key not established" }, { 0x056f02, "Copy protection key exchange failure"
" - Key not established" },
{ 0x056f03, "Read of scrambled sector without authentication" }, { 0x056f03, "Read of scrambled sector without authentication" },
{ 0x056f04, "Media region code is mismatched to logical unit" }, { 0x056f04, "Media region code is mismatched to logical unit" },
{ 0x056f05, "Drive region must be permanent / region reset count error" }, { 0x056f05, "Drive region must be permanent"
" / region reset count error" },
{ 0x057203, "Session fixation error - incomplete track in session" }, { 0x057203, "Session fixation error - incomplete track in session" },
{ 0x057204, "Empty or partially written reserved track" }, { 0x057204, "Empty or partially written reserved track" },
{ 0x057205, "No more RZONE reservations are allowed" }, { 0x057205, "No more RZONE reservations are allowed" },
...@@ -364,7 +364,6 @@ static const struct { ...@@ -364,7 +364,6 @@ static const struct {
{ 0x065a00, "Operator request or state change input (unspecified)" }, { 0x065a00, "Operator request or state change input (unspecified)" },
{ 0x065a01, "Operator medium removal request" }, { 0x065a01, "Operator medium removal request" },
{ 0x0bb900, "Play operation aborted" }, { 0x0bb900, "Play operation aborted" },
/* Here we use 0xff for the key (not a valid key) to signify /* Here we use 0xff for the key (not a valid key) to signify
* that these can have _any_ key value associated with them... */ * that these can have _any_ key value associated with them... */
{ 0xff0401, "Logical unit is in process of becoming ready" }, { 0xff0401, "Logical unit is in process of becoming ready" },
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment