summaryrefslogtreecommitdiff
path: root/miscutils
diff options
context:
space:
mode:
authorRob Landley2006-05-31 22:52:57 +0000
committerRob Landley2006-05-31 22:52:57 +0000
commit06208417581fb8dcc05c8ca0250397d33ef3be52 (patch)
tree8cf532e46f72f88e3d978729b85da33c05708ae0 /miscutils
parentc8b360ebb75ff41a5263828a591e7c0932ea944e (diff)
downloadbusybox-06208417581fb8dcc05c8ca0250397d33ef3be52.zip
busybox-06208417581fb8dcc05c8ca0250397d33ef3be52.tar.gz
Patch from Tito to remove long options from hdparm.
Diffstat (limited to 'miscutils')
-rw-r--r--miscutils/Config.in7
-rw-r--r--miscutils/hdparm.c29
2 files changed, 11 insertions, 25 deletions
diff --git a/miscutils/Config.in b/miscutils/Config.in
index eae977f..fc90b54 100644
--- a/miscutils/Config.in
+++ b/miscutils/Config.in
@@ -169,11 +169,12 @@ config CONFIG_HDPARM
config CONFIG_FEATURE_HDPARM_GET_IDENTITY
bool "Support obtaining detailed information directly from drives"
default y
- depends on CONFIG_HDPARM && CONFIG_GETOPT_LONG
+ depends on CONFIG_HDPARM
help
- Enables the -I and -Istdin options to obtain detailed information
+ Enables the -I and -i options to obtain detailed information
directly from drives about their capabilities and supported ATA
- feature set. Enabling this option will add about 16k...
+ feature set. If no device name is specified, hdparm will read
+ identify data from stdin. Enabling this option will add about 16k...
config CONFIG_FEATURE_HDPARM_HDIO_SCAN_HWIF
bool "Register an IDE interface (DANGEROUS)"
diff --git a/miscutils/hdparm.c b/miscutils/hdparm.c
index 55ed2f6..37b87ea 100644
--- a/miscutils/hdparm.c
+++ b/miscutils/hdparm.c
@@ -23,7 +23,6 @@
#include <sys/times.h>
#include <sys/mount.h>
#include <sys/mman.h>
-#include <getopt.h>
#include <linux/types.h>
#include <linux/hdreg.h>
@@ -2095,21 +2094,9 @@ static void parse_xfermode(int flag, unsigned long *get, unsigned long *set, int
}
}
-/*-------------------------------------*/
-/* getopt long options */
-#if ENABLE_FEATURE_HDPARM_GET_IDENTITY
-static const struct option HDPARM_LONG_OPT[] =
-{
- {"Istdin", 0, NULL, '\256'},
- {NULL, 0, NULL, 0}
-};
-#else
-#define HDPARM_LONG_OPT NULL
-#endif
-/*-------------------------------------*/
-/* getopt short options */
+/*------- getopt short options --------*/
static const char hdparm_options[]= "gfu::n::p:r::m::c::k::a::B:tTh"\
- USE_FEATURE_HDPARM_GET_IDENTITY("\256iI")
+ USE_FEATURE_HDPARM_GET_IDENTITY("iI")
USE_FEATURE_HDPARM_HDIO_GETSET_DMA("d::")
#ifdef HDIO_DRIVE_CMD
"S::D::P::X::K::A::L::W::CyYzZ"
@@ -2134,13 +2121,9 @@ int hdparm_main(int argc, char **argv)
int c;
int flagcount = 0;
- while ((c = getopt_long (argc, argv, hdparm_options, HDPARM_LONG_OPT , NULL)) >= 0) {
+ while ((c = getopt(argc, argv, hdparm_options)) >= 0) {
flagcount++;
-#if ENABLE_FEATURE_HDPARM_GET_IDENTITY
- if (c == '\256') {
- identify_from_stdin(); /* EXIT */
- }
-#endif
+ if (c == 'h') bb_show_usage(); /* EXIT */
USE_FEATURE_HDPARM_GET_IDENTITY(get_IDentity = (c == 'I'));
USE_FEATURE_HDPARM_GET_IDENTITY(get_identity = (c == 'i'));
get_geom |= (c == 'g');
@@ -2204,9 +2187,11 @@ int hdparm_main(int argc, char **argv)
argc -= optind;
argv += optind;
+#if ENABLE_FEATURE_HDPARM_GET_IDENTITY
if (argc < 1) {
- bb_show_usage();
+ identify_from_stdin(); /* EXIT */
}
+#endif
while (argc--) {
process_dev(*argv);