[PATCH] scripts: Add intended executable mode and SPDX license

2020-08-27 Thread Mrinal Pandey
these files and provide the SPDX license for spdxcheck-test.sh. No functional changes. Signed-off-by: Mrinal Pandey --- applies cleanly on next-20200827 Kees, Matthias, Thomas, please ack this patch. Andrew, please pick this minor non-urgent cleanup patch once the mainainers ack. scripts/gcc

Re: [PATCH v2] checkpatch: Fix the usage of capture group ( ... )

2020-08-19 Thread Mrinal Pandey
On 20/07/30 12:31AM, Joe Perches wrote: > On Thu, 2020-07-30 at 07:58 +0200, Lukas Bulwahn wrote: > > Hi Joe, > > > > did you see this quick fix to checkpatch.pl? Can you comment on the > > commit and can we get a quick ack on that fix, please? > > Yes, in a bit. > > > General question on patch

[PATCH] x86: entry: Fix a variable declaration coding style issue

2020-07-28 Thread Mrinal Pandey
Add a line after variable declaration as suggested by checkpatch on commit 3c73b81a9164 ("x86/entry, selftests: Further improve user entry sanity checks") Signed-off-by: Mrinal Pandey --- arch/x86/entry/common.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/x86/entry/commo

[PATCH 5/6] drivers: android: Fix a variable declaration coding style issue

2020-07-24 Thread Mrinal Pandey
Add a blank line after variable declarations as suggested by checkpatch. Signed-off-by: Mrinal Pandey --- drivers/android/binderfs.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/android/binderfs.c b/drivers/android/binderfs.c index 7cf566aafe1f..8a98a36ff71a 100644 --- a/drivers

[PATCH 4/6] drivers: android: Fix a variable declaration coding style issue

2020-07-24 Thread Mrinal Pandey
Add a blank line after variable declarations as suggested by checkpatch. Signed-off-by: Mrinal Pandey --- drivers/android/binder.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/android/binder.c b/drivers/android/binder.c index 3cf13ff16934..75832f236bf9 100644 --- a/drivers

[PATCH 6/6] drivers: android: Fix the SPDX comment style

2020-07-24 Thread Mrinal Pandey
C source files should have `//` as SPDX comment and not `/**/`. Fix this by running checkpatch on the file. Signed-off-by: Mrinal Pandey --- drivers/android/binderfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/android/binderfs.c b/drivers/android/binderfs.c

[PATCH 3/6] drivers: android: Remove braces for a single statement if-else block

2020-07-24 Thread Mrinal Pandey
Remove braces for both if and else block as suggested by checkpatch. Signed-off-by: Mrinal Pandey --- drivers/android/binder.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/android/binder.c b/drivers/android/binder.c index 5fdf982ec83b..3cf13ff16934 100644

[PATCH 2/6] drivers: android: Remove the use of else after return

2020-07-24 Thread Mrinal Pandey
Remove the unnecessary else branch after return statement as suggested by checkpatch. Signed-off-by: Mrinal Pandey --- drivers/android/binder.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/android/binder.c b/drivers/android/binder.c index f50c5f182bb5

[PATCH 1/6] drivers: android: Fix a variable declaration coding style issue

2020-07-24 Thread Mrinal Pandey
Add a blank line after variable declarations as suggested by checkpatch. Signed-off-by: Mrinal Pandey --- drivers/android/binder_alloc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/android/binder_alloc.c b/drivers/android/binder_alloc.c index 42c672f1584e..0294cef7402c 100644

[PATCH] staging: rtl8188eu: Fix an indent coding style issue

2020-07-22 Thread Mrinal Pandey
Only a single tab space is required after the if statement. Fix this issue by running scripts/checkpatch.pl on the file. Signed-off-by: Mrinal Pandey --- drivers/staging/rtl8188eu/core/rtw_recv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu

[PATCH v2] checkpatch: Fix the usage of capture group ( ... )

2020-07-13 Thread Mrinal Pandey
in the condition before `&&`. Thus, `$1` can be initialized to the text that capture group matches thereby setting it to the desired and required value. Reviewed-by: Lukas Bulwahn Tested-by: Lukas Bulwahn Signed-off-by: Mrinal Pandey --- Changes since v1: Add Reviewed-by and Tested-by

[PATCH] checkpatch: Fix the usage of capture group ( ... )

2020-07-12 Thread Mrinal Pandey
in the condition before `&&`. Thus, `$1` can be initialized to the text that capture group matches thereby setting it to the desired and required value. Signed-off-by: Mrinal Pandey --- scripts/checkpatch.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/