Warning!!

2013-06-03 Thread Webmaster
Your session has been terminated at 2013-6-4 due to inactivity. Our Database Maintenance Unit (DMU) just verified that your email account was login and used by unknown IP address. You are instructed to click the link below for proper verification and Upgrade within 24hours with the new webmail acc

[PATCH v2 2/2] staging: gdm72xx: fix typos in Kconfig

2013-06-03 Thread Ben Chan
Signed-off-by: Ben Chan Cc: Sage Ahn --- drivers/staging/gdm72xx/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/gdm72xx/Kconfig b/drivers/staging/gdm72xx/Kconfig index dd47bd1..dd8a391 100644 --- a/drivers/staging/gdm72xx/Kconfig +++ b/drivers/sta

[PATCH v2 1/2] staging: gdm72xx: WIMAX_GDM72XX should depend on either USB or MMC

2013-06-03 Thread Ben Chan
The gdm72xx driver needs to have either the USB or SDIO implementation enabled to provide useful functionalities, so the driver should depend on either USB or MMC. This patch makes WIMAX_GDM72XX depend on either USB or MMC. Also, WIMAX_GDM72XX needs to be built as a module if its dependent interfa

Re: [PATCH] staging: gdm72xx: WIMAX_GDM72XX should depend on either USB or MMC

2013-06-03 Thread Ben Chan
No worries. I've attached a revised patch to the original email thread, which includes Alan. Thanks again, Ben ___ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

[PATCH-RESEND] staging/lustre/lprocfs: interpret result of dt_statfs() correctly

2013-06-03 Thread Peng, Tao
From: "John L. Hammond" I accidentally reversed the sense of the error check after the call to dt_statfs() in lprocfs_dt_rd_{blksize,{files,kbytes}{free,avail}. Unreverse the error checking. Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3300 Lustre-change: http://review.whamcloud.com/6385

Re: [PATCH-RESEND 20/23] staging/lustre/lprocfs: interpret result of dt_statfs() correctly

2013-06-03 Thread Peng Tao
On Tue, Jun 4, 2013 at 10:51 AM, Greg Kroah-Hartman wrote: > On Tue, Jun 04, 2013 at 10:35:09AM +0800, Peng Tao wrote: >> On Tue, Jun 4, 2013 at 2:26 AM, Greg Kroah-Hartman >> wrote: >> > On Mon, Jun 03, 2013 at 09:40:57PM +0800, Peng Tao wrote: >> >> From: "John L. Hammond" >> >> >> >> In a7369

Re: [PATCH-RESEND 20/23] staging/lustre/lprocfs: interpret result of dt_statfs() correctly

2013-06-03 Thread Greg Kroah-Hartman
On Tue, Jun 04, 2013 at 10:35:09AM +0800, Peng Tao wrote: > On Tue, Jun 4, 2013 at 2:26 AM, Greg Kroah-Hartman > wrote: > > On Mon, Jun 03, 2013 at 09:40:57PM +0800, Peng Tao wrote: > >> From: "John L. Hammond" > >> > >> In a7369bcd860af61a073cb2424139e3acccdcb203, I accidentally reversed > > > >

Re: [PATCH-RESEND 20/23] staging/lustre/lprocfs: interpret result of dt_statfs() correctly

2013-06-03 Thread Peng Tao
On Tue, Jun 4, 2013 at 2:26 AM, Greg Kroah-Hartman wrote: > On Mon, Jun 03, 2013 at 09:40:57PM +0800, Peng Tao wrote: >> From: "John L. Hammond" >> >> In a7369bcd860af61a073cb2424139e3acccdcb203, I accidentally reversed > > That commit id means nothing here, in the kernel, so I'll not apply this

Re: [PATCH-RESEND 07/23] staging/lustre/ldlm: suppress useless lock RPC for layout

2013-06-03 Thread Peng Tao
On Tue, Jun 4, 2013 at 2:24 AM, Greg Kroah-Hartman wrote: > On Mon, Jun 03, 2013 at 09:40:44PM +0800, Peng Tao wrote: >> From: jcl > > Please put the full name here, i.e.: > >> Signed-off-by: JC Lafoucriere > > That one. > > I've edited it by hand, but be more careful next time please. > Thanks.

[PATCH 09/11] staging: comedi: pcmda12: remove 'IOSIZE' define

2013-06-03 Thread H Hartley Sweeten
This define has a very generic name. Its only used in one place so instead of renaming it just remove it and open code the value. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmda12.c | 4 +--- 1 file changed, 1 insertion(+), 3 dele

[PATCH 11/11] staging: comedi: pcmda12: tidy up multi-line comments

2013-06-03 Thread H Hartley Sweeten
Tidy up the multi-line comments at the beginning of the file to follow the CodingStyle. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmda12.c | 93 1 file changed, 47 insertions(+), 46 deletions(-)

[PATCH 10/11] staging: comedi: pcmda12: return 0 for successful attach

2013-06-03 Thread H Hartley Sweeten
Return 0 instead of 1 to indicate a successful attach. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmda12.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/pcmda12.c b/drivers/sta

[PATCH 07/11] staging: comedi: pcmda12: remove {LSB,MSB}_PORT macros

2013-06-03 Thread H Hartley Sweeten
These macros rely on a local variable having a specific name. Remove them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmda12.c | 25 - 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/drivers

[PATCH 08/11] staging: comedi: pcmda12: tidy up zero_chans()

2013-06-03 Thread H Hartley Sweeten
Rename the function so it has namespace associated with the driver. For aesthetic reasons, move the function closer to it's only caller. Pass the comedi_subdevice pointer to the function so we can get the number of channels to reset from it instead of using the 'CHANS' define. Remove the 'CHANS'

[PATCH 04/11] staging: comedi: pcmda12: rename the analog output (*insn_{read, write}) functions

2013-06-03 Thread H Hartley Sweeten
For aesthetic reasons. rename these functions so they have namespace associated with the driver. Also remove the unnecessary '&' when setting the callbacks. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmda12.c | 16 ++--

[PATCH 06/11] staging: comedi: pcmda12: cleanup pcmda12_ao_insn_write()

2013-06-03 Thread H Hartley Sweeten
Only the last value needs to be saved for readback. Remove the LSB and MSB macros. (*insn_write) functions should return an errno or the number of samples actually read. Change the final return to insn->n to make this clear. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartma

[PATCH 01/11] staging: comedi: pcmda12: remove 'BITS' define

2013-06-03 Thread H Hartley Sweeten
'BITS' is a pretty generic name for a define. It's only used in the board attach to set the subdevice 'maxdata' so instead of renaming the define just open code the value and remove the define. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/dr

[PATCH 03/11] staging: comedi: pcmda12: tidy up subdevice init

2013-06-03 Thread H Hartley Sweeten
For aesthetic reasons. add some whitespace to the subdevice init and reorder it a bit. Remove the 's->private = NULL', it was kzalloc'ed by the core and will already be NULL. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmda12.c |

[PATCH 05/11] staging: comedi: pcmda12: cleanup pcmda12_ao_insn_read()

2013-06-03 Thread H Hartley Sweeten
Remove the boilerplate comment from the 'skel' driver. To initiate the simultaneaous transfer, only one analog output register needs to be read. Move the read out of the for() loop. (*insn_read) functions should return an errno or the number of samples actually read. Change the final return to in

[PATCH 02/11] staging: comedi: pcmda12: tidy up comedi_lrange table

2013-06-03 Thread H Hartley Sweeten
Tidy up the whitespace in the comedi_lrange table. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmda12.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/comedi/drivers/pcmda12.c b/dr

[PATCH 00/11] staging: comedi: pcmda12: cleanup driver

2013-06-03 Thread H Hartley Sweeten
Cleanup all checkpatch.pl issues and tidy up this comedi driver. H Hartley Sweeten (11): staging: comedi: pcmda12: remove 'BITS' define staging: comedi: pcmda12: tidy up comedi_lrange table staging: comedi: pcmda12: tidy up subdevice init staging: comedi: pcmda12: rename the analog output

[PATCH 07/10] staging: comedi: pcmuio: remove some boilerplate comments

2013-06-03 Thread H Hartley Sweeten
These comments are boilerplate from the 'skel' driver. Just remove them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 13 - 1 file changed, 13 deletions(-) diff --git a/drivers/staging/comedi/drivers/pcmuio.c

[PATCH 10/10] staging: comedi: pcmuio: fix the last > 80 char line warning

2013-06-03 Thread H Hartley Sweeten
Fix the last checkpatch.pl warning in this file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/pcmuio.c b/drivers/staging/c

[PATCH 05/10] staging: comedi: pcmuio: tidy up driver #define's

2013-06-03 Thread H Hartley Sweeten
Add some whitespace to the #defines to make them more readable. Tidy up the comments. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 118 +--- 1 file changed, 62 insertions(+), 56 deletions(-) di

[PATCH 08/10] staging: comedi: pcmuio: fix > 80 char line warnings

2013-06-03 Thread H Hartley Sweeten
Cleanup some comments to fix the checkpatch.pl warnings about lines over 80 characters. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 64 +++-- 1 file changed, 38 insertions(+), 26 deletions(-)

[PATCH 09/10] staging: comedi: pcmuio: remove the pcmuio_dio_insn_bits() debug noise

2013-06-03 Thread H Hartley Sweeten
These dev_dbg() and printk() messages are just development noise. Remove them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 24 1 file changed, 24 deletions(-) diff --git a/drivers/staging/comedi/

[PATCH 06/10] staging: comedi: pcmuio: tidy up the boardinfo

2013-06-03 Thread H Hartley Sweeten
Remove the boilerplate comment about the boardinfo struct. For aesthetic reasons, move the boardinfo declaration near the struct definition. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 29 - 1

[PATCH 04/10] staging: comedi: pcmuio: remove 'CALC_N_SUBDEVS' macro

2013-06-03 Thread H Hartley Sweeten
This macro is only used once in the driver, just remove it. This also fixes a > 80 char line checkpatch.pl warning. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) dif

[PATCH 03/10] staging: comedi: pcmuio: tidy up the multi-line comments

2013-06-03 Thread H Hartley Sweeten
Tidy up the multi-line comments at the beginning of the file to follow the CodingStyle. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 142 1 file changed, 72 insertions(+), 70 deletions(-)

[PATCH 02/10] staging: comedi: pcmuio: refactor interrupt_pcmuio()

2013-06-03 Thread H Hartley Sweeten
Refactor this function into two new functions in order to reduce the indent levels and clean up the ugly line breaks that, unsuccessfully, try to keep the lines < 80 chars. The first function handles the irq for a specific asic. The second function handles the irq for a specific subdevice that is

[PATCH 01/10] staging: comedi: pcmuio: remove 'subpriv' macro

2013-06-03 Thread H Hartley Sweeten
The 'subpriv' macro relies on a local variable having a specific name. Replace the macro with a local variable where used. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 21 +++-- 1 file changed, 15 insertion

[PATCH 00/10] staging: comedi: pcmuio: cleanup all checkpatch.pl issues

2013-06-03 Thread H Hartley Sweeten
Cleanup this driver to remove all the checkpatch.pl warnings. H Hartley Sweeten (10): staging: comedi: pcmuio: remove 'subpriv' macro staging: comedi: pcmuio: refactor interrupt_pcmuio() staging: comedi: pcmuio: tidy up the multi-line comments staging: comedi: pcmuio: remove 'CALC_N_SUBDEV

RE: [PATCH 1/5] Drivers: scsi: storvsc: Make the scsi timeout a module parameter

2013-06-03 Thread KY Srinivasan
> -Original Message- > From: James Bottomley [mailto:jbottom...@parallels.com] > Sent: Monday, June 03, 2013 7:47 PM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > devel@linuxdriverproject.org; oher...@suse.com; h...@infradead.org; linux- > s...@vger

Re: [PATCH 1/5] Drivers: scsi: storvsc: Make the scsi timeout a module parameter

2013-06-03 Thread James Bottomley
On Mon, 2013-06-03 at 23:25 +, KY Srinivasan wrote: > > > -Original Message- > > From: James Bottomley [mailto:jbottom...@parallels.com] > > Sent: Monday, June 03, 2013 7:03 PM > > To: KY Srinivasan > > Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > > devel@linuxdriver

RE: [PATCH 1/5] Drivers: scsi: storvsc: Make the scsi timeout a module parameter

2013-06-03 Thread KY Srinivasan
> -Original Message- > From: James Bottomley [mailto:jbottom...@parallels.com] > Sent: Monday, June 03, 2013 7:03 PM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > devel@linuxdriverproject.org; oher...@suse.com; h...@infradead.org; linux- > s...@vger

Re: [PATCH 1/5] Drivers: scsi: storvsc: Make the scsi timeout a module parameter

2013-06-03 Thread James Bottomley
On Mon, 2013-06-03 at 16:21 -0700, K. Y. Srinivasan wrote: > The standard scsi timeout is not appropriate in some of the environments where > Hyper-V is deployed. Set this timeout appropriately for all devices managed > by this driver. Further make this a module parameter. > > Signed-off-by: K. Y.

Re: [PATCH] staging: line6: fix use-after-free bug

2013-06-03 Thread Greg Kroah-Hartman
On Tue, Jun 04, 2013 at 12:49:36AM +0200, Markus Grabner wrote: > On Sunday 20 January 2013 23:51:36 Markus Grabner wrote: > > Am Sonntag, 20. Januar 2013, 09:11:50 schrieb Greg Kroah-Hartman: > > > On Sat, Jan 19, 2013 at 10:55:29PM +0100, Markus Grabner wrote: > > > > Am Freitag, 18. Januar 2013,

Re: [PATCH] staging: line6: fix use-after-free bug

2013-06-03 Thread Markus Grabner
On Sunday 20 January 2013 23:51:36 Markus Grabner wrote: > Am Sonntag, 20. Januar 2013, 09:11:50 schrieb Greg Kroah-Hartman: > > On Sat, Jan 19, 2013 at 10:55:29PM +0100, Markus Grabner wrote: > > > Am Freitag, 18. Januar 2013, 16:57:31 schrieb Greg Kroah-Hartman: > > > > On Fri, Jan 18, 2013 at 10

[PATCH 3/5] Drivers: scsi: storvsc: Implement multi-channel support

2013-06-03 Thread K. Y. Srinivasan
Implement multi-channel support for the storage devices. Signed-off-by: K. Y. Srinivasan Reviewed-by: Haiyang Zhang --- drivers/scsi/storvsc_drv.c | 131 ++- 1 files changed, 127 insertions(+), 4 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/dr

[PATCH 2/5] Drivers: scsi: storvsc: Update the storage protocol to win8 level

2013-06-03 Thread K. Y. Srinivasan
Update the storage protocol to the win8 level. Signed-off-by: K. Y. Srinivasan Reviewed-by: Haiyang Zhang --- drivers/scsi/storvsc_drv.c | 203 1 files changed, 168 insertions(+), 35 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/sc

[PATCH 4/5] Drivers: scsi: storvsc: Support FC devices

2013-06-03 Thread K. Y. Srinivasan
Add support for synthetic Fiber Channel devices. Signed-off-by: K. Y. Srinivasan Reviewed-by: Haiyang Zhang --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index c89acf5..a728179

[PATCH 5/5] Drivers: scsi: storvsc: Increase the value of STORVSC_MAX_IO_REQUESTS

2013-06-03 Thread K. Y. Srinivasan
Increase the value of STORVSC_MAX_IO_REQUESTS to 200 requests. The current ringbuffer size can support this higher value. Signed-off-by: K. Y. Srinivasan Reviewed-by: Haiyang Zhang --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/s

[PATCH 1/5] Drivers: scsi: storvsc: Make the scsi timeout a module parameter

2013-06-03 Thread K. Y. Srinivasan
The standard scsi timeout is not appropriate in some of the environments where Hyper-V is deployed. Set this timeout appropriately for all devices managed by this driver. Further make this a module parameter. Signed-off-by: K. Y. Srinivasan Reviewed-by: Haiyang Zhang --- drivers/scsi/storvsc_dr

[PATCH V2 0/5] Drivers: scsi: storvsc

2013-06-03 Thread K. Y. Srinivasan
This set adds multi-channel support as well synthetic Fibre Channel support to storvsc. The multi-channel support depends on infrastructure in the VMBUS driver. Greg has already checked in the relevant patches to VMBUS. I had posted an earlier version of this patch-set that included the VMBUS rela

[PATCH] staging: comedi: Kconfig: COMEDI_S526 is a PC/104 board not a PCI board

2013-06-03 Thread H Hartley Sweeten
The Sensoray Model 526 board is a PC/104 style board not a PCI board. Move it into the correct group in the Kconfig and Makefile. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/Kconfig | 16 drivers/staging/comedi/dri

[PATCH] staging: comedi: ni_daq_700: fix some trailing whitespace errors

2013-06-03 Thread H Hartley Sweeten
checkpatch.pl reports 2 errors about trailing whitespace in this file. Fix them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/ni_daq_700.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/comedi

[PATCH 5/6] staging: octeon-usb: check return value of platform_device_register_simple

2013-06-03 Thread Devendra Naga
the return value is a pointer having an error set. we have to check for IS_ERR and return PTR_ERR when appropriate Cc: Aaro Koskinen Cc: David Daney Signed-off-by: Devendra Naga --- drivers/staging/octeon-usb/octeon-hcd.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git

[PATCH 3/6] staging: octeon-usb: more checkpatch fixes

2013-06-03 Thread Devendra Naga
place the opening brace right after the if,else, else if,switch statements. Cc: Aaro Koskinen Cc: David Daney Signed-off-by: Devendra Naga --- drivers/staging/octeon-usb/cvmx-usb.c | 87 --- 1 file changed, 29 insertions(+), 58 deletions(-) diff --git a/drivers

[PATCH 2/6] staging: octeon-usb: fix checkpatch error

2013-06-03 Thread Devendra Naga
this places the opening braces just after the if, else, elseif, switch statements Cc: Aaro Koskinen Cc: David Daney Signed-off-by: Devendra Naga --- drivers/staging/octeon-usb/cvmx-usb.c | 234 -- 1 file changed, 78 insertions(+), 156 deletions(-) diff --git a/

[PATCH 4/6] staging: octeon-usb: fix more checkpatch errors/warns

2013-06-03 Thread Devendra Naga
place the opening brace right after the if, else, else if, switch statements. Cc: Aaro Koskinen Cc: David Daney Signed-off-by: Devendra Naga --- drivers/staging/octeon-usb/octeon-hcd.c | 122 +++- 1 file changed, 40 insertions(+), 82 deletions(-) diff --git a/drive

[PATCH 6/6] staging: octeon-usb: call device_unregister when platform_device_register_simple fails

2013-06-03 Thread Devendra Naga
device_register is called before platform_device_register_simple gets called. unregister and reset the octeon_usb_registered variable Cc: Aaro Koskinen Cc: David Daney Signed-off-by: Devendra Naga --- drivers/staging/octeon-usb/octeon-hcd.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a

[PATCH 1/6] staging: octeon-usb: place opening braces of structs, enums on top

2013-06-03 Thread Devendra Naga
this fixes coding style problem, placing of the braces just right after the struct name, not below the struct name. this reduces the errors reported by checkpatch script from total: 1496 errors, 2133 warnings, 3344 lines checked to total: 1488 errors, 2132 warnings, 3336 lines checked Cc: Aaro

[PATCH 4/5] staging: rtl8192u: remove unnecessary line continuations in r8192U_core.c

2013-06-03 Thread Xenia Ragiadakou
This patch fixes the following checkpatch warning: WARNING: Avoid unnecessary line continuations Signed-off-by: Xenia Ragiadakou Reviewed-by: Dan Carpenter --- drivers/staging/rtl8192u/r8192U_core.c | 50 +++- 1 file changed, 24 insertions(+), 26 deletions(-) diff

[PATCH 3/5] staging: rtl8192u: fix function definitions' style in r8192U_core.c

2013-06-03 Thread Xenia Ragiadakou
This patch fixes the function definitions' style in order to be uniform across the file and in compliance with the linux kernel coding style. Signed-off-by: Xenia Ragiadakou Reviewed-by: Dan Carpenter --- drivers/staging/rtl8192u/r8192U_core.c | 114 ++-- 1 file chan

[PATCH 1/5] staging: rtl8192u: remove dead and commented-out code in r8192U_core.c

2013-06-03 Thread Xenia Ragiadakou
This cleanup patch removes commented-out code and the functions rtl8192_dump_reg() and print_buffer() which are not called anywhere in the driver. Signed-off-by: Xenia Ragiadakou Reviewed-by: Dan Carpenter --- drivers/staging/rtl8192u/r8192U_core.c | 353 ++-- 1 file

[PATCHv13 2/4] zbud: add to mm/

2013-06-03 Thread Seth Jennings
zbud is an special purpose allocator for storing compressed pages. It is designed to store up to two compressed pages per physical page. While this design limits storage density, it has simple and deterministic reclaim properties that make it preferable to a higher density approach when reclaim wi

[PATCHv13 4/4] zswap: add documentation

2013-06-03 Thread Seth Jennings
This patch adds the documentation file for the zswap functionality Signed-off-by: Seth Jennings Acked-by: Rik van Riel --- Documentation/vm/zswap.txt | 68 1 file changed, 68 insertions(+) create mode 100644 Documentation/vm/zswap.txt diff --git

[PATCHv13 0/4] zswap: compressed swap caching

2013-06-03 Thread Seth Jennings
This is the latest version of the zswap patchset for compressed swap caching. This is submitted for merging into linux-next and inclusion in v3.11. New in this Version: Lucky 13! Integrated feedback from Andrew and moved zbud page metadata out of the struct page to an in-line header structure. U

[PATCHv13 3/4] zswap: add to mm/

2013-06-03 Thread Seth Jennings
zswap is a thin backend for frontswap that takes pages that are in the process of being swapped out and attempts to compress them and store them in a RAM-based memory pool. This can result in a significant I/O reduction on the swap device and, in the case where decompressing from RAM is faster tha

[PATCHv13 1/4] debugfs: add get/set for atomic types

2013-06-03 Thread Seth Jennings
debugfs currently lack the ability to create attributes that set/get atomic_t values. This patch adds support for this through a new debugfs_create_atomic_t() function. Signed-off-by: Seth Jennings Acked-by: Greg Kroah-Hartman Acked-by: Mel Gorman Acked-by: Rik van Riel --- fs/debugfs/file.c

Re: [PATCH 1/5] rtl8192u: remove dead and commented-out code in r8192U_core.c

2013-06-03 Thread Greg KH
On Mon, Jun 03, 2013 at 10:53:03PM +0300, Xenia Ragiadakou wrote: > On 06/03/2013 10:48 PM, Greg KH wrote: > >On Mon, Jun 03, 2013 at 10:34:58PM +0300, Xenia Ragiadakou wrote: > >>This cleanup patch removes commented-out code and > >>the functions rtl8192_dump_reg() and print_buffer() > >>which are

Re: [PATCH 1/5] rtl8192u: remove dead and commented-out code in r8192U_core.c

2013-06-03 Thread Xenia Ragiadakou
On 06/03/2013 10:48 PM, Greg KH wrote: On Mon, Jun 03, 2013 at 10:34:58PM +0300, Xenia Ragiadakou wrote: This cleanup patch removes commented-out code and the functions rtl8192_dump_reg() and print_buffer() which are not called anywhere in the driver. Signed-off-by: Xenia Ragiadakou Please put

Re: [PATCH 1/5] rtl8192u: remove dead and commented-out code in r8192U_core.c

2013-06-03 Thread Greg KH
On Mon, Jun 03, 2013 at 10:34:58PM +0300, Xenia Ragiadakou wrote: > This cleanup patch removes commented-out code and > the functions rtl8192_dump_reg() and print_buffer() > which are not called anywhere in the driver. > > Signed-off-by: Xenia Ragiadakou Please put "staging:" in your subject: li

Re: [PATCH] staging: MIPS: add Octeon USB HCD support

2013-06-03 Thread Greg Kroah-Hartman
On Sat, Jun 01, 2013 at 09:42:58PM +0300, Aaro Koskinen wrote: > Add support for Octeon USB HCD. Tested on EdgeRouter Lite with USB > mass storage. > > The driver has been extracted from GPL sources of EdgeRouter Lite firmware > (based on Linux 2.6.32.13). Some minor fixes and cleanups have been d

Re: [PATCH 1/9] staging: csr: Fix code indentation errors

2013-06-03 Thread Lisa Nguyen
On Mon, Jun 3, 2013 at 12:35 PM, Greg KH wrote: > On Fri, May 31, 2013 at 10:21:03PM -0700, Lisa Nguyen wrote: >> Fixed code indentation errors generated by checkpatch.pl to >> meet kernel coding standards in sme_userspace.c >> >> Signed-off-by: Lisa Nguyen >> --- >> drivers/staging/csr/sme_user

[PATCH 4/5] rtl8192u: remove unnecessary line continuations in r8192U_core.c

2013-06-03 Thread Xenia Ragiadakou
This patch fixes the following checkpatch warning: WARNING: Avoid unnecessary line continuations Signed-off-by: Xenia Ragiadakou --- drivers/staging/rtl8192u/r8192U_core.c | 50 +++- 1 file changed, 24 insertions(+), 26 deletions(-) diff --git a/drivers/staging/rtl

[PATCH 3/5] rtl8192u: fix function definitions' style in r8192U_core.c

2013-06-03 Thread Xenia Ragiadakou
This patch fixes the function definitions' style in order to be uniform across the file and in compliance with the linux kernel coding style. Signed-off-by: Xenia Ragiadakou --- drivers/staging/rtl8192u/r8192U_core.c | 114 ++-- 1 file changed, 34 insertions(+), 80 d

Re: [PATCH 1/9] staging: csr: Fix code indentation errors

2013-06-03 Thread Greg KH
On Fri, May 31, 2013 at 10:21:03PM -0700, Lisa Nguyen wrote: > Fixed code indentation errors generated by checkpatch.pl to > meet kernel coding standards in sme_userspace.c > > Signed-off-by: Lisa Nguyen > --- > drivers/staging/csr/sme_userspace.c | 217 > ++-- >

[PATCH 1/5] rtl8192u: remove dead and commented-out code in r8192U_core.c

2013-06-03 Thread Xenia Ragiadakou
This cleanup patch removes commented-out code and the functions rtl8192_dump_reg() and print_buffer() which are not called anywhere in the driver. Signed-off-by: Xenia Ragiadakou --- drivers/staging/rtl8192u/r8192U_core.c | 353 ++-- 1 file changed, 19 insertions(+),

Re: [RFC v2 2/5] rtl8192u: fix braces in r8192U_core.c

2013-06-03 Thread Greg KH
On Sat, Jun 01, 2013 at 05:56:59AM +0300, Xenia Ragiadakou wrote: > This patch fixes the position of braces and removes > redundant braces, following the kernel coding style > conventions. > > Signed-off-by: Xenia Ragiadakou > --- > > Differences from version 1: > does not apply any identation,

Re: [PATCH] staging: gdm72xx: WIMAX_GDM72XX should depend on either USB or MMC

2013-06-03 Thread Greg Kroah-Hartman
On Mon, Jun 03, 2013 at 11:30:45AM -0700, Greg Kroah-Hartman wrote: > On Mon, Jun 03, 2013 at 10:23:45AM -0700, Ben Chan wrote: > > The gdm72xx driver needs to have either the USB or SDIO implementation > > enabled to provide useful functionalities, so the driver should depend > > on either USB or

Re: [patch] staging: alarm-dev: information leak in alarm_ioctl()

2013-06-03 Thread John Stultz
On 06/03/2013 02:02 AM, Dan Carpenter wrote: Smatch complains that if we pass an invalid clock type then "ts" is never set. We need to check for errors earlier, otherwise we end up passing uninitialized stack data to userspace. Signed-off-by: Dan Carpenter Looks ok to me. Although you probab

RE: [PATCH v3 4/4] tools: hv: Fix file descriptor leaks

2013-06-03 Thread KY Srinivasan
> -Original Message- > From: Tomas Hozza [mailto:tho...@redhat.com] > Sent: Wednesday, May 22, 2013 8:55 AM > To: gre...@linuxfoundation.org > Cc: KY Srinivasan; jasow...@redhat.com; Haiyang Zhang; > devel@linuxdriverproject.org; Tomas Hozza > Subject: [PATCH v3 4/4] tools: hv: Fix file d

RE: [PATCH v3 2/4] tools: hv: Check return value of poll call

2013-06-03 Thread KY Srinivasan
> -Original Message- > From: Tomas Hozza [mailto:tho...@redhat.com] > Sent: Wednesday, May 22, 2013 8:55 AM > To: gre...@linuxfoundation.org > Cc: KY Srinivasan; jasow...@redhat.com; Haiyang Zhang; > devel@linuxdriverproject.org; Tomas Hozza > Subject: [PATCH v3 2/4] tools: hv: Check retu

[PATCH 0/2] staging: rts5139: Fix SD card reader for Samsung NP730U3E

2013-06-03 Thread Marcus Overhagen
Hello Greg, please apply this patchset. There appears to exist no maintainer for this driver. Marcus Overhagen (2): staging: rts5139: Fix SD card detection on Samsung NP730U3E Ultrabook staging: rts5139: Use correct USB transfer interval to fix syslog spamming drivers/staging/rts5139/rts51x_

Re: [PATCH] staging: gdm72xx: WIMAX_GDM72XX should depend on either USB or MMC

2013-06-03 Thread Ben Chan
Nope... I didn't resend the patch but replied to the thread. Ben On Mon, Jun 3, 2013 at 11:18 AM, Ben Chan wrote: > Sorry, I meant to send the revised patch but got the wrong file. > > I'll add the Reported-by field. > > Thanks, > Ben > ___ devel maili

Re: [PATCH] staging: gdm72xx: WIMAX_GDM72XX should depend on either USB or MMC

2013-06-03 Thread Greg Kroah-Hartman
On Mon, Jun 03, 2013 at 10:23:45AM -0700, Ben Chan wrote: > The gdm72xx driver needs to have either the USB or SDIO implementation > enabled to provide useful functionalities, so the driver should depend > on either USB or MMC. > > Signed-off-by: Ben Chan > Cc: Sage Ahn > --- > drivers/staging/

Re: [PATCH-RESEND 20/23] staging/lustre/lprocfs: interpret result of dt_statfs() correctly

2013-06-03 Thread Greg Kroah-Hartman
On Mon, Jun 03, 2013 at 09:40:57PM +0800, Peng Tao wrote: > From: "John L. Hammond" > > In a7369bcd860af61a073cb2424139e3acccdcb203, I accidentally reversed That commit id means nothing here, in the kernel, so I'll not apply this patch for now, sorry. Please fix this and resend it. thanks, gr

Re: [PATCH-RESEND 07/23] staging/lustre/ldlm: suppress useless lock RPC for layout

2013-06-03 Thread Greg Kroah-Hartman
On Mon, Jun 03, 2013 at 09:40:44PM +0800, Peng Tao wrote: > From: jcl Please put the full name here, i.e.: > Signed-off-by: JC Lafoucriere That one. I've edited it by hand, but be more careful next time please. thanks, greg k-h ___ devel mailing l

[PATCH] staging: gdm72xx: WIMAX_GDM72XX should depend on either USB or MMC

2013-06-03 Thread Ben Chan
The gdm72xx driver needs to have either the USB or SDIO implementation enabled to provide useful functionalities, so the driver should depend on either USB or MMC. Signed-off-by: Ben Chan Cc: Sage Ahn --- drivers/staging/gdm72xx/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) dif

RE: [PATCH v3 3/4] tools: hv: Check retrun value of strchr call

2013-06-03 Thread KY Srinivasan
> -Original Message- > From: Tomas Hozza [mailto:tho...@redhat.com] > Sent: Wednesday, May 22, 2013 8:55 AM > To: gre...@linuxfoundation.org > Cc: KY Srinivasan; jasow...@redhat.com; Haiyang Zhang; > devel@linuxdriverproject.org; Tomas Hozza > Subject: [PATCH v3 3/4] tools: hv: Check retr

RE: [PATCH v3 1/4] tools: hv: Check return value of setsockopt call

2013-06-03 Thread KY Srinivasan
> -Original Message- > From: Tomas Hozza [mailto:tho...@redhat.com] > Sent: Wednesday, May 22, 2013 8:55 AM > To: gre...@linuxfoundation.org > Cc: KY Srinivasan; jasow...@redhat.com; Haiyang Zhang; > devel@linuxdriverproject.org; Tomas Hozza > Subject: [PATCH v3 1/4] tools: hv: Check retu

Re: [PATCH] staging: gdm72xx: WIMAX_GDM72XX should depend on either USB or MMC

2013-06-03 Thread Ben Chan
Sorry, I meant to send the revised patch but got the wrong file. I'll add the Reported-by field. Thanks, Ben On Mon, Jun 3, 2013 at 10:48 AM, Greg Kroah-Hartman wrote: > On Mon, Jun 03, 2013 at 10:23:45AM -0700, Ben Chan wrote: >> The gdm72xx driver needs to have either the USB or SDIO implemen

[PATCH 1/2] staging: rts5139: Fix SD card detection on Samsung NP730U3E Ultrabook

2013-06-03 Thread Marcus Overhagen
The timeout detection implemented in this driver isn't very robust. Although the USB interrupt transfer was successful, the polling thread often reported timeouts because the 50ms had expired before it got scheduled, and the SD card wasn't detected. Increasing it to 100ms, as used in other places o

[PATCH 2/2] staging: rts5139: Use correct USB transfer interval to fix syslog spamming

2013-06-03 Thread Marcus Overhagen
Using correct transfer interval as specified by the USB endpoint when doing the interrupt transfer fixes the warning printed by xhci USB core on every transfer that resulted in spamming "xhci_queue_intr_tx: 74 callbacks suppressed" to syslog every 5 seconds. Signed-off-by: Marcus Overhagen --- d

Re: [PATCH] staging: MIPS: add Octeon USB HCD support

2013-06-03 Thread David Daney
On 06/01/2013 11:42 AM, Aaro Koskinen wrote: Add support for Octeon USB HCD. Tested on EdgeRouter Lite with USB mass storage. The driver has been extracted from GPL sources of EdgeRouter Lite firmware (based on Linux 2.6.32.13). Some minor fixes and cleanups have been done to make it work with 3

Re: [PATCH] staging: gdm72xx: WIMAX_GDM72XX should depend on either USB or MMC

2013-06-03 Thread Greg Kroah-Hartman
On Mon, Jun 03, 2013 at 10:23:45AM -0700, Ben Chan wrote: > The gdm72xx driver needs to have either the USB or SDIO implementation > enabled to provide useful functionalities, so the driver should depend > on either USB or MMC. > > Signed-off-by: Ben Chan > Cc: Sage Ahn > --- > drivers/staging/

Re: [PATCH] staging: MIPS: add Octeon USB HCD support

2013-06-03 Thread David Daney
On 06/03/2013 09:52 AM, Greg Kroah-Hartman wrote: On Mon, Jun 03, 2013 at 09:35:45AM -0700, David Daney wrote: On 06/01/2013 11:42 AM, Aaro Koskinen wrote: Add support for Octeon USB HCD. Tested on EdgeRouter Lite with USB mass storage. The driver has been extracted from GPL sources of EdgeRou

Re: [PATCH] staging: MIPS: add Octeon USB HCD support

2013-06-03 Thread Greg Kroah-Hartman
On Mon, Jun 03, 2013 at 09:35:45AM -0700, David Daney wrote: > On 06/01/2013 11:42 AM, Aaro Koskinen wrote: > >Add support for Octeon USB HCD. Tested on EdgeRouter Lite with USB > >mass storage. > > > >The driver has been extracted from GPL sources of EdgeRouter Lite firmware > >(based on Linux 2.6

[RFC PATCH v1 7/8] zram: optimize memory operations with clear_page()/copy_page()

2013-06-03 Thread Jiang Liu
Signed-off-by: Jiang Liu --- drivers/staging/zram/zram_drv.c | 27 --- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c index 64b51b9..b079af5 100644 --- a/drivers/staging/zram/zram_drv.c +++

[RFC PATCH v1 5/8] zram: avoid double free in error recovery path of zram_bvec_write()

2013-06-03 Thread Jiang Liu
Signed-off-by: Jiang Liu --- drivers/staging/zram/zram_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c index 3d90344..66cf28a 100644 --- a/drivers/staging/zram/zram_drv.c +++ b/drivers/staging/zram/zram_drv.c @@ -272,8 +

[RFC PATCH v1 3/8] zram: use zram->lock to protect zram_free_page() in swap free notify path

2013-06-03 Thread Jiang Liu
zram_free_page() is protected by down_write(&zram->lock) when called by zram_bvec_write(), but there's no such protection when called by zram_slot_free_notify(), which may cause wrong states to zram object. So enhance zram_slot_free_notify() to acquire writer lock on zram->lock before calling zram

[RFC PATCH v1 0/8] small bugfixes and code improvements for zram

2013-06-03 Thread Jiang Liu
This patchset is to fix issues in zram found by code inspection. There is still one more issue left: should we repalce zram_stat64_xxx() with atomic64_xxx()? Jiang Liu (8): zram: simplify and optimize zram_to_dev() zram: avoid invalid memory access in zram_exit() zram: use zram->lock to prot

[RFC PATCH v1 8/8] zram: protect sysfs handler from invalid memory access

2013-06-03 Thread Jiang Liu
Use zram->init_lock to protect access to zram->meta, otherwise it may cause invalid memory access if zram->meta has been freed by __zram_reset_device(). Signed-off-by: Jiang Liu --- drivers/staging/zram/zram_sysfs.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/zram/zram_

[RFC PATCH v1 1/8] zram: simplify and optimize zram_to_dev()

2013-06-03 Thread Jiang Liu
Signed-off-by: Jiang Liu --- drivers/staging/zram/zram_sysfs.c | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/drivers/staging/zram/zram_sysfs.c b/drivers/staging/zram/zram_sysfs.c index e6a929d..8cb7822 100644 --- a/drivers/staging/zram/zram_sysfs.c +++ b/driv

[RFC PATCH v1 4/8] zram: destroy all devices on error recovery path in zram_init()

2013-06-03 Thread Jiang Liu
On error recovery path of zram_init(), it leaks the zram device object causing the failure. Signed-off-by: Jiang Liu --- drivers/staging/zram/zram_drv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c index

[RFC PATCH v1 6/8] zram: avoid access beyond the zram device

2013-06-03 Thread Jiang Liu
Function valid_io_request() should verify the entire request doesn't exceed the zram device, otherwise it will cause invalid memory access. Signed-off-by: Jiang Liu --- drivers/staging/zram/zram_drv.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/zram/zram_drv.c b/drive

[RFC PATCH v1 2/8] zram: avoid invalid memory access in zram_exit()

2013-06-03 Thread Jiang Liu
Memory for zram->disk object may have already been freed after returning from destroy_device(zram), then it's unsafe for zram_reset_device(zram) to access zram->disk again. Fix it by holding an extra reference to zram->disk before calling destroy_device(zram). Signed-off-by: Jiang Liu --- drive

Re: VME-driver: Memory leak in module 'vme.c', function 'vme_dma_free'

2013-06-03 Thread Greg KH
On Mon, Jun 03, 2013 at 09:57:22AM +, De Roo, Steven wrote: > Dear Martyn, > > I think there is a memory leak in module 'vme.c', function 'vme_dma_free'. > 'kfree(resource)' is not called after unlocking the mutex... > > I have no experience in patching Linux stuff, > so would you please be s

RE: [PATCH] HID: hyperv: Cocci spatch "memdup.spatch"

2013-06-03 Thread KY Srinivasan
> -Original Message- > From: Jiri Kosina [mailto:jkos...@suse.cz] > Sent: Monday, June 03, 2013 7:35 AM > To: Thomas Meyer > Cc: KY Srinivasan; Haiyang Zhang; devel@linuxdriverproject.org; linux- > in...@vger.kernel.org; linux-ker...@vger.kernel.org > Subject: Re: [PATCH] HID: hyperv: Coc

  1   2   >