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.kathat@xxxxxxxxxxxxx> --- 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