As reported by David Binderman, this test is useless as of
if (a < 3) {
/* ... */
} else if (a >= 3) {
/* ... */
}
so this patch removes the second check.
Matthias Beyer (1):
drivers: staging: rtl8723au: hal: Remove pointless test
drivers/staging/rtl8723
This patch removes the pointless `else if` test.
Signed-off-by: Matthias Beyer
Reported-by: David Binderman
---
drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c
b/drivers
Pinging here as nobody responded yet.
Maybe this was overlooked.
On 23-08-2016 21:44:16, Matthias Beyer wrote:
> As reported by David Binderman, this test is useless as of
>
> if (a < 3) {
> /* ... */
> } else if (a >= 3) {
> /* ... */
> }
Re-sending as
On 01-09-2016 17:54:47, Greg KH wrote:
> There is no patch here :(
8<
As reported by David Binderman, this test is useless as of
if (a < 3) {
/* ... */
} else if (a >= 3) {
/* ... */
}
so this patch removes the second check.
Matth
This patch removes the pointless `else if` test.
Signed-off-by: Matthias Beyer
Reported-by: David Binderman
---
drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c
b/drivers
This patchset fixes some errors and warnings reported by checkpatch.pl.
Matthias Beyer (5):
drivers: staging: rtl8723au: core: Fix checkpatch.pl errors
drivers: staging: rtl8723au: core: simplify if-break-else
drivers: staging: rtl8723au: core: Refactor pointless branching
drivers
This patch fixes the ERRORs which are reported from the checkpatch.pl
script for this file.
Signed-off-by: Matthias Beyer
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c
b
As the if statement breaks/continues the loop, the else block is not
useful and can be omitted.
Signed-off-by: Matthias Beyer
Signed-off-by: Manuel Messner
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 25 +++--
1 file changed, 11 insertions(+), 14 deletions(-)
diff
As ether_addr_equal returns a bool, we can simply use the return value
to set the variable.
Signed-off-by: Matthias Beyer
Signed-off-by: Manuel Messner
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/staging
This patch fixes the "space prohibited between function name and open
parenthesis '('" warning from checkpatch.pl
Signed-off-by: Matthias Beyer
Signed-off-by: Manuel Messner
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 10 +-
1 file changed, 5 insertions(+),
Signed-off-by: Matthias Beyer
Signed-off-by: Manuel Messner
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c
b/drivers/staging/rtl8723au/core/rtw_mlme.c
index
On 06-09-2016 12:00:37, Jes Sorensen wrote:
> Matthias Beyer writes:
> > This patchset fixes some errors and warnings reported by checkpatch.pl.
> >
> > Matthias Beyer (5):
> > drivers: staging: rtl8723au: core: Fix checkpatch.pl errors
> > drivers: staging: r
Reviewed-by: tags, but I would like to apply
them on my kernel source copy first, just to re-check them!
--
Mit freundlichen Grüßen,
Kind regards,
Matthias Beyer
Proudly sent with mutt.
Happily signed with gnupg.
pgpSDKDxKgmEC.pgp
Description: PGP signature
13 matches
Mail list logo