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
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
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
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
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
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
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
> >
> >
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
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.
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
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(-)
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
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
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'
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 ++--
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
'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
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 |
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
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
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
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
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
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
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(-)
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/
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
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
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(-)
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
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
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
> -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
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
> -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
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.
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,
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
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
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
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
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
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
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
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
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
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
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
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/
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
> ++--
>
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(+),
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,
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
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
> -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
> -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
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_
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
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/
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
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
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
> -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
> -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
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
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
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
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
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/
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
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
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
+++
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 +
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
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
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_
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
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
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
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
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
> -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 - 100 of 142 matches
Mail list logo