From: wengjianfeng
in st_nci_spi_write function, first assign a value to a variable then
goto exit label. return statement just follow the label and exit label
just used once, so we should directly return and remove exit label.
Signed-off-by: wengjianfeng
---
drivers/nfc/st-nci/spi.c | 7
From: wengjianfeng
In many places,first assign a value to a variable and then return
the variable. which is redundant, we should directly return the value.
in pn533_rf_field funciton,return rc also in the if statement, so we
use return 0 to replace the last return rc.
Signed-off-by:
From: wengjianfeng
In many places,first assign a value to a variable and then return
the variable. which is redundant, we should directly return the value.
in pn533_rf_field funciton,return statement in the if statement is
redundant, we just delete it.
Signed-off-by: wengjianfeng
---
From: wengjianfeng
change 'purpous' to 'purpose'.
change 'frequecy' to 'frequency'.
remove redundant words struct and enum.
Signed-off-by: wengjianfeng
---
drivers/media/dvb-frontends/drx39xyj/drxj.h | 35 +++--
1 file changed, 18 insertions(+), 17 deletions(-)
diff
From: wengjianfeng
In function fdp_nci_patch_otp and fdp_nci_patch_ram,many goto
out statements are used, and out label just return variable r.
in some places,just jump to the out label, and in other places,
assign a value to the variable r,then jump to the out label.
It is unnecessary, we just
From: wengjianfeng
some function's label meaningless, the label statement follows
the goto statement, no other statements, so just remove it.
Reported-by: kernel test robot
Signed-off-by: wengjianfeng
---
drivers/net/wireless/quantenna/qtnfmac/commands.c | 67 ---
1 file
From: wengjianfeng
some function's label meaningless, the return statement follows
the goto statement, so just remove it.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/quantenna/qtnfmac/cfg80211.c | 27 +--
1 file changed, 6 insertions(+), 21 deletions(-)
diff
From: wengjianfeng
In function s3fwrn5_nci_post_setup, the variable ret is assigned then
goto out label, which just return ret, so we use return to replace it.
Other goto sentences are similar, we use return sentences to replace
goto sentences and delete out label.
Signed-off-by: wengjianfeng
From: wengjianfeng
In function s3fwrn5_nci_post_setup, The variable ret is assigned to 0,
then goto out label, but just return ret in out label, so we use
return 0 to replace it. and other goto sentences are similar, we use
return sentences to replace it and delete out label.
Signed-off-by:
From: wengjianfeng
change 'freqency' to 'frequecy'
Signed-off-by: wengjianfeng
---
include/uapi/sound/asoc.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/uapi/sound/asoc.h b/include/uapi/sound/asoc.h
index da61398..0066eee 100644
---
From: wengjianfeng
change 'addres' to 'address'
Signed-off-by: wengjianfeng
---
drivers/gpu/drm/amd/pm/powerplay/smumgr/smu7_smumgr.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/pm/powerplay/smumgr/smu7_smumgr.c
From: wengjianfeng
change 'lenth' to 'length'.
Signed-off-by: wengjianfeng
---
drivers/scsi/lpfc/lpfc_debugfs.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/scsi/lpfc/lpfc_debugfs.c b/drivers/scsi/lpfc/lpfc_debugfs.c
index 8c23806..658a962 100644
---
From: wengjianfeng
some function's label meaningless, the label statement follows
the goto statement, no other statements, so just remove it.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/quantenna/qtnfmac/commands.c | 68 ---
1 file changed, 68 deletions(-)
diff
From: wengjianfeng
some function's label meaningless, the return statement follows
the goto statement, so just remove it.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/quantenna/qtnfmac/cfg80211.c | 27 +--
1 file changed, 6 insertions(+), 21 deletions(-)
diff
From: wengjianfeng
The variable r is defined at the beginning and initialized
to 0 until the function returns r, and the variable r is
not reassigned.Therefore, we do not need to define the
variable r, just return 0 directly at the end of the function.
Signed-off-by: wengjianfeng
---
From: wengjianfeng
INVALID_QUEUE has been used as a return value,it is not necessary to
assign it to q_num,so just return INVALID_QUEUE.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/rsi/rsi_91x_core.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
From: wengjianfeng
define refilled and then assign value to it, which should do
that at the same time.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/marvell/mwl8k.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/wireless/marvell/mwl8k.c
From: wengjianfeng
-ENOMEM has been used as a return value,it is not necessary to
assign it, and if kzalloc fail,not need free it,so just return
-ENOMEM when kzalloc fail.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/ti/wl1251/cmd.c | 36
1 file
From: wengjianfeng
remove duplicate word 'we' in comment
change 'then' to 'than' in comment
Signed-off-by: wengjianfeng
---
drivers/net/wireless/ralink/rt2x00/rt2x00crypto.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/ralink/rt2x00/rt2x00crypto.c
From: wengjianfeng
change 'regster' to 'register'.
Signed-off-by: wengjianfeng
---
kernel/debug/gdbstub.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/debug/gdbstub.c b/kernel/debug/gdbstub.c
index a77df59..e149a0a 100644
--- a/kernel/debug/gdbstub.c
+++
From: wengjianfeng
remove duplicate word 'we' in comment
Signed-off-by: wengjianfeng
---
drivers/net/wireless/ralink/rt2x00/rt2x00crypto.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/ralink/rt2x00/rt2x00crypto.c
From: wengjianfeng
define refilled and then assign value to it, which should do
that at the same time.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/marvell/mwl8k.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/wireless/marvell/mwl8k.c
From: wengjianfeng
change 'piority' to 'priority'
change 'succesfult' to 'successful'
Signed-off-by: wengjianfeng
---
drivers/nfc/pn533/pn533.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/nfc/pn533/pn533.c b/drivers/nfc/pn533/pn533.c
index f7464bd..f1469ac
From: wengjianfeng
change 'regster' to 'register'.
Signed-off-by: wengjianfeng
---
kernel/debug/gdbstub.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/debug/gdbstub.c b/kernel/debug/gdbstub.c
index a77df59..e149a0a 100644
--- a/kernel/debug/gdbstub.c
+++
From: wengjianfeng
INVALID_QUEUE has been used as a return value,it is not necessary to
assign it to q_num,so just return INVALID_QUEUE.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/rsi/rsi_91x_core.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
From: wengjianfeng
-ENOMEM has been used as a return value,it is not necessary to
assign it, and if kzalloc fail,not need free it,so just return
-ENOMEM when kzalloc fail.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/ti/wl1251/cmd.c | 36
1 file
From: wengjianfeng
at first, ret was assigned to zero, but later assigned to
a funciton,so the assignment to zero is no use, which can
simple be removed instead.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c | 2 --
1 file changed, 2 deletions(-)
diff
From: wengjianfeng
-ENOMEM has been used as a return value,it is not necessary to
assign it, and if kzalloc fail,not need free it,so just return
-ENOMEM when kzalloc fail.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/ti/wl1251/cmd.c | 36
1 file
From: wengjianfeng
define refilled and then assign value to it, which should do
at the same time.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/marvell/mwl8k.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/wireless/marvell/mwl8k.c
From: wengjianfeng
define ret and then assign value to it, which we should do one time.
Signed-off-by: wengjianfeng
---
drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git
From: wengjianfeng
If the length of hide ssid is zero in scan, don't pass
it to driver, which doesn't make any sense.
Signed-off-by: wengjianfeng
---
net/wireless/nl80211.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c
index
From: wengjianfeng
change 'paquet' to 'packet'
Signed-off-by: wengjianfeng
---
drivers/nfc/fdp/i2c.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nfc/fdp/i2c.c b/drivers/nfc/fdp/i2c.c
index ad0abb1..adaa1a7 100644
--- a/drivers/nfc/fdp/i2c.c
+++
From: wengjianfeng
change 'regster' to 'register'
Signed-off-by: wengjianfeng
---
drivers/nfc/trf7970a.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nfc/trf7970a.c b/drivers/nfc/trf7970a.c
index c70f62fe..3397802 100644
--- a/drivers/nfc/trf7970a.c
+++
From: wengjianfeng
change 'regster' to 'register'
Signed-off-by: wengjianfeng
---
drivers/nfc/trf7970a.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nfc/trf7970a.c b/drivers/nfc/trf7970a.c
index c70f62fe..3397802 100644
--- a/drivers/nfc/trf7970a.c
+++
34 matches
Mail list logo