Signed-off-by: Daeseok Youn daeseok.y...@gmail.com
---
drivers/staging/bcm/InterfaceDld.c |6 ++
1 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/bcm/InterfaceDld.c
b/drivers/staging/bcm/InterfaceDld.c
index 005e460..e1925bd 100644
---
Fix the following warning in lib-eq.c
warning: symbol 'lnet_eq_dequeue_event' was not declared. Should it be static?
Signed-off-by: Toby Smith t...@tismith.id.au
---
drivers/staging/lustre/lnet/lnet/lib-eq.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Fix the following warning in lib-eq.c
warning: context imbalance in 'lnet_eq_wait_locked' - unexpected unlock
Signed-off-by: Toby Smith t...@tismith.id.au
---
drivers/staging/lustre/lnet/lnet/lib-eq.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/lustre/lnet/lnet/lib-eq.c
Fix warnings from sparse in lib-eq.c
Toby Smith (2):
staging: lustre: fix sparse warning should it be static
staging: lustre: fix sparse warning unexpected unlock
drivers/staging/lustre/lnet/lnet/lib-eq.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
--
1.9.1
The only board served by this driver always has a timer, so we can init
the timer subdevice without checking first.
The boardinfo about the timer can also be removed and the data used
directly.
Signed-off-by: Chase Southwood chase.southw...@gmail.com
Cc: Ian Abbott abbo...@mev.co.uk
Cc: H
This driver only supports a single board type. Remove the boardinfo and
its use in the driver.
Signed-off-by: Chase Southwood chase.southw...@gmail.com
Cc: Ian Abbott abbo...@mev.co.uk
Cc: H Hartley Sweeten hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/addi_apci_1564.c | 13
On Thu, 2014-05-01 at 11:45 +0200, Christian Engelmayer wrote:
Fix a potential leak in the error path of r871x_wx_set_enc_ext(). In case the
requested algorithm is not supported by the driver, the function returns
without freeing the already allocated 'param' struct. Move the input
On Wed, May 07, 2014 at 07:23:33AM +0100, John Church wrote:
This patch fixes a coding style issue. It modifies a single line
that was over 80 characters long.
Thanks,
John
Signed-off-by: John Church j.chu...@ymail.com
This patch isn't sent in the correct way. Use git send-email or
read
On Wed, May 07, 2014 at 09:55:47AM +0300, Andy Shevchenko wrote:
@@ -1824,6 +1817,15 @@ static int r871x_wx_set_enc_ext(struct net_device
*dev,
default:
return -EINVAL;
}
+
+ param_len = sizeof(struct ieee_param) + pext-key_len;
+ param = (struct ieee_param
On 2014/05/07 08:02 AM, Chase Southwood wrote:
This patchset removes the last dependency on this driver's boardinfo (the
timer subdevice init), and then removes the boardinfo from the driver.
To be applied on top of my prior patchset staging: comedi:
addi_apci_1564: further cleanups
Chase
On 2014/05/07 12:08 AM, H Hartley Sweeten wrote:
The Diamond Systems OPMM-1616-XT board is a PC/104 module with 16
optoisolated inputs and 16 relay outputs.
The PEARL-MM-P board is a PC/104 module with 16 relay outputs.
Both of these boards can be supported by the pcl730 driver.
Hi All,
This is patch for fixing of minor coding style problems. This is part
of Eudyptula Challenge.
Thanks,
Adithya.K
diff --git a/drivers/staging/gdm72xx/gdm_qos.c
b/drivers/staging/gdm72xx/gdm_qos.c
index 50d43ad..5ed32b9 100644
--- a/drivers/staging/gdm72xx/gdm_qos.c
+++
On Wed, May 07, 2014 at 04:05:27PM +0530, Adithya wrote:
Hi All,
This is patch for fixing of minor coding style problems. This is part
of Eudyptula Challenge.
Thanks,
Adithya.K
Can you please read the Documentation/SubmittingPatches file and then
resend this in a format I can apply it
These days most people don't use git to send patches so I have added a
section about that.
Signed-off-by: Dan Carpenter dan.carpen...@oracle.com
diff --git a/Documentation/email-clients.txt b/Documentation/email-clients.txt
index 4e30eba..9004a5fd 100644
--- a/Documentation/email-clients.txt
+++
On Wed, May 07, 2014 at 04:05:27PM +0530, Adithya wrote:
Hi All,
This is patch for fixing of minor coding style problems. This is
part of Eudyptula Challenge.
Write what you fixed.
Thanks,
Adithya.K
Not in the correct format. Use git send-email or read
Fix smatch warning dgnc_cleanup_board() info:
redundant null check on brd-flipbuf calling kfree()
in dgnc_driver.c
Signed-off-by: Ebru Akagunduz ebru.akagun...@gmail.com
---
drivers/staging/dgnc/dgnc_driver.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
This patch fixes a coding style issue for a line that was over 80 characters
long.
Signed-off-by: John Church sleeverol...@gmail.com
---
drivers/staging/android/timed_gpio.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/android/timed_gpio.c
On Wed, May 07, 2014 at 03:39:15PM +0300, Dan Carpenter wrote:
I wonder if there is some kind of tutorial which teaches people to
choose the least specific subject possible? We should find which
tutorial it is and update it. John, can you give us a hint here which
tutorial you are using?
Fixes the following warning:
drivers/staging/wlags49_h2/sta_h2.c:4471:10: warning: symbol 'fw_image' was not
declared. Should it be static?
Signed-off-by: Luis Ortega luior...@gmail.com
---
drivers/staging/wlags49_h2/sta_h2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On Wednesday, May 07, 2014 12:03 AM, Chase Southwood wrote:
This patchset removes the last dependency on this driver's boardinfo (the
timer subdevice init), and then removes the boardinfo from the driver.
To be applied on top of my prior patchset staging: comedi:
addi_apci_1564: further
From: Fernando Apesteguia fernando.apesteg...@gmail.com
Add blank line after declarations and delete extra blank line at the
beginning of the function
Signed-off-by: Fernando Apesteguia fernando.apesteg...@gmail.com
---
drivers/staging/silicom/bpctl_mod.c |2 +-
1 file changed, 1
On 5/7/2014 4:17 AM, Dan Carpenter wrote:
These days most people don't use git to send patches so I have added a
section about that.
Do you mean most people *do* use git to send patches? Or most people don't
use e-mail clients?
Laura
--
Qualcomm Innovation Center, Inc. is a member of Code
On Wed, May 07, 2014 at 09:06:54AM -0700, Laura Abbott wrote:
On 5/7/2014 4:17 AM, Dan Carpenter wrote:
These days most people don't use git to send patches so I have added a
section about that.
Do you mean most people *do* use git to send patches? Or most people don't
use e-mail
On Wed, May 07, 2014 at 06:02:44PM +0200, fernando.apesteg...@gmail.com wrote:
From: Fernando Apesteguia fernando.apesteg...@gmail.com
Add blank line after declarations and delete extra blank line at the
beginning of the function
Signed-off-by: Fernando Apesteguia
Pointer 'pwdev_priv' in function rtw_wdev_free() is unused - thus remove it.
Signed-off-by: Christian Engelmayer cenge...@gmx.at
---
v2: Resend after v1 failed to apply
* rebased against staging-next - commit 09c3fbba (staging: rtl8188eu:
Remove 'u8 *pbuf' from struct recv_buf)
*
Function rtw_mp_QueryDrv() dynamically allocates a temporary buffer that
is not freed in all error paths. Use a centralized exit path and make sure
that all memory is freed correctly. Detected by Coverity - CID 1077713.
Signed-off-by: Christian Engelmayer cenge...@gmx.at
---
This is a cleanup of staging/rtl8188eu/os_dep/ioctl_linux.c regarding Coverity
resource leak findings.
The changes leave the current implementation intact and just attack the problems
in the error paths, however, it seems that we could get easily rid of some of
the mallocs altogether.
char
Function rtw_mp_pwrtrk() dynamically allocates a temporary buffer that
is not freed in all error paths. Use a centralized exit path and make sure
that all memory is freed correctly. Detected by Coverity - 1077715.
Signed-off-by: Christian Engelmayer cenge...@gmx.at
---
This driver supports the PCI-20001C-1a and PCI-20001C-2a carrier boards.
The -2a version has 32 on-board DIO channels. In case this variant is
detected, local variable 'has_dio' is set accordingly. Otherwise it is
left uninitialized and the following subdevice instantiation depends on
the stack.
Function rtw_mp_SetRFPath() dynamically allocates a temporary buffer that
is not freed in all error paths. Use a centralized exit path and make sure
that all memory is freed correctly. Detected by Coverity - CID 1077714.
Signed-off-by: Christian Engelmayer cenge...@gmx.at
---
Fix potential leaks in the error paths of r8712_set_key(). In case the
algorithm specific checks fail, the function returns without enqueuing
or freeing the already allocated command and parameter structs. Use a
centralized exit path and make sure that all memory is freed correctly.
Detected by
Fix a potential leak in the error path of r871x_wx_set_enc_ext(). In case the
requested algorithm is not supported by the driver, the function returns
without freeing the already allocated 'param' struct. Move the input
verification to the beginning of the function so that the direct return is
Fix a potential leak in the error path of rtw_set_key(). In case the requested
algorithm is not supported by the driver, the function returns without
enqueuing or freeing the already allocated command and parameter structs. Use
a centralized exit path and make sure that all memory is freed
Fix the error path when a cookie mismatch is detected. In that case the
function jumps to the exit label without setting the uninitialized, local
variable 'return_error'. Detected by Coverity - CID 201453.
Signed-off-by: Christian Engelmayer cenge...@gmx.at
---
v2: Resend after v1 failed to apply
Commit fb841d67 (staging: vt6656: don't leak 'param' in vt6656_hostap_ioctl()
when returning -EOPNOTSUPP) cleaned up direct returns in the ioctl switch
statement that leaked already allocated memory. Fix the same issue for
VIAWGET_HOSTAPD_SCAN_REQ and VIAWGET_HOSTAPD_MLME that are not supported by
Function rtw_wx_set_enc_ext() dynamically allocates a temporary buffer that
is not freed in all error paths. Use a centralized exit path and make sure
that all memory is freed correctly. Detected by Coverity - CID 1077712.
Signed-off-by: Christian Engelmayer cenge...@gmx.at
---
Fix a potential leak in the error path of function update_bcn_wps_ie().
Move the affected input verification to the beginning of the function so
that it may return directly without leaking already allocated memory.
Detected by Coverity - CID 1077718.
Signed-off-by: Christian Engelmayer
Pointer 'pbpctl_dev_c' in function bypass_init_module() is unused. Thus remove
it. With the last variable declaration gone, there is no more need for an own
block. Remove it and adapt the indenting accordingly.
Signed-off-by: Christian Engelmayer cenge...@gmx.at
---
v3: Resend after v2 failed to
Function rtw_wx_read32() dynamically allocates a temporary buffer that is not
freed in all error paths. Use a centralized exit path and make sure that all
memory is freed correctly. Detected by Coverity - CID 1077711.
Signed-off-by: Christian Engelmayer cenge...@gmx.at
---
On Wed, May 07, 2014 at 09:33:20PM +0200, Christian Engelmayer wrote:
Function rtw_mp_SetRFPath() dynamically allocates a temporary buffer that
is not freed in all error paths. Use a centralized exit path and make sure
that all memory is freed correctly. Detected by Coverity - CID 1077714.
-Original Message-
From: David Miller [mailto:da...@davemloft.net]
Sent: Friday, May 2, 2014 3:19 PM
To: Haiyang Zhang
Cc: jim.ep...@gmail.com; s...@canb.auug.org.au; linux-n...@vger.kernel.org;
linux-ker...@vger.kernel.org; KY Srinivasan; de...@linuxdriverproject.org;
Hi Lee
It seems Chris is too busy to responding, so would you help to pick this
patch for 3.15 fix.
Best Regards.
micky.
On 04/29/2014 03:30 PM, Ulf Hansson wrote:
On 29 April 2014 03:54,micky_ch...@realsil.com.cn wrote:
From: Micky Chingmicky_ch...@realsil.com.cn
This reverts commit
Hi Lee
Sorry for previous email, only [PATCH 2/2] mmc: rtsx: Revert mmc: rtsx:
add support for pre_reqand post_req
if need for 3.15 fix.
Best Regards.
micky.
On 04/29/2014 03:36 PM, Ulf Hansson wrote:
On 29 April 2014 03:54,micky_ch...@realsil.com.cn wrote:
From: Micky
43 matches
Mail list logo