Skip to contentSkip to conversation

ide: don't BUG() on unsupported transfer modes

PATCH In review Bartlomiej Zolnierkiewicz 2 participants 2 messages Nov 4, 2007 – Nov 6, 2007 libata

lore
Share & tools
lore

Bartlomiej Zolnierkiewicz

author
#0 lore raw

PATCH 13/18 ide: don't BUG() on unsupported transfer modes

Fix ide-cris, cs5530, sc1200 and sis5513 host drivers to just return instead of OOPS-ing for unsupported modes in ->set_dma_mode methods.

Signed-off-by Bartlomiej Zolnierkiewicz <[email protected]>
---
 drivers/ide/cris/ide-cris.c |    3 +--
 drivers/ide/pci/cs5530.c    |    3 +--
 drivers/ide/pci/sc1200.c    |    3 +--
 drivers/ide/pci/sis5513.c   |    1 -
 4 files changed, 3 insertions(+), 7 deletions(-)
drivers/ide/cris/ide-cris.c +1 -2
Index: b/drivers/ide/cris/ide-cris.c
===================================================================
@@ -748,8 +748,7 @@ static void cris_set_dma_mode(ide_drive_
 			hold = ATA_DMA2_HOLD;
 			break;
 		default:
-			BUG();
-			break;
+			return;
 	}
 
 	if (speed >= XFER_UDMA_0)
drivers/ide/pci/cs5530.c +1 -2
Index: b/drivers/ide/pci/cs5530.c
===================================================================
@@ -117,8 +117,7 @@ static void cs5530_set_dma_mode(ide_driv
 		case XFER_MW_DMA_1:	timings = 0x00012121; break;
 		case XFER_MW_DMA_2:	timings = 0x00002020; break;
 		default:
-			BUG();
-			break;
+			return;
 	}
 	basereg = CS5530_BASEREG(drive->hwif);
 	reg = inl(basereg + 4);			/* get drive0 config register */
drivers/ide/pci/sc1200.c +1 -2
Index: b/drivers/ide/pci/sc1200.c
===================================================================
@@ -186,8 +186,7 @@ static void sc1200_set_dma_mode(ide_driv
 			}
 			break;
 		default:
-			BUG();
-			break;
+			return;
 	}
 
 	if (unit == 0) {			/* are we configuring drive0? */
drivers/ide/pci/sis5513.c +0 -1
Index: b/drivers/ide/pci/sis5513.c
===================================================================
@@ -368,7 +368,6 @@ static void sis_set_dma_mode(ide_drive_t
 			sis_program_timings(drive, speed);
 			break;
 		default:
-			BUG();
 			break;
 	}
 }