[PATCH 1/5] staging: wilc1000: avoid arrray of 'wilc_debugfs_info_t' type

[Ajay Singh]

Refactor code by removing array of 'wilc_debugfs_info_t' type and use
single variable to store 'wilc_debugfs_info_t' struct value.

Signed-off-by: Ajay Singh <ajay.kat...@microchip.com>
---
 drivers/staging/wilc1000/wilc_debugfs.c | 26 +++++++++-----------------
 1 file changed, 9 insertions(+), 17 deletions(-)

diff --git a/drivers/staging/wilc1000/wilc_debugfs.c 
b/drivers/staging/wilc1000/wilc_debugfs.c
index edc7287..c5f67c9 100644
--- a/drivers/staging/wilc1000/wilc_debugfs.c
+++ b/drivers/staging/wilc1000/wilc_debugfs.c
@@ -78,29 +78,21 @@ struct wilc_debugfs_info_t {
        const struct file_operations fops;
 };
 
-static struct wilc_debugfs_info_t debugfs_info[] = {
-       {
-               "wilc_debug_level",
-               0666,
-               (DEBUG | ERR),
-               FOPS(NULL, wilc_debug_level_read, wilc_debug_level_write, NULL),
-       },
+static struct wilc_debugfs_info_t debugfs_info = {
+       "wilc_debug_level",
+       0666,
+       (DEBUG | ERR),
+       FOPS(NULL, wilc_debug_level_read, wilc_debug_level_write, NULL),
 };
 
 int wilc_debugfs_init(void)
 {
-       int i;
-       struct wilc_debugfs_info_t *info;
+       struct wilc_debugfs_info_t *info = &debugfs_info;
 
        wilc_dir = debugfs_create_dir("wilc_wifi", NULL);
-       for (i = 0; i < ARRAY_SIZE(debugfs_info); i++) {
-               info = &debugfs_info[i];
-               debugfs_create_file(info->name,
-                                   info->perm,
-                                   wilc_dir,
-                                   &info->data,
-                                   &info->fops);
-       }
+       debugfs_create_file(info->name, info->perm, wilc_dir, &info->data,
+                           &info->fops);
+
        return 0;
 }
 
-- 
2.7.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel