diff options
| author | Sascha Hauer <s.hauer@pengutronix.de> | 2013-02-25 12:50:30 +0100 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2013-03-04 11:00:33 +0100 |
| commit | 938f748103318b937bac50a8b23b150bb947a574 (patch) | |
| tree | 30d47888d38d53a2a494c37b0fad0166bd3de869 /commands | |
| parent | 29c589c8dcd6a4914ce30b1cdae4a8de7d7f24b8 (diff) | |
| download | barebox-938f748103318b937bac50a8b23b150bb947a574.tar.gz barebox-938f748103318b937bac50a8b23b150bb947a574.tar.xz | |
nand command: use enumeration for command instead of bitmask
A command can only be one of NAND_*, so use an enumeration instead
of a bitmask.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'commands')
| -rw-r--r-- | commands/nand.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/commands/nand.c b/commands/nand.c index d00845f316..79f6735a72 100644 --- a/commands/nand.c +++ b/commands/nand.c @@ -29,9 +29,9 @@ #include <fcntl.h> #include <libgen.h> -#define NAND_ADD (1 << 0) -#define NAND_DEL (1 << 1) -#define NAND_MARKBAD (1 << 2) +#define NAND_ADD 1 +#define NAND_DEL 2 +#define NAND_MARKBAD 3 static int do_nand(int argc, char *argv[]) { @@ -58,7 +58,7 @@ static int do_nand(int argc, char *argv[]) } } - if (command & NAND_ADD) { + if (command == NAND_ADD) { while (optind < argc) { if (dev_add_bb_dev(basename(argv[optind]), NULL)) return 1; @@ -67,14 +67,14 @@ static int do_nand(int argc, char *argv[]) } } - if (command & NAND_DEL) { + if (command == NAND_DEL) { while (optind < argc) { dev_remove_bb_dev(basename(argv[optind])); optind++; } } - if (command & NAND_MARKBAD) { + if (command == NAND_MARKBAD) { if (optind < argc) { int ret = 0, fd; |