Patch from Steven Scholz:

should we rename the define into CONFIG_FEATURE_HDPARM_HDIO_DMA and
use it for "set" and "get" ???

Since although CONFIG_FEATURE_HDPARM_HDIO_GET_DMA is _not_ set I still
can use "hdparm -d 1 /dev/hda" to switch the DMA on.
Is this desireable or meant to be like that?

So how about the applied patch?

1 files changed, 7 insertions, 1 deletions

diff --git a/include/usage.h b/include/usage.h
index 60a9f99f2..1e489a5c6 100644
--- a/include/usage.h
+++ b/include/usage.h
@@ -835,6 +835,12 @@
 #define USAGE_TRISTATE_HWIF(a)
 #endif
 
+#ifdef CONFIG_FEATURE_HDPARM_HDIO_GETSET_DMA
+#define USAGE_GETSET_DMA(a) a
+#else
+#define USAGE_GETSET_DMA(a)
+#endif
+
 #define hdparm_trivial_usage \
         "[options] [device] .."
 #define hdparm_full_usage \
@@ -845,7 +851,7 @@
         " -B   set Advanced Power Management setting (1-255)\n" \
         " -c   get/set IDE 32-bit IO setting\n" \
         " -C   check IDE power mode status\n" \
-        " -d   get/set using_dma flag\n" \
+        USAGE_GETSET_DMA(" -d   get/set using_dma flag\n") \
         " -D   enable/disable drive defect-mgmt\n" \
         " -f   flush buffer cache for device on exit\n" \
         " -g   display drive geometry\n" \