Commit 68e7f6f3 authored by Alessandro Rubini's avatar Alessandro Rubini

trivial: tools: use standard optind variable

Signed-off-by: Alessandro Rubini's avatarAlessandro Rubini <rubini@gnudd.com>
parent 95b2216c
...@@ -56,7 +56,6 @@ int main(int argc, char *argv[]) ...@@ -56,7 +56,6 @@ int main(int argc, char *argv[])
{0, 0, 0, 0} {0, 0, 0, 0}
}; };
int opt_index = 0, err = 0, i; int opt_index = 0, err = 0, i;
int argcd = 0; /* number of argument elaborated */
unsigned int dev_id = 0; unsigned int dev_id = 0;
char c; char c;
...@@ -72,11 +71,6 @@ int main(int argc, char *argv[]) ...@@ -72,11 +71,6 @@ int main(int argc, char *argv[])
/* Parse options */ /* Parse options */
while( (c = getopt_long(argc, argv, "p:P:n:d:D:t:c:h", while( (c = getopt_long(argc, argv, "p:P:n:d:D:t:c:h",
options, &opt_index)) >=0 ){ options, &opt_index)) >=0 ){
if (!c) {
argcd++;
continue;
}
switch(c){ switch(c){
case 'p': case 'p':
fmcadc_set_attr(&acq, FMCADC_CONF_ACQ_PRE_SAMP, fmcadc_set_attr(&acq, FMCADC_CONF_ACQ_PRE_SAMP,
...@@ -111,15 +105,14 @@ int main(int argc, char *argv[]) ...@@ -111,15 +105,14 @@ int main(int argc, char *argv[])
exit(1); exit(1);
break; break;
} }
argcd += 2;
} }
if (argcd == argc - 1 ) { if (optind != argc - 1 ) {
printf("Error: DEVICE ID is a mandatory argument\n"); printf("Error: DEVICE ID is a mandatory argument\n");
fald_help(); fald_help();
exit(1); exit(1);
} else { } else {
sscanf(argv[argc - 1], "0x%x", &dev_id); sscanf(argv[optind], "0x%x", &dev_id);
} }
printf("Open ADC fmcadc_100MS_4ch_14bit dev_id 0x%04x ...\n", dev_id); printf("Open ADC fmcadc_100MS_4ch_14bit dev_id 0x%04x ...\n", dev_id);
......
...@@ -34,7 +34,6 @@ int main(int argc, char *argv[]) ...@@ -34,7 +34,6 @@ int main(int argc, char *argv[])
{0, 0, 0, 0} {0, 0, 0, 0}
}; };
int opt_index = 0, err = 0, i; int opt_index = 0, err = 0, i;
int argcd = 0; /* number of argument elaborated */
unsigned int dev_id = 0; unsigned int dev_id = 0;
char c; char c;
...@@ -50,26 +49,20 @@ int main(int argc, char *argv[]) ...@@ -50,26 +49,20 @@ int main(int argc, char *argv[])
/* Parse options */ /* Parse options */
while( (c = getopt_long(argc, argv, "h", while( (c = getopt_long(argc, argv, "h",
options, &opt_index)) >=0 ){ options, &opt_index)) >=0 ){
if (!c) {
argcd++;
continue;
}
switch(c){ switch(c){
case 'h': case 'h':
fald_help(); fald_help();
exit(1); exit(1);
break; break;
} }
argcd += 2;
} }
if (argcd == argc - 1 ) { if (optind != argc - 1 ) {
printf("Error: DEVICE ID is a mandatory argument\n"); printf("Error: DEVICE ID is a mandatory argument\n");
fald_help(); fald_help();
exit(1); exit(1);
} else { } else {
sscanf(argv[argc - 1], "0x%x", &dev_id); sscanf(argv[optind], "0x%x", &dev_id);
} }
printf("Open ADC fmcadc_100MS_4ch_14bit dev_id 0x%04x ...\n", dev_id); printf("Open ADC fmcadc_100MS_4ch_14bit dev_id 0x%04x ...\n", dev_id);
......
...@@ -118,7 +118,6 @@ int main(int argc, char *argv[]) ...@@ -118,7 +118,6 @@ int main(int argc, char *argv[])
{0, 0, 0, 0} {0, 0, 0, 0}
}; };
int opt_index = 0, err = 0; int opt_index = 0, err = 0;
int argcd = 0; /* number of argument elaborated */
char c; char c;
long time1[3], time2[3]; long time1[3], time2[3];
...@@ -135,7 +134,7 @@ int main(int argc, char *argv[]) ...@@ -135,7 +134,7 @@ int main(int argc, char *argv[])
} }
} }
if (argcd == argc - 1 ) { if (optind != argc - 1 ) {
printf("Error: DEVICE is a mandatory argument\n"); printf("Error: DEVICE is a mandatory argument\n");
fau_help(); fau_help();
exit(1); exit(1);
......
...@@ -117,7 +117,6 @@ int main(int argc, char *argv[]) ...@@ -117,7 +117,6 @@ int main(int argc, char *argv[])
{0, 0, 0, 0} {0, 0, 0, 0}
}; };
int i, opt_index = 0, err = 0, cur_val; int i, opt_index = 0, err = 0, cur_val;
int argcd = 0; /* number of argument elaborated */
char c; char c;
if (argc == 1) { if (argc == 1) {
...@@ -127,11 +126,6 @@ int main(int argc, char *argv[]) ...@@ -127,11 +126,6 @@ int main(int argc, char *argv[])
while( (c = getopt_long(argc, argv, "p:P:n:d:t:c:h", while( (c = getopt_long(argc, argv, "p:P:n:d:t:c:h",
options, &opt_index)) >=0 ){ options, &opt_index)) >=0 ){
if (!c) {
argcd++;
continue;
}
switch(c){ switch(c){
case 'p': case 'p':
attrval[FAU_TRG_PRE] = atoi(optarg); attrval[FAU_TRG_PRE] = atoi(optarg);
...@@ -156,16 +150,15 @@ int main(int argc, char *argv[]) ...@@ -156,16 +150,15 @@ int main(int argc, char *argv[])
exit(1); exit(1);
break; break;
} }
argcd += 2;
} }
if (argcd == argc - 1 ) { if (optind != argc - 1 ) {
printf("Error: DEVICE is a mandatory argument\n"); printf("Error: DEVICE is a mandatory argument\n");
fau_help(); fau_help();
exit(1); exit(1);
} }
strcat(basepath, argv[argc-1]); strcat(basepath, argv[optind]);
printf("Sysfs path to device is: %s\n", basepath); printf("Sysfs path to device is: %s\n", basepath);
for (i = 0; i < FAU_TRIG_NUM_ATTR; ++i) { for (i = 0; i < FAU_TRIG_NUM_ATTR; ++i) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment