Re: [PATCH] staging: android: ashmem: Declared file operation with const keyword

Hi Thais, Thank you for the patch! Yet something to improve: [auto build test ERROR on staging/staging-testing] url: https://github.com/0day-ci/linux/commits/Thais-Camacho/staging-android-ashmem-Declared-file-operation-with-const-keyword/20210214-103440 base:

[PATCH] staging: android: ashmem: Declared file operation with const keyword

Warning found by checkpatch.pl script Signed-off-by: Thais Camacho --- drivers/staging/android/ashmem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ashmem.c b/drivers/staging/android/ashmem.c index 4789d36ddfd3..2cb665e0228c 100644 ---

[PATCH] fixed coding style warning

removed unnecessary out of memory message Signed-off-by: Ashish Vara --- drivers/staging/gasket/gasket_page_table.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/gasket/gasket_page_table.c b/drivers/staging/gasket/gasket_page_table.c index 6f6273c83822..2dbf3d9b8f34

[PATCH] staging: wfx: logical continuations should be on the previous line

Move logical and operator to previous line to comply with the standard kernel coding style. Signed-off-by: Zhuoran He --- drivers/staging/wfx/hif_rx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wfx/hif_rx.c b/drivers/staging/wfx/hif_rx.c index

Re: [PATCH] staging: wfx: logical continuations should be on the previous line

On Sun, Feb 14, 2021 at 05:16:41AM +0800, zhuo1a...@outlook.com wrote: > Move logical or operator to previous line to comply with > the standard kernel coding style. > > Signed-off-by: Zhuoran He > --- > drivers/staging/wfx/hif_rx.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) >

[PATCH] staging: wfx: logical continuations should be on the previous line

Move logical or operator to previous line to comply with the standard kernel coding style. Signed-off-by: Zhuoran He --- drivers/staging/wfx/hif_rx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wfx/hif_rx.c b/drivers/staging/wfx/hif_rx.c index

[PATCH] staging: rtl8192e: remove multiple blank lines

This patch removes some blank lines in order to fix a checkpatch issue. Signed-off-by: William Durand --- This is my very first patch. drivers/staging/rtl8192e/rtllib_wx.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/rtl8192e/rtllib_wx.c

[PATCH] staging: wfx: logical continuations should be on the previous line

Move logical or operator to previous line to comply with the standard kernel coding style. Signed-off-by: Zhuoran He --- drivers/staging/wfx/hif_rx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wfx/hif_rx.c b/drivers/staging/wfx/hif_rx.c index

[PATCH] staging:r8188eu: use IEEE80211_FCTL_* kernel definitions

_TO_DS_, _FROM_DS_, _MORE_FRAG_, _RETRY_, _PWRMGT_, _MORE_DATA_, _PRIVACY_, _ORDER_ definitions are duplicate IEEE80211_FCTL_* kernel definitions. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/include/wifi.h | 51 ++-- 1 file changed, 21 insertions(+), 30

[PATCH] staging:wlan-ng: use memdup_user instead of kmalloc/copy_from_user

memdup_user() is shorter and safer equivalent of kmalloc/copy_from_user pair. Signed-off-by: Ivan Safonov --- drivers/staging/wlan-ng/p80211netdev.c | 28 -- 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/drivers/staging/wlan-ng/p80211netdev.c

[PATCH]: staging: hikey9xx: Fix alignment of function parameters

This patch fixes the following checkpatch.pl check: CHECK: Alignment should match open parenthesis Signed-off-by: Mukul Mehar --- drivers/staging/hikey9xx/hi6421-spmi-pmic.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hikey9xx/hi6421-spmi-pmic.c