Skip to content

Commit c1de91e

Browse files
committed
Improve mctp help message
Clarify that invalid arguments are from the command line as oppposed to EINVAL. Give an example of physaddr syntax. Signed-off-by: Matt Johnston <[email protected]>
1 parent 9b0a936 commit c1de91e

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

src/mctp.c

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -760,7 +760,7 @@ static int cmd_addr_addremove(struct ctx *ctx,
760760
int ifindex;
761761

762762
if (argc != 4) {
763-
warnx("%s: invalid arguments", cmdname);
763+
warnx("%s: invalid command line arguments", cmdname);
764764
return -1;
765765
}
766766

@@ -905,7 +905,7 @@ static int cmd_route_add(struct ctx *ctx, int argc, const char **argv)
905905
rc = -EINVAL;
906906
}
907907
if (rc) {
908-
warnx("add: invalid arguments");
908+
warnx("add: invalid command line arguments");
909909
return -1;
910910
}
911911

@@ -935,7 +935,7 @@ static int cmd_route_del(struct ctx *ctx, int argc, const char **argv)
935935
rc = -EINVAL;
936936
}
937937
if (rc) {
938-
warnx("del: invalid arguments");
938+
warnx("del: invalid command line arguments");
939939
return -1;
940940
}
941941
eid = tmp & 0xff;
@@ -986,12 +986,12 @@ static int cmd_neigh_show(struct ctx *ctx, int argc, const char **argv)
986986
int rc;
987987

988988
if (!(argc <= 1 || argc == 3)) {
989-
warnx("show: invalid arguments");
989+
warnx("show: invalid command line arguments");
990990
return -1;
991991
}
992992
if (argc == 3) {
993993
if (strcmp(argv[1], "dev")) {
994-
warnx("show: invalid arguments");
994+
warnx("show: invalid command line arguments");
995995
return -1;
996996
}
997997
linkstr = argv[2];
@@ -1089,7 +1089,7 @@ static int cmd_neigh_add(struct ctx *ctx, int argc, const char **argv)
10891089
}
10901090
}
10911091
if (rc) {
1092-
warnx("add: invalid arguments");
1092+
warnx("add: invalid command line arguments");
10931093
return -1;
10941094
}
10951095

@@ -1131,7 +1131,7 @@ static int cmd_neigh_del(struct ctx *ctx, int argc, const char **argv)
11311131
}
11321132
}
11331133
if (rc) {
1134-
warnx("del: invalid arguments");
1134+
warnx("del: invalid command line arguments");
11351135
return -1;
11361136
}
11371137

@@ -1154,6 +1154,7 @@ static int cmd_neigh(struct ctx *ctx, int argc, const char **argv) {
11541154
fprintf(stderr, "%s neigh\n", ctx->top_cmd);
11551155
fprintf(stderr, "%s neigh show [dev <network>]\n", ctx->top_cmd);
11561156
fprintf(stderr, "%s neigh add <eid> dev <device> lladdr <physaddr>\n", ctx->top_cmd);
1157+
fprintf(stderr, " <physaddr> syntax is for example \"1d\" or \"aa:bb:cc:11:22:33\"\n");
11571158
fprintf(stderr, "%s neigh del <eid> dev <device>\n", ctx->top_cmd);
11581159
return 255;
11591160
}

0 commit comments

Comments
 (0)