[PATCH] staging: rtl8712: cleanup checkpatch.pl warnings

2021-02-14 Thread Shreesh Adiga
Cleanup warnings displayed by checkpatch.pl for the file rtl8171x_mlme.c for the below warning: WARNING: Avoid multiple line dereference Signed-off-by: Shreesh Adiga <16567adigashre...@gmail.com> --- drivers/staging/rtl8712/rtl871x_mlme.c | 112 + 1 file chang

[PATCH] staging: ks7010: cleanup checkpatch.pl macro error

2021-02-14 Thread Shreesh Adiga
Add parentheses around the macro definitions in ks_hostif.h to clear the error output by checkpatch.pl Signed-off-by: Shreesh Adiga <16567adigashre...@gmail.com> --- drivers/staging/ks7010/ks_hostif.h | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff

[PATCH] staging: octeon: refactor interface check logic in ethernet.c

2021-02-21 Thread Shreesh Adiga
The check for interface is duplicated in 3 places and has been refactored into a function. Also the if condition was wrapping the whole body in all three places, so it has been changed to return if the condition is false to reduce the indentation levels. Signed-off-by: Shreesh Adiga

[PATCH 1/2] staging: octeon: refactor interface check logic in ethernet.c

2021-02-21 Thread Shreesh Adiga
The check for interface is duplicated in 3 places and has been refactored into a function. Signed-off-by: Shreesh Adiga <16567adigashre...@gmail.com> --- drivers/staging/octeon/ethernet.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/s

[PATCH 2/2] staging: octeon: reduce indentation levels for functions in ethernet.c

2021-02-21 Thread Shreesh Adiga
The if condition was wrapping the whole body in three functions, it has been changed to return if the condition is false to reduce the indentation levels. Signed-off-by: Shreesh Adiga <16567adigashre...@gmail.com> --- drivers/staging/octeon/ethernet.c | 194 +++---

[PATCH] staging: octeon: cleanup unnecessary parentheses in ethernet-spmi.c

2021-02-19 Thread Shreesh Adiga
checkpatch.pl reported issue of unnecessary parentheses for the expression. It has been removed to fix the report. Signed-off-by: Shreesh Adiga <16567adigashre...@gmail.com> --- drivers/staging/octeon/ethernet-spi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d