[PATCH 33/33] Staging: android: ion: ion.c: Compression of lines for

2016-07-30 Thread Nadim almas
This patch compresses two lines in to a single line in file rtw_android.c if immediate return statement is found. It also removes variable bytes_written as it is no longer needed. It is done using script Coccinelle. And coccinelle uses following semantic patch for this compression function: @@

[PATCH 33/33] Staging: android: ion: ion.c: Compression of lines for

2016-07-30 Thread Nadim almas
This patch compresses two lines in to a single line in file rtw_android.c if immediate return statement is found. It also removes variable bytes_written as it is no longer needed. It is done using script Coccinelle. And coccinelle uses following semantic patch for this compression function: @@

Re: [PATCH 33/33] Staging: android: ion: ion.c: Compression of lines for

2016-07-30 Thread Joe Perches
On Sat, 2016-07-30 at 02:54 -0700, Nadim almas wrote: > This patch compresses two lines in to a single line in file rtw_android.c > if immediate return statement is found. It also removes variable  > bytes_written as it is no longer needed. > > It is done using script Coccinelle. And coccinelle us

[PATCH 33/33] Staging: android: ion: ion.c: Compression of lines for

2016-07-30 Thread Nadim almas
This patch compresses two lines in to a single line in file rtw_android.c if immediate return statement is found. It also removes variable bytes_written as it is no longer needed. It is done using script Coccinelle. And coccinelle uses following semantic patch for this compression function: @@

[PATCH] Staging: rts5208: fix double blank line coding style issues

2016-07-30 Thread Jonas Rickert
This is a patch for double blank lines and a missing blank line reported by checkpatch.pl Signed-off-by: Jonas Rickert --- drivers/staging/rts5208/ms.c | 11 --- drivers/staging/rts5208/ms.h | 1 - drivers/staging/rts5208/rtsx.c | 7 --- drivers/st

[PATCH 0/1] drivers: staging: android: Remove checkpatch.pl-TODO

2016-07-30 Thread Matthias Beyer
Hi Arve, Hi Riley, I just noticed that the drivers/staging/android driver has only 4 remaining checkpatch.pl warnings, two of them beeing 80-character-per-line warnings (the two lines are 81 characters exactly). The other warnings are: ./sw_sync.c:258: WARNING: memory barrier without comment

[PATCH 1/1] Remove checkpatch.pl cleanups todo

2016-07-30 Thread Matthias Beyer
Signed-off-by: Matthias Beyer --- drivers/staging/android/TODO | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO index 64d8c87..4ce0ac6 100644 --- a/drivers/staging/android/TODO +++ b/drivers/staging/android/TODO @@ -1,5 +1,4 @@ TODO:

[PATCH 2/2] net: ethernet: octeon: use phy_ethtool_{get|set}_link_ksettings

2016-07-30 Thread Philippe Reynes
There are two generics functions phy_ethtool_{get|set}_link_ksettings, so we can use them instead of defining the same code in the driver. There was a check on CAP_NET_ADMIN in cvm_oct_set_settings, but this check is already done in dev_ethtool, so no need to repeat it before calling the generic f

[PATCH 1/2] net: ethernet: octeon: use phydev from struct net_device

2016-07-30 Thread Philippe Reynes
The private structure contain a pointer to phydev, but the structure net_device already contain such pointer. So we can remove the pointer phydev in the private structure, and update the driver to use the one contained in struct net_device. Signed-off-by: Philippe Reynes --- drivers/staging/octe

Re: [PATCH 33/33] Staging: android: ion: ion.c: Compression of lines for

2016-07-30 Thread Markus Böhme
On 07/30/2016 01:12 PM, Nadim almas wrote: > This patch compresses two lines in to a single line in file rtw_android.c > if immediate return statement is found. It also removes variable > bytes_written as it is no longer needed. This commit message is wrong. Neither are you editing rtw_android.c,

[PATCH 33/33] Staging: android: ion: ion.c: Compression of lines for

2016-07-30 Thread Nadim almas
This patch compresses two lines in to a single line in file ion.c if immediate return statement is found. It also removes variable bytes_written as it is no longer needed. It is done using script Coccinelle. And coccinelle uses following semantic patch for this compression function: @@ expressi

[PATCH] staging: comedi/drivers: Fixed sparse warnings

2016-07-30 Thread Nayeemahmed Badebade
Fixed below sparse warnings in dt2811.c warning: symbol 'dt2811_clk_dividers' was not declared. Should it be static? warning: symbol 'dt2811_clk_multipliers' was not declared. Should it be static? Signed-off-by: Nayeemahmed Badebade --- drivers/staging/comedi/drivers/dt2811.c | 4 ++-- 1 file

[PATCH] staging: vt6656: Fix block comment style

2016-07-30 Thread Edward Lipinsky
This patch fixes the checkpatch.pl warnings: WARNING: Block comments use * on subsequent lines WARNING: Block comments use a trailing */ on a separate line Signed-off-by: Edward Lipinsky --- drivers/staging/vt6656/card.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a

[PATCH 34/34] Staging: comedi: drivers: cd_pcidas: Compression of

2016-07-30 Thread Nadim almas
This patch compresses two lines in to a single line in file cb_pcidas.c if immediate return statement is found. It also removes variable bytes_written as it is no longer needed. It is done using script Coccinelle. And coccinelle uses following semantic patch for this compression function: @@ expr

Re: [PATCH 33/33] Staging: android: ion: ion.c: Compression of lines for

2016-07-30 Thread Laura Abbott
On 07/30/2016 09:11 AM, Nadim almas wrote: This patch compresses two lines in to a single line in file ion.c if immediate return statement is found. It also removes variable bytes_written as it is no longer needed. This still says bytes_written. None of the patches after your first one apply