Signed-off-by: Antoine Schweitzer-Chaput
---
drivers/staging/imx-drm/ipu-v3/ipu-common.c | 6 --
drivers/staging/imx-drm/ipu-v3/ipu-dc.c | 3 ++-
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/imx-drm/ipu-v3/ipu-common.c
b/drivers/staging/imx-drm/ipu-v3/ip
On Sat, Apr 19, 2014 at 1:23 AM, Greg KH wrote:
> On Fri, Apr 18, 2014 at 07:57:51PM +0200, Bojan Prtvar wrote:
>> This fixes the following sparse warnings:
>> drivers/staging/crystalhd/crystalhd_lnx.c:631:5: warning: symbol
>> 'chd_dec_pci_suspend' was not declared. Should it be static?
>> drive
On Sat, Apr 19, 2014 at 9:58 AM, bojan prtvar wrote:
> On Sat, Apr 19, 2014 at 1:23 AM, Greg KH wrote:
>> On Fri, Apr 18, 2014 at 07:57:51PM +0200, Bojan Prtvar wrote:
>>> This fixes the following sparse warnings:
>>> drivers/staging/crystalhd/crystalhd_lnx.c:631:5: warning: symbol
>>> 'chd_dec_
On Thu, Apr 17, 2014 at 04:31:44PM +0300, Alexandr Terekhov wrote:
> Fix naked sscanf
>
> Signed-off-by: Alexandr Terekhov
Sign off with the email you use to send patches so we can at least try
to verify that the are real.
>
> diff --git a/drivers/staging/gdm72xx/gdm_wimax.c
> b/drivers/stag
All code in this file is encapsulated in a useless #if 1
declaration, remove it.
Signed-off-by: Mark Einon
---
drivers/staging/rtl8821ae/btcoexist/halbtc8192e2ant.c |5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8821ae/btcoexist/halbtc8192e2ant.c
b
Fixes all checkpatch errors reporting that statics shouldn't be initialised
to their defualt values.
Signed-off-by: Mark Einon
---
drivers/staging/rtl8821ae/btcoexist/halbtc8192e2ant.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/rtl8821ae/btco
We set btrssi to zero and then immediately assign it. Assign it
on declaration instead.
Signed-off-by: Mark Einon
---
drivers/staging/rtl8821ae/btcoexist/halbtc8192e2ant.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8821ae/btcoexist/halbtc8192e2an
Fix checkpatch issues relating to using leading spaces instead of tabs.
Whitespace changes only.
Signed-off-by: Mark Einon
---
.../staging/rtl8821ae/btcoexist/halbtc8192e2ant.c | 42 ++--
1 file changed, 21 insertions(+), 21 deletions(-)
diff --git a/drivers/staging/rtl8821ae
Sparse complains about halbtc8192e2ant_low_penalty_ra() not being used,
and subsequently halbtc8192e2ant_set_sw_penalty_tx_rateadaptive() when
this is removed. remove both functions.
Signed-off-by: Mark Einon
---
.../staging/rtl8821ae/btcoexist/halbtc8192e2ant.c | 48
1 f
In halbtc8192e2ant.c, fix all instances of the sparse warning:
warning: symbol 'xxx' was not declared. Should it be static?
Signed-off-by: Mark Einon
---
.../staging/rtl8821ae/btcoexist/halbtc8192e2ant.c | 218 ++--
1 file changed, 114 insertions(+), 104 deletions(-)
diff --g
Fixes checkpatch error:
ERROR: switch and case should be at the same indent
+ switch (coex_dm->cur_retrylimit_type) {
+ case 0: /* normal mode */
[...]
+ case 1: /* retry limit=8 */
[...]
+ default:
Signed-off-by: Mar
Whitespace changes only to appease checkpatch warnings.
Signed-off-by: Mark Einon
---
.../staging/rtl8821ae/btcoexist/halbtc8192e2ant.c | 100 ++--
1 file changed, 50 insertions(+), 50 deletions(-)
diff --git a/drivers/staging/rtl8821ae/btcoexist/halbtc8192e2ant.c
b/drivers/s
Remove functions not being called from anywhere:
halbtc8192e2ant_set_adc_backoff()
halbtc8192e2ant_adc_backoff()
Signed-off-by: Mark Einon
---
.../staging/rtl8821ae/btcoexist/halbtc8192e2ant.c | 38
1 file changed, 38 deletions(-)
diff --git a/drivers/staging/rtl8821ae/
This patch removes the following checkpatch errors:
rtl8188eu/core/rtw_p2p.c:
- 1508: ERROR: code indent should use tabs where possible
- 1509: ERROR: code indent should use tabs where possible
- 1510: ERROR: code indent should use tabs where possible
rtl8188eu/core/rtw_recv.c
- 18
This commit fixes the following checkpatch errors:
rtl8188eu/core/rtw_p2p.c
- 1508: ERROR: code indent should use tabs where possible
- 1509: ERROR: code indent should use tabs where possible
- 1510: ERROR: code indent should use tabs where possible
Signed-off-by: Konrad Zapalowicz
-
This commit fixes the following checkpatch errors:
rtl8188eu/core/rtw_recv.c
- 1874: ERROR: code indent should use tabs where possible
Signed-off-by: Konrad Zapalowicz
---
drivers/staging/rtl8188eu/core/rtw_recv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/s
A sparse error fixup removed a htons() which is required for the driver
to function. Put the htons() back.
Signed-off-by: Sean MacLennan
---
diff --git a/drivers/staging/rtl8192e/rtllib_tx.c
b/drivers/staging/rtl8192e/rtllib_tx.c
index 11d0a9d..3e79bff 100644
--- a/drivers/staging/rtl8192e/rtlli
Fix a sparse error in the htons() call. htons() returns a __be16 not a
u16.
Signed-off-by: Sean MacLennan
---
diff --git a/drivers/staging/rtl8192e/rtllib_tx.c
b/drivers/staging/rtl8192e/rtllib_tx.c
index 3e79bff..b7dd153 100644
--- a/drivers/staging/rtl8192e/rtllib_tx.c
+++ b/drivers/staging/rt
On Sat, Apr 19, 2014 at 02:12:34PM -0400, Sean MacLennan wrote:
> Fix a sparse error in the htons() call. htons() returns a __be16 not a
> u16.
>
> Signed-off-by: Sean MacLennan
> ---
You sent 2 patches, which should I apply?
I've deleted both from my inbox, please resend the proper one, with a
On Sat, 19 Apr 2014 11:17:23 -0700
Greg KH wrote:
> On Sat, Apr 19, 2014 at 02:12:34PM -0400, Sean MacLennan wrote:
> > Fix a sparse error in the htons() call. htons() returns a __be16
> > not a u16.
> >
> > Signed-off-by: Sean MacLennan
> > ---
>
> You sent 2 patches, which should I apply?
>
On Sat, Apr 19, 2014 at 03:10:04PM -0400, Sean MacLennan wrote:
> On Sat, 19 Apr 2014 11:17:23 -0700
> Greg KH wrote:
>
> > On Sat, Apr 19, 2014 at 02:12:34PM -0400, Sean MacLennan wrote:
> > > Fix a sparse error in the htons() call. htons() returns a __be16
> > > not a u16.
> > >
> > > Signed-o
On Fri, Apr 18, 2014 at 04:09:58PM -0700, Greg KH wrote:
> This patch does not apply, can you refresh it, and the 4/4 patch,
> against my latest tree and resend?
Sending them now as v3. I'm not sure a v3 is most fitting for this
situation, so please tell me if you want me to resend them in a
differ
Update the third and fourth v2 patches to apply cleanly on greg's
current tree.
Guido Martínez (2):
staging: vt6655: remove dead code
staging: vt6655: fix checkpatch bracing issues
drivers/staging/vt6655/IEEE11h.c | 6 -
drivers/staging/vt6655/baseband.c| 101 -
drivers/sta
A sparse error fixup removed a htons() which is required for the driver
to function. This patch puts the htons() back and fixes the sparse
warning correctly by changing the left side cast.
Signed-off-by: Sean MacLennan
---
diff --git a/drivers/staging/rtl8192e/rtllib_tx.c
b/drivers/staging/rtl81
From: "Son P. Nguyen"
Fixed coding style of dgnc_pci_tbl definition (longer than 80 chars)
Signed-off-by: Son P. Nguyen
---
drivers/staging/dgnc/dgnc_driver.c | 36 +++-
1 file changed, 19 insertions(+), 17 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_dr
From: "Son P. Nguyen"
Fixed incorrect braces (coding style)
PS: This is for task 10 of eudyptula-challenge.
Signed-off-by: Son P. Nguyen
---
drivers/staging/dgnc/dgnc_driver.c | 17 ++---
1 file changed, 6 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_driv
The two declarations with issues in rtl_wx.h:
* function: struct iw_statistics *r8192_get_wireless_stats();
* variable: extern struct iw_handler_def r8192_wx_handlers_def;
The symbol 'r8192_get_wireless_stats' is declared as 'extern'
but it is only used in the local scope. The declaration is
The following patches perform various cleanups on the gdm72xx driver.
PATCH 1/3 fixes some small coding style issues
PATCH 2/3 removes an already completed item from the TODO list
PATCH 3/3 replaces some jiffies comparisons with wrap-safe functions
After all patches have been applied, the only re
Minor white-space fixes as suggested by checkpatch.pl.
Signed-off-by: Michalis Pappas
---
drivers/staging/gdm72xx/gdm_qos.c | 2 +-
drivers/staging/gdm72xx/gdm_wimax.c | 2 ++
drivers/staging/gdm72xx/usb_boot.c | 1 +
3 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/stag
Removed line related to replacement of kernel_thread with kthread,
as issue was fixed on ff5e4a1d2702582614996f6f6d005e9b5caadeb8.
Signed-off-by: Michalis Pappas
---
drivers/staging/gdm72xx/TODO | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/gdm72xx/TODO b/drivers/staging/gd
Replaced comparisons on jiffies values with wrap-safe functions.
Signed-off-by: Michalis Pappas
---
drivers/staging/gdm72xx/gdm_usb.c | 2 +-
drivers/staging/gdm72xx/sdio_boot.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/gdm72xx/gdm_usb.c
b/drive
Hi All,
Any update on below patch. Do i need to make any modification?
On Thursday 17 April 2014 09:25 PM, Kumar Gaurav wrote:
dgap driver uses semaphore for obtaining lock. I upgraded it to use MUTEX.
Signed-off-by: Kumar Gaurav
---
drivers/staging/dgap/dgap_tty.c | 17 +
Signed-off-by: Jinqiang Zeng
---
drivers/staging/ced1401/usb1401.c |8 ++--
drivers/staging/ced1401/use14_ioc.h |4
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/ced1401/usb1401.c
b/drivers/staging/ced1401/usb1401.c
index 284abc0..5028318 1006
Hi Bastien,
On Fri, Apr 18, 2014 at 06:09:18PM +0200, Bastien Armand wrote:
> This serie of two patch fix the following sparse warnings in panel.c :
> panel.c:1235:26: warning: incorrect type in argument 1 (different address
> spaces)
> panel.c:1235:26:expected void const volatile [noderef] *
34 matches
Mail list logo
