diff options
author | Marc Kleine-Budde <mkl@pengutronix.de> | 2010-10-25 12:30:52 +0200 |
---|---|---|
committer | Marc Kleine-Budde <mkl@pengutronix.de> | 2015-09-18 12:55:24 +0200 |
commit | b182cc719baa9b6de31a21e919d85707eebb4bdb (patch) | |
tree | 879b2e238e4b8388fc4309709e6b00f50c583142 | |
parent | f9d600c8ec852241139273e09822d582974a0c08 (diff) | |
download | canutils-b182cc719baa9b6de31a21e919d85707eebb4bdb.tar.gz canutils-b182cc719baa9b6de31a21e919d85707eebb4bdb.tar.xz |
canconfig: codingstyle cleanups
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
-rw-r--r-- | src/canconfig.c | 46 |
1 files changed, 23 insertions, 23 deletions
diff --git a/src/canconfig.c b/src/canconfig.c index 2dfa559..01e32c2 100644 --- a/src/canconfig.c +++ b/src/canconfig.c @@ -1,7 +1,7 @@ /* * canutils/canconfig.c * - * Copyright (C) 2005, 2008 Marc Kleine-Budde <mkl@pengutronix.de>, Pengutronix + * Copyright (C) 2005, 2008, 2010 Marc Kleine-Budde <mkl@pengutronix.de>, Pengutronix * Copyright (C) 2007 Juergen Beisert <jbe@pengutronix.de>, Pengutronix * Copyright (C) 2009 Luotao Fu <l.fu@pengutronix.de>, Pengutronix * @@ -38,19 +38,19 @@ #define MIN(a, b) ((a) < (b) ? (a) : (b)) #endif -const char *can_states[CAN_STATE_MAX] = { +static const char *can_states[CAN_STATE_MAX] = { "ERROR-ACTIVE", "ERROR-WARNING", "ERROR-PASSIVE", "BUS-OFF", "STOPPED", - "SLEEPING" + "SLEEPING", }; -const char *config_keywords[] = { - "baudrate", "bitrate", "bittiming", "ctrlmode", "restart", - "start", "stop", "restart-ms", "state", "clockfreq", - "bittiming-const", "berr-counter"}; +static const char *config_keywords[] = { + "baudrate", "bitrate", "bittiming", "ctrlmode", "restart", + "start", "stop", "restart-ms", "state", "clockfreq", + "bittiming-const", "berr-counter" }; /* this is shamelessly stolen from iproute and slightly modified */ #define NEXT_ARG() \ @@ -58,12 +58,12 @@ const char *config_keywords[] = { argv++; \ if (--argc < 0) { \ fprintf(stderr, "missing parameter for %s\n", *argv); \ - exit(EXIT_FAILURE);\ - }\ + exit(EXIT_FAILURE); \ + } \ } while(0) static inline int find_str(const char** haystack, unsigned int stack_size, - const char* needle) + const char* needle) { int i, found = 0; @@ -153,8 +153,8 @@ static void cmd_bitrate(int argc, char *argv[], const char *name) if (argc > 0) show_only = find_str(config_keywords, - sizeof(config_keywords) / sizeof(char*), - argv[1]); + sizeof(config_keywords) / sizeof(char*), + argv[1]); if (! show_only) do_set_bitrate(argc, argv, name); @@ -207,8 +207,8 @@ static void do_set_bittiming(int argc, char *argv[], const char *name) * parameters have to be set */ if (bt_par_count < 4) { fprintf(stderr, "%s: missing bittiming parameters, " - "try help to figure out the correct format\n", - name); + "try help to figure out the correct format\n", + name); exit(1); } if (can_set_bittiming(name, &bt) < 0) { @@ -238,8 +238,8 @@ static void cmd_bittiming(int argc, char *argv[], const char *name) if (argc > 0) show_only = find_str(config_keywords, - sizeof(config_keywords) / sizeof(char*), - argv[1]); + sizeof(config_keywords) / sizeof(char*), + argv[1]); if (! show_only) do_set_bittiming(argc, argv, name); @@ -297,7 +297,7 @@ static void do_show_clockfreq(const char *name) memset(&clock, 0, sizeof(struct can_clock)); if (can_get_clock(name, &clock) < 0) { fprintf(stderr, "%s: failed to get clock parameters\n", - name); + name); exit(EXIT_FAILURE); } @@ -380,7 +380,7 @@ static void do_show_ctrlmode(const char *name) /* this is shamelessly stolen from iproute and slightly modified */ static inline void set_ctrlmode(char* name, char *arg, - struct can_ctrlmode *cm, __u32 flags) + struct can_ctrlmode *cm, __u32 flags) { if (strcmp(arg, "on") == 0) { cm->flags |= flags; @@ -437,8 +437,8 @@ static void cmd_ctrlmode(int argc, char *argv[], const char *name) if (argc > 0) show_only = find_str(config_keywords, - sizeof(config_keywords) / sizeof(char*), - argv[1]); + sizeof(config_keywords) / sizeof(char*), + argv[1]); if (! show_only) do_set_ctrlmode(argc, argv, name); @@ -461,7 +461,7 @@ static void do_show_restart_ms(const char *name) static void do_set_restart_ms(int argc, char* argv[], const char *name) { if (can_set_restart_ms(name, - (__u32)strtoul(argv[1], NULL, 10)) < 0) { + (__u32)strtoul(argv[1], NULL, 10)) < 0) { fprintf(stderr, "failed to set restart_ms of %s to %lu\n", name, strtoul(argv[1], NULL, 10)); exit(EXIT_FAILURE); @@ -474,8 +474,8 @@ static void cmd_restart_ms(int argc, char *argv[], const char *name) if (argc > 0) show_only = find_str(config_keywords, - sizeof(config_keywords) / sizeof(char*), - argv[1]); + sizeof(config_keywords) / sizeof(char*), + argv[1]); if (! show_only) do_set_restart_ms(argc, argv, name); |