[PATCH] staging:fbtft:Fix build warnings for incorrently using %d for size_t variables in dev_err statements

2015-02-16 Thread Nicholas Krause
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 --- driv

Re: [PATCH 2/2] dgnc: Cleanup coding style

2015-02-16 Thread Salah Triki
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 >> --- >> drivers/staging/dgnc/dgnc_driver.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/staging/dgnc/dgnc_driver.c >> b/driver

Re: [PATCH 2/2] dgnc: Cleanup coding style

2015-02-16 Thread Greg KH
On Tue, Feb 17, 2015 at 04:42:34AM +0100, Salah Triki wrote: > Signed-off-by: Salah Triki > --- > 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

Re: [PATCH 2/2] dgnc: Cleanup coding style

2015-02-16 Thread Greg KH
On Tue, Feb 17, 2015 at 04:42:34AM +0100, Salah Triki wrote: > Signed-off-by: Salah Triki > --- > 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 different thing

[PATCH 2/2] dgnc: Cleanup coding style

2015-02-16 Thread Salah Triki
Signed-off-by: Salah Triki --- 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 +++ b/drivers/stagi

[PATCH 1/2] dgnc: Cleanup coding style

2015-02-16 Thread Salah Triki
This fixes the following checkpatch.pl warnings: WARNING: printk() should include KERN_ facility level Signed-off-by: Salah Triki --- 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/

[PATCH 1/3] staging: lustre: Convert return seq_printf(...) uses to seq_has_overflowed

2015-02-16 Thread Joe Perches
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, ...); +

Re: [PATCH 1/1] dgnc: Cleanup coding style

2015-02-16 Thread Greg Kroah-Hartman
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 > --- > drivers/staging/dgnc/dgnc_driver.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-

[PATCH 1/1] dgnc: Cleanup coding style

2015-02-16 Thread Salah Triki
This fixes the following checkpatch.pl warnings: WARNING: printk() should include KERN_ facility level Signed-off-by: Salah Triki --- 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/

[PATCH 3.2 123/152] x86, hyperv: Mark the Hyper-V clocksource as being continuous

2015-02-16 Thread Ben Hutchings
3.2.67-rc1 review patch. If anyone has any objections, please let me know. -- From: "K. Y. Srinivasan" commit 32c6590d126836a062b3140ed52d898507987017 upstream. The Hyper-V clocksource is continuous; mark it accordingly. Signed-off-by: K. Y. Srinivasan Acked-by: jasow...@red

Joining the Mailing Lists

2015-02-16 Thread Salah Triki
joining the Mailing Lists -- Salah Triki ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8192u: r8192U_wx.h is now checkpatch-conform

2015-02-16 Thread Patrick Boettcher
This patch fixes warnings/errors raised by checkpatch.pl relating to line-length and comments in r8192I_wx.h. Signed-off-by: Patrick Boettcher --- drivers/staging/rtl8192u/r8192U_wx.h | 32 1 file changed, 20 insertions(+), 12 deletions(-) diff --git a/drivers/s

[PATCH 2/3] staging: lustre: Convert uses of "int rc = seq_printf(...)"

2015-02-16 Thread Joe Perches
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, ...);

[PATCH 0/3] staging: lustre: Remove use of return val of seq_printf

2015-02-16 Thread Joe Perches
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 = seq_printf(...

[PATCH 3/3] staging: lustre: Convert remaining uses of "= seq_printf(...)"

2015-02-16 Thread Joe Perches
The seq_printf return value will eventually be converted to void. Convert the remaining uses by hand. Signed-off-by: Joe Perches --- drivers/staging/lustre/lustre/llite/lproc_llite.c | 34 +--- .../lustre/lustre/obdclass/lprocfs_status.c| 94 +++--- 2 files changed,

Re: [PATCH v3 1/2] dgnc: Remove superfluous EXTRA_CFLAGS variable

2015-02-16 Thread Cass May
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 > > --- > > Having done some build tests, it seems that DG_NAME is not needed,

Re: [RESEND PATCH v2 8/9] mfd: rtsx: add support for rts525A

2015-02-16 Thread Lee Jones
On Thu, 22 Jan 2015, micky_ch...@realsil.com.cn wrote: > From: Micky Ching > > add support for new chip rts525A. > > Signed-off-by: Micky Ching > --- > drivers/mfd/rts5249.c| 104 > ++- > drivers/mfd/rtsx_pcr.c | 13 -- > drivers/mfd

Re: [RESEND PATCH v2 6/9] mfd: rtsx: remove LCTLR defination

2015-02-16 Thread Lee Jones
On Thu, 22 Jan 2015, micky_ch...@realsil.com.cn wrote: > From: Micky Ching > > 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 LINK CONTROL address

Re: [RESEND PATCH v2 7/9] mfd: rtsx: add support for rts524A

2015-02-16 Thread Lee Jones
On Thu, 22 Jan 2015, micky_ch...@realsil.com.cn wrote: > From: Micky Ching > > add support for new chip rts524A. > > Signed-off-by: Micky Ching > --- > drivers/mfd/rts5249.c| 186 > --- > drivers/mfd/rtsx_pcr.c | 25 +- > drivers/mfd

Re: [RESEND PATCH v2 5/9] mfd: rtsx: update phy register

2015-02-16 Thread Lee Jones
On Thu, 22 Jan 2015, micky_ch...@realsil.com.cn wrote: > From: Micky Ching > > 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 the configure. This is

Re: [PATCH 2/2] staging: rtl8192e: added parentheses in macros

2015-02-16 Thread Dan Carpenter
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 __

[PATCH 2/2] staging: rtl8192e: added parentheses in macros

2015-02-16 Thread Pushpendra Singh
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 --- drivers/staging/rtl8192e/rtllib_rx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/driv

[PATCH 1/2] staging: rtl8192e: remove unnecessary spaces

2015-02-16 Thread Pushpendra Singh
Removed checkpatch.pl error ERROR: space prohibited before that ',' (ctx:WxE) +433, 433, 477} } , Signed-off-by: Pushpendra Singh --- drivers/staging/rtl8192e/rtl819x_HTProc.c | 4 ++-- drivers/staging/rtl8192e/rtllib_rx.c | 4 ++-- drivers/staging/rtl8192e/rtllib_softmac.c | 2 +-

Re: [PATCH 3/4] staging: unisys: style: Replace typedefs with structs in visorchipset_main.c

2015-02-16 Thread Dan Carpenter
On Mon, Feb 16, 2015 at 01:39:46AM +0100, Frederico Cadete wrote: > Signed-off-by: Frederico Cadete > --- > .../staging/unisys/visorchipset/visorchipset_main.c | 20 > ++-- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff --git a/drivers/staging/unisys/visorchipset/

Re: [PATCH v3 1/2] dgnc: Remove superfluous EXTRA_CFLAGS variable

2015-02-16 Thread Dan Carpenter
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 > --- > 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 the forme

RE: [PATCH 0/6] Drivers: hv: vmbus

2015-02-16 Thread Dexuan Cui
> -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: