<html><head><meta name="color-scheme" content="light dark"></head><body><pre style="word-wrap: break-word; white-space: pre-wrap;">
From: Alan Cox &lt;alan@lxorguk.ukuu.org.uk&gt;

Replace the foot long pile of festering garbage in eighty_ninty_three with
some actual clean code.  All the ifdefs are fixed and havent changed since
2.4

Cc: Bartlomiej Zolnierkiewicz &lt;B.Zolnierkiewicz@elka.pw.edu.pl&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
---

 drivers/ide/ide-iops.c |   41 ++++++-----------------------------------
 1 files changed, 6 insertions(+), 35 deletions(-)

diff -puN drivers/ide/ide-iops.c~ide-clean-up-the-garbage-in-eighty_ninty_three drivers/ide/ide-iops.c
--- 25/drivers/ide/ide-iops.c~ide-clean-up-the-garbage-in-eighty_ninty_three	Mon Aug 29 15:50:03 2005
+++ 25-akpm/drivers/ide/ide-iops.c	Mon Aug 29 15:50:03 2005
@@ -601,44 +601,15 @@ EXPORT_SYMBOL(ide_wait_stat);
  */
 u8 eighty_ninty_three (ide_drive_t *drive)
 {
-#if 0
-	if (!HWIF(drive)-&gt;udma_four)
+	if(HWIF(drive)-&gt;udma_four == 0)
+		return 0;
+	if (!(drive-&gt;id-&gt;hw_config &amp; 0x6000))
 		return 0;
-
-	if (drive-&gt;id-&gt;major_rev_num) {
-		int hssbd = 0;
-		int i;
-		/*
-		 * Determine highest Supported SPEC
-		 */
-		for (i=1; i&lt;=15; i++)
-			if (drive-&gt;id-&gt;major_rev_num &amp; (1&lt;&lt;i))
-				hssbd++;
-
-		switch (hssbd) {
-			case 7:
-			case 6:
-			case 5:
-		/* ATA-4 and older do not support above Ultra 33 */
-			default:
-				return 0;
-		}
-	}
-
-	return ((u8) (
-#ifndef CONFIG_IDEDMA_IVB
-		(drive-&gt;id-&gt;hw_config &amp; 0x4000) &amp;&amp;
-#endif /* CONFIG_IDEDMA_IVB */
-		 (drive-&gt;id-&gt;hw_config &amp; 0x6000)) ? 1 : 0);
-
-#else
-
-	return ((u8) ((HWIF(drive)-&gt;udma_four) &amp;&amp;
 #ifndef CONFIG_IDEDMA_IVB
-			(drive-&gt;id-&gt;hw_config &amp; 0x4000) &amp;&amp;
+	if(!(drive-&gt;id-&gt;hw_config &amp; 0x4000))
+		return 0;
 #endif /* CONFIG_IDEDMA_IVB */
-			(drive-&gt;id-&gt;hw_config &amp; 0x6000)) ? 1 : 0);
-#endif
+	return 1;
 }
 
 EXPORT_SYMBOL(eighty_ninty_three);
_
</pre></body></html>