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
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
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
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
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
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
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
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
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
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
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
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/
12 matches
Mail list logo