[PATCH] staging: wilc1000: remove unused return variable

From: HariPrasath Elango In this function,removed the unused integer variable as it is not actually used to return function success or failure. Return is a pointer to net_device structure. Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/linux_mon.c | 6 ++ 1 file changed, 2

[PATCH] staging: fsl-dpaa2: replace simple switch case by if statement

From: HariPrasath Elango Replace a couple of simple switch cases by if condition Signed-off-by: HariPrasath Elango --- drivers/staging/fsl-dpaa2/ethernet/dpaa2-ethtool.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/staging/fsl-dpaa2/ethernet/dpaa2

[PATCHv3] staging: wilc1000: replace switch statement by simple if condition

From: HariPrasath Elango In this case,there is only a single switch case statement.So replacing by a simple if condition Signed-off-by: HariPrasath Elango --- changes since v3: *rebase on latest code *missed revision history in v2 drivers/staging/wilc1000

[PATCHv2] staging: wilc1000: replace switch statement by simple if condition

From: HariPrasath Elango In this case,there is only a single switch case statement.So replacing by a simple if condition Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers

[PATCH] staging: mt7621-gpio: remove redundant owner assignments of drivers

From: HariPrasath Elango Remove the reduntant owner initialization from this platform driver as the platform_driver_register() takes care of it. Signed-off-by: HariPrasath Elango --- drivers/staging/mt7621-gpio/gpio-mt7621.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging

[PATCH 4/7] staging: wilc1000: destroy initialized mutex object

From: HariPrasath Elango A mutex object that is initialized but not destroyed.This patch destroys the mutex object Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/wilc1000

[PATCH 3/7] staging: wilc1000: use kmemdup instead of kmalloc and memcpy

From: HariPrasath Elango Kmalloc followed by memcpy can be replaced by kmemdup. Signed-off-by: HariPrasath Elango Reviewed-by: Ajay Singh --- drivers/staging/wilc1000/linux_mon.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b

[PATCH 5/7] staging: wilc1000: replace switch statement by simple if condition

From: HariPrasath Elango In this case,there is only a single switch case statement.So replacing by a simple if condition. Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers

[PATCH 7/7] staging: wilc1000: use kmemdup to replace kmalloc/memcpy

From: HariPrasath Elango kmalloc followed by memcpy can be replaced by kmemdup.Also added the related error handling part Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/host_interface.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers

[PATCH 6/7] staging: wilc1000: remove unwanted braces and correct code alignment

From: HariPrasath Elango Remove the unwated brace and corrected the code block alignment accordingly Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/host_interface.c | 25 - 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/drivers/staging

[PATCH 1/7] staging: wilc1000: Fix code block alignment

From: HariPrasath Elango Fix the code alignment for a block of code to adhere to coding guidelines Signed-off-by: HariPrasath Elango Reviewed-by: Ajay Singh --- drivers/staging/wilc1000/linux_wlan.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging

[PATCH 2/7] staging: wilc1000: Destroy mutex object in deinitialization

From: HariPrasath Elango Destroy the mutex object that is initialized in wlan_init_locks() Signed-off-by: HariPrasath Elango Reviewed-by: Ajay Singh --- drivers/staging/wilc1000/linux_wlan.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers

[PATCH 0/7] Cleanup patches for wilc1000 driver

From: HariPrasath Elango This patchset has few cleanup patches related to coding guidelines and few trivial changes HariPrasath Elango (7): staging: wilc1000: Fix code block alignment staging: wilc1000: Destroy mutex object in deinitialization staging: wilc1000: use kmemdup instead of

[PATCH] staging: pi433: Remove comments inside code

From: HariPrasath Elango Remove comments inbetween code as in this case. Signed-off-by: HariPrasath Elango --- drivers/staging/pi433/pi433_if.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index 88da91d

[PATCH] staging: wilc1000: use kmemdup to replace kmalloc/memcpy

From: HariPrasath Elango kmalloc followed by memcpy can be replaced by kmemdup.Also added the related error handling part Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/host_interface.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers

[PATCH] staging: wilc1000: remove unwanted braces and correct code alignment

From: HariPrasath Elango Remove the unwated brace and corrected the code block alignment accordingly Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/host_interface.c | 25 - 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/drivers/staging

[PATCH] staging: wilc1000: replace switch statement by simple if condition

From: HariPrasath Elango In this case,there is only a single switch case statement.So replacing by a simple if condition. Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers

[PATCH] staging: wilc1000: destroy initialized mutex object

From: HariPrasath Elango A mutex object that is initialized but not destroyed.This patch destroys the mutex object Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/wilc1000

[PATCH] staging: wilc1000: use kmemdup instead of kmalloc and memcpy

From: HariPrasath Elango Kmalloc followed by memcpy can be replaced by kmemdup. Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/linux_mon.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drivers/staging/wilc1000

[PATCH] staging: wilc1000: Destroy mutex object in deinitialization

From: HariPrasath Elango Destroy the mutex object that is initialized in wlan_init_locks() Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/linux_wlan.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000

[PATCH] staging: wilc1000: Fix code block alignment

From: HariPrasath Elango Fix the code alignment for a block of code to adhere to coding guidelines Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/linux_wlan.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b

[PATCHv2] staging: wilc1000: use pre-defined macro is_broadcast_ether_addr

From: HariPrasath Elango Use the kernel pre-defined macro is_broadcast_ether_addr() instead of doing a memcmp here. Signed-off-by: HariPrasath Elango --- drivers/staging/wilc1000/linux_mon.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000

[PATCH] staging: wilc1000: use pre-defined macro is_broadcast_ether_addr

From: Hari Prasath Use the kernel pre-defined macro is_broadcast_ether_addr() instead of doing a memcmp here. Signed-off-by: Hari Prasath --- drivers/staging/wilc1000/linux_mon.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drive

[PATCH] staging: iio: meter: Remove reduntant __func__ from debug print

From: HariPrasath Elango dev_dbg includes the function name & line number by default when dynamic debugging is enabled. Hence__func__ is reduntant here and removed. Signed-off-by: HariPrasath Elango --- drivers/staging/iio/meter/ade7758_trigger.c | 2 +- 1 file changed, 1 insertion(+

[PATCH] staging: iio: adc: Remove reduntant __func__ from debug print

From: HariPrasath Elango dev_dbg includes the function name & line number by default when dynamic debugging is enabled. Hence__func__ is reduntant here and removed. Signed-off-by: HariPrasath Elango --- drivers/staging/iio/meter/ade7758_trigger.c | 2 +- 1 file changed, 1 insertion(+