From: Bo Yu
Fix sparse warnings:
drivers/staging/erofs/utils.c:134:6: warning: symbol
'erofs_try_to_release_workgroup' was not declared. Should it be static?
Fixes: 51232df5e4b26("staging: erofs: fix race when the managed cache is
enabled")
Cc: Gao Xiang
Cc: Chao Yu
CC
From: Bo Yu
Compiling the kernel with W=1 results in the following warning:
drivers/staging/ks7010/ks_hostif.c:465:6: warning: variable ‘mib_val_type’
set but not used [-Wunused-but-set-variable]
u16 mib_val_type;
drivers/staging/ks7010/ks_hostif.c:464:6: warning: variable ‘mib_val_size’
set
From: Bo Yu
Compiling the kernel with W=1 results in the following warning:
drivers/staging/ks7010/ks_hostif.c:465:6: warning: variable ‘mib_val_type’
set but not used [-Wunused-but-set-variable]
u16 mib_val_type;
drivers/staging/ks7010/ks_hostif.c:464:6: warning: variable ‘mib_val_size’
set
From: Bo Yu
Compiling the kernel with W=1 results in the following warning:
drivers/staging/ks7010/ks_hostif.c:465:6: warning: variable ‘mib_val_type’
set but not used [-Wunused-but-set-variable]
u16 mib_val_type;
drivers/staging/ks7010/ks_hostif.c:464:6: warning: variable ‘mib_val_size’
set
/host_interface.c:451:25:got unsigned short
[usertype] capability
Signed-off-by: Bo YU
---
I have no hardware to test it and just to compile it
---
drivers/staging/wilc1000/host_interface.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wilc1000
:got struct
cfg80211_bss_ies const [noderef] *ies
Signed-off-by: Bo YU
---
drivers/staging/wilc1000/host_interface.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wilc1000/host_interface.c
index cdcb52aec779
:25:got unsigned short
[usertype] capability
Signed-off-by: Bo YU
---
V2: use cpu_to_le16 assign valid type according to Ajay's suggestions
---
drivers/staging/wilc1000/host_interface.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wil
struct
cfg80211_bss_ies const [noderef] *ies
Signed-off-by: Bo YU
---
V2: Adding cc wireless-list
---
drivers/staging/wilc1000/host_interface.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wilc1000
There maybe cause potential string overflow issue due to use
strcpy without checking the length
Detected By CoversityScan CID# 1444760
Fixes: 131ac62253dba:(staging: most: core: use device description as name)
Signed-off-by: Bo YU
---
drivers/staging/most/core.c | 2 +-
1 file changed, 1
On Wed, Apr 24, 2019 at 10:55 PM Dan Carpenter wrote:
>
> On Mon, Apr 22, 2019 at 10:20:18PM -0400, Bo YU wrote:
> > There maybe cause potential string overflow issue due to use
> > strcpy without checking the length
> >
> > Detected By CoversityScan CID# 144476
Statements should start on a tabstop
Signed-off-by: YU Bo
---
drivers/staging/wlan-ng/prism2mgmt.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/prism2mgmt.c
b/drivers/staging/wlan-ng/prism2mgmt.c
index c558ad656c49..b641a59052ac 100644
--- a/
WARNING: Block comments should align the * on each line
Signed-off-by: Bo YU
---
drivers/staging/wlan-ng/p80211metastruct.h | 88 ++--
1 file changed, 44 insertions(+), 44 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211metastruct.h
b/drivers/staging/wlan-ng
Missing a blank line after declarations
Signed-off-by: Bo YU
---
drivers/staging/bcm2835-audio/bcm2835-pcm.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/bcm2835-audio/bcm2835-pcm.c
b/drivers/staging/bcm2835-audio/bcm2835-pcm.c
index 014bf7ab69f4..16127e062661 100644
Right-shift a space key to align the * on each line.
Signed-off-by: Bo YU
---
drivers/staging/wlan-ng/p80211metastruct.h | 88 ++--
1 file changed, 44 insertions(+), 44 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211metastruct.h
b/drivers/staging/wlan-ng
The following Coccinelle script was used to detect this:
@@
local idexpression ret;
expression e;
@@
-ret =
+return
e;
-return ret;
Signed-off-by: Bo YU
---
drivers/staging/iio/meter/ade7754.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging
Fixed checkpatch warning by adding a blank line after declare
expression
Signed-off-by: Bo YU
---
drivers/staging/speakup/serialio.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/speakup/serialio.c
b/drivers/staging/speakup/serialio.c
index 969373201356..8d2f7c672cc6
As checkpatch.pl reported, i fixed these complaint one by one. Maybe
patch 04 depend on patch 02.
Thanks!
Bo YU (4):
staging: speakup: add a missing blank line after declaration
staging: speakup: add a space around '|'
staging: speakup: in serialio.c no over 80 chars long
Add a space around logical symbol '|' to wipe out checkpatch check
Signed-off-by: Bo YU
---
drivers/staging/speakup/serialio.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/speakup/serialio.c
b/drivers/staging/speakup/serialio.c
index 8d
I have aligned argument with parenthesis, so checkpatch no check also.
Signed-off-by: Bo YU
---
drivers/staging/speakup/serialio.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/speakup/serialio.c
b/drivers/staging/speakup/serialio.c
index 00b25d3591d2
Fixed the checkpatch.pl warning:
WARNING: line over 80 characters
Signed-off-by: Bo YU
---
drivers/staging/speakup/serialio.c |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/speakup/serialio.c
b/drivers/staging/speakup/serialio.c
index f38eb66943bf
On Mon, Mar 21, 2016 at 03:38:21PM -0400, Kroah-Hartman wrote:
hi,
On Sat, Mar 12, 2016 at 03:40:35AM -0500, YU Bo wrote:
Fix comments to use trailing */ on separste lines.
Signed-off-by: YU BO
---
drivers/staging/xgifb/vb_init.c|3 ++-
drivers/staging/xgifb/vb_setmode.c |3 ++-
2 fi
This is a patch to the comedilib.h file that fixes "bloch comment use
* on subsequent lines" warning found by the checkpatch.pl tool
Signed-off-by: YU Bo
---
drivers/staging/comedi/comedilib.h | 32
1 个文件被修改,插入 16 行(+),删除 16 行(-)
diff --git a/drivers/staging
This is a patch to the comedi_pcmcia.h file that fixes up a "line over
80 characters" warning found by the checkpatch.pl tool
Signed-off-by: YU Bo
---
drivers/staging/comedi/comedi_pcmcia.h |3 ++-
1 个文件被修改,插入 2 行(+),删除 1 行(-)
diff --git a/drivers/staging/comedi/comedi_pcmcia.h
b/drivers
Hi,
Sorry for the delay to reply.I have send the patch to Arnaud,but he
didn't reply me...Maybe i was wrong with something.
Thanks again.
On Mon, Feb 8, 2016 at 12:14 PM, Kroah-Hartman
wrote:
> On Fri, Jan 15, 2016 at 05:30:02PM +0800, YU Bo wrote:
>> This patch is to vgatypes.h file that fixes
Fix comments to use trailing */ on separate lines.
Signed-off-by: YU BO
---
drivers/staging/xgifb/vb_init.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/xgifb/vb_init.c b/drivers/staging/xgifb/vb_init.c
index 26b539b..c7f02c7 100644
--- a/dri
Hello,
On Sat, Feb 20, 2016 at 03:38:15PM -0800, Kroah-Hartman wrote:
>On Wed, Feb 17, 2016 at 02:53:34PM +0800, Bo YU wrote:
>> Fix comments to use trailing */ on separate lines.
>>
>> Signed-off-by: YU BO
>> ---
>> drivers/staging/xgifb/vb_init.c | 10
Hi,
On Sun, Feb 21, 2016 at 11:33:48AM -0800, Kroah-Hartman wrote:
>On Sun, Feb 21, 2016 at 04:59:17PM +0800, Bo YU wrote:
>> Hello,
>>
>> On Sat, Feb 20, 2016 at 03:38:15PM -0800, Kroah-Hartman wrote:
>> >Patch doesn't apply :(
>> I don't know th
tell
me as if my git configure that have problems.
When i run ` git am patch` it reports errors.There is a problem
Thanks a lot.
> Regards:
> Roger H. Newell
>
> On Sun, Feb 21, 2016 at 5:29 AM, Bo YU wrote:
>> Hello,
>>
>> On Sat, Feb 20, 2016 at 03:38:15PM -0800
28 matches
Mail list logo