commit 7cc386e9def8f99bb7bbe8ca06a07a7b17198684
author Chaehyun Lim <chaehyun.lim@gmail.com> Thu Mar 03 21:05:20 2016 +0900
committer Greg Kroah-Hartman <gregkh@linuxfoundation.org> Thu Mar 03 17:58:46 2016 -0800
tree f2a61ca55b3bde4bae7db643a2981f64b381c540
parent 76e6121c90c046c75ae06ece9081d874d1570d20

staging: wilc1000: add enum cfg_type_cmd

This patch adds a new enum cfg_type_cmd to change hard-coded command
type.

Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

drivers/staging/wilc1000/wilc_wlan_cfg.c

diff --git a/drivers/staging/wilc1000/wilc_wlan_cfg.c b/drivers/staging/wilc1000/wilc_wlan_cfg.c
index 6a33187..9cf68b7 100644
--- a/drivers/staging/wilc1000/wilc_wlan_cfg.c
+++ b/drivers/staging/wilc1000/wilc_wlan_cfg.c
@@ -18,6 +18,13 @@
  *      Global Data
  *
  ********************************************/
+enum cfg_cmd_type {
+	CFG_BYTE_CMD	= 0,
+	CFG_HWORD_CMD	= 1,
+	CFG_WORD_CMD	= 2,
+	CFG_STR_CMD	= 3,
+	CFG_BIN_CMD	= 4
+};
 
 struct wilc_mac_cfg {
 	int mac_status;
@@ -371,18 +378,18 @@
 	u8 type = (id >> 12) & 0xf;
 	int ret = 0;
 
-	if (type == 0) {                                        /* byte command */
+	if (type == CFG_BYTE_CMD) {                                        /* byte command */
 		if (size >= 1)
 			ret = wilc_wlan_cfg_set_byte(frame, offset, id, *buf);
-	} else if (type == 1) {                 /* half word command */
+	} else if (type == CFG_HWORD_CMD) {                 /* half word command */
 		if (size >= 2)
 			ret = wilc_wlan_cfg_set_hword(frame, offset, id, *((u16 *)buf));
-	} else if (type == 2) {                 /* word command */
+	} else if (type == CFG_WORD_CMD) {                 /* word command */
 		if (size >= 4)
 			ret = wilc_wlan_cfg_set_word(frame, offset, id, *((u32 *)buf));
-	} else if (type == 3) {                 /* string command */
+	} else if (type == CFG_STR_CMD) {                 /* string command */
 		ret = wilc_wlan_cfg_set_str(frame, offset, id, buf, size);
-	} else if (type == 4) {                 /* binary command */
+	} else if (type == CFG_BIN_CMD) {                 /* binary command */
 		ret = wilc_wlan_cfg_set_bin(frame, offset, id, buf, size);
 	}
 
@@ -415,7 +422,7 @@
 	}
 
 	i = 0;
-	if (type == 0) {                                        /* byte command */
+	if (type == CFG_BYTE_CMD) {                                        /* byte command */
 		do {
 			if (g_cfg_byte[i].id == WID_NIL)
 				break;
@@ -427,7 +434,7 @@
 			}
 			i++;
 		} while (1);
-	} else if (type == 1) {                 /* half word command */
+	} else if (type == CFG_HWORD_CMD) {                 /* half word command */
 		do {
 			if (g_cfg_hword[i].id == WID_NIL)
 				break;
@@ -439,7 +446,7 @@
 			}
 			i++;
 		} while (1);
-	} else if (type == 2) {                 /* word command */
+	} else if (type == CFG_WORD_CMD) {                 /* word command */
 		do {
 			if (g_cfg_word[i].id == WID_NIL)
 				break;
@@ -451,7 +458,7 @@
 			}
 			i++;
 		} while (1);
-	} else if (type == 3) {                 /* string command */
+	} else if (type == CFG_STR_CMD) {                 /* string command */
 		do {
 			if (g_cfg_str[i].id == WID_NIL)
 				break;