ide: Tidy up ide_reset()
authorSimon Glass <sjg@chromium.org>
Tue, 25 Apr 2023 16:54:52 +0000 (10:54 -0600)
committerTom Rini <trini@konsulko.com>
Thu, 27 Apr 2023 17:51:07 +0000 (13:51 -0400)
Avoid using #ifdef and use a single function declaration, so it is easier
to read.

Signed-off-by: Simon Glass <sjg@chromium.org>
drivers/block/ide.c

index 7221654..fb40933 100644 (file)
@@ -45,24 +45,23 @@ ulong ide_bus_offset[CONFIG_SYS_IDE_MAXBUS] = {
 
 #define IDE_SPIN_UP_TIME_OUT 5000 /* 5 sec spin-up timeout */
 
-#ifdef CONFIG_IDE_RESET
 static void ide_reset(void)
 {
-       ide_set_reset(1);       /* assert reset */
+       if (IS_ENABLED(CONFIG_IDE_RESET)) {
+               /* assert reset */
+               ide_set_reset(1);
 
-       /* the reset signal shall be asserted for et least 25 us */
-       udelay(25);
+               /* the reset signal shall be asserted for et least 25 us */
+               udelay(25);
 
-       schedule();
+               schedule();
 
-       /* de-assert RESET signal */
-       ide_set_reset(0);
+               /* de-assert RESET signal */
+               ide_set_reset(0);
 
-       mdelay(250);
+               mdelay(250);
+       }
 }
-#else
-#define ide_reset()    /* dummy */
-#endif /* CONFIG_IDE_RESET */
 
 static void ide_outb(int dev, int port, unsigned char val)
 {