The seq_printf return value will eventually be converted to void.
Convert the remaining uses by hand.
Signed-off-by: Joe Perches j...@perches.com
---
drivers/staging/lustre/lustre/llite/lproc_llite.c | 34 +---
.../lustre/lustre/obdclass/lprocfs_status.c| 94 +++---
seq_printf should eventually be converted to a void function.
Remove the uses of seq_printf return values and use seq_has_overflowed()
instead.
Joe Perches (3):
staging: lustre: Convert return seq_printf(...) uses to seq_has_overflowed
staging: lustre: Convert uses of int rc =
The seq_printf return value will eventually be converted to void.
Convert these uses to:
seq_printf(seq, ...);
return seq_has_overflowed(seq);
Done via cocci script:
@@
struct seq_file *seq;
int i;
@@
- i = seq_printf(seq,
+ seq_printf(seq,
...);
This patch fixes warnings/errors raised by checkpatch.pl relating to
line-length and comments in r8192I_wx.h.
Signed-off-by: Patrick Boettcher patrick.boettc...@posteo.de
---
drivers/staging/rtl8192u/r8192U_wx.h | 32
1 file changed, 20 insertions(+), 12
3.2.67-rc1 review patch. If anyone has any objections, please let me know.
--
From: K. Y. Srinivasan k...@microsoft.com
commit 32c6590d126836a062b3140ed52d898507987017 upstream.
The Hyper-V clocksource is continuous; mark it accordingly.
Signed-off-by: K. Y. Srinivasan
On Thu, 22 Jan 2015, micky_ch...@realsil.com.cn wrote:
From: Micky Ching micky_ch...@realsil.com.cn
update phy register value and using direct value instead of macros.
It is much easier to debug using constant value than a lot of macros.
We usually need compare the value directly to check
On Thu, 22 Jan 2015, micky_ch...@realsil.com.cn wrote:
From: Micky Ching micky_ch...@realsil.com.cn
To enable/disable ASPM we should find LINK CONTROL register
in PCI config space. All old chip use 0x80 address, but new
chip may use another address, so we using pci_find_capability()
to get
On Thu, 22 Jan 2015, micky_ch...@realsil.com.cn wrote:
From: Micky Ching micky_ch...@realsil.com.cn
add support for new chip rts524A.
Signed-off-by: Micky Ching micky_ch...@realsil.com.cn
---
drivers/mfd/rts5249.c| 186
---
On Tue, Feb 17, 2015 at 03:11:38AM +0100, Salah Triki wrote:
This fixes the following checkpatch.pl warnings:
WARNING: printk() should include KERN_ facility level
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_driver.c | 2 +-
1 file changed, 1 insertion(+),
The seq_printf return value will eventually be converted to void.
Convert these uses to:
seq_printf(seq, ...);
return seq_has_overflowed(seq);
Done via cocci script:
@@
struct seq_file *seq;
@@
- return
\(seq_printf\|seq_puts\|seq_putc\)(seq,
...);
+
On Tue, Feb 17, 2015 at 04:42:34AM +0100, Salah Triki wrote:
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_driver.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc/dgnc_driver.c
b/drivers/staging/dgnc/dgnc_driver.c
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_driver.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc/dgnc_driver.c
b/drivers/staging/dgnc/dgnc_driver.c
index 0a05ed2..45c584c 100644
--- a/drivers/staging/dgnc/dgnc_driver.c
This fixes the following checkpatch.pl warnings:
WARNING: printk() should include KERN_ facility level
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_driver.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc/dgnc_driver.c
On Tue, Feb 17, 2015 at 04:42:34AM +0100, Salah Triki wrote:
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_driver.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
No changelog comment?
And you sent in 2 patches with identical subject lines, yet they did
This fixes the following checkpatch.pl warnings:
WARNING: printk() should include KERN_ facility level
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_driver.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc/dgnc_driver.c
On 17/02/2015 04:54, Greg KH wrote:
On Tue, Feb 17, 2015 at 04:42:34AM +0100, Salah Triki wrote:
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_driver.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc/dgnc_driver.c
Fixes the build warnings for using %d for a variable of type size_t in dev_err
statements
for the functions,fbtft_read_spi and fbtft_write_spi_emulate_9 respectfully by
changing
both dev_error calls to use the correct format specifier for size_t instead.
Signed-off-by: Nicholas Krause
On Thu, 22 Jan 2015, micky_ch...@realsil.com.cn wrote:
From: Micky Ching micky_ch...@realsil.com.cn
add support for new chip rts525A.
Signed-off-by: Micky Ching micky_ch...@realsil.com.cn
---
drivers/mfd/rts5249.c| 104
++-
On Mon, 2015-02-16 at 12:12 +0300, Dan Carpenter wrote:
On Sun, Feb 15, 2015 at 11:40:17PM +, Cass May wrote:
Clean up Makefile by removing unnecessary definition of DG_NAME.
Signed-off-by: Cass May c...@cassm.net
---
Having done some build tests, it seems that DG_NAME is not
-Original Message-
From: KY Srinivasan
Sent: Monday, February 16, 2015 13:28 PM
To: Dexuan Cui; gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com;
vkuzn...@redhat.com
Subject: RE: [PATCH 0/6] Drivers: hv: vmbus
On Mon, Feb 16, 2015 at 01:39:46AM +0100, Frederico Cadete wrote:
Signed-off-by: Frederico Cadete freder...@cadete.eu
---
.../staging/unisys/visorchipset/visorchipset_main.c | 20
++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git
On Sun, Feb 15, 2015 at 11:40:17PM +, Cass May wrote:
Clean up Makefile by removing unnecessary definition of DG_NAME.
Signed-off-by: Cass May c...@cassm.net
---
Having done some build tests, it seems that DG_NAME is not needed,
but DG_PART is referenced in dgnc_mgmt.c. I have removed
Removed checkpatch.pl error
ERROR: Macros with complex values should be enclosed in parentheses
+#define MFIE_STRING(x) case MFIE_TYPE_ ##x: return #x
Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com
---
drivers/staging/rtl8192e/rtllib_rx.c | 2 +-
1 file changed, 1 insertion(+),
Removed checkpatch.pl error
ERROR: space prohibited before that ',' (ctx:WxE)
+433, 433, 477} } ,
Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 4 ++--
drivers/staging/rtl8192e/rtllib_rx.c | 4 ++--
On Mon, Feb 16, 2015 at 03:12:25PM +0530, Pushpendra Singh wrote:
Removed checkpatch.pl error
ERROR: Macros with complex values should be enclosed in parentheses
+#define MFIE_STRING(x) case MFIE_TYPE_ ##x: return #x
You didn't try to compile this code.
regards,
dan carpenter
25 matches
Mail list logo