diff options
| author | Rob Landley <rob@landley.net> | 2006-05-31 22:52:57 +0000 |
|---|---|---|
| committer | Rob Landley <rob@landley.net> | 2006-05-31 22:52:57 +0000 |
| commit | 06208417581fb8dcc05c8ca0250397d33ef3be52 (patch) | |
| tree | 8cf532e46f72f88e3d978729b85da33c05708ae0 /miscutils | |
| parent | c8b360ebb75ff41a5263828a591e7c0932ea944e (diff) | |
| download | busybox-06208417581fb8dcc05c8ca0250397d33ef3be52.tar.gz | |
Patch from Tito to remove long options from hdparm.
Diffstat (limited to 'miscutils')
| -rw-r--r-- | miscutils/Config.in | 7 | ||||
| -rw-r--r-- | miscutils/hdparm.c | 29 |
2 files changed, 11 insertions, 25 deletions
diff --git a/miscutils/Config.in b/miscutils/Config.in index eae977fac..fc90b5447 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 55ed2f6ff..37b87ea9f 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); |