From: Matteo Semenzato
The condition pxmitpriv->hwxmit_entry == 5 is always false because
HWXMIT_ENTRY is always 4.
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/core/rtw_xmit.c | 17 -
1 file changed, 4 insertions(+), 13 deletions(-)
diff --git a/driv
From: Matteo Semenzato
This patch fixes the buffer overflow caused by accessing index 4 of hwxmits.
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/core/rtw_xmit.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_xmit.c
b
From: Matteo Semenzato
Check if kmalloc succeded before using the pointer in memcpy.
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8712/rtl871x_mlme.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/rtl8712/rtl871x_mlme.c
b/drivers/staging/rtl8712
From: Matteo Semenzato
The bitmask should be used on msignals since the signals variable is not
used anywhere in the function.
Signed-off-by: Matteo Semenzato
---
drivers/staging/dgnc/dgnc_cls.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc
From: Matteo Semenzato
The comedi_cmd struct has an hole after chanlist_len that could contain
uninitialized
memory, this struct is copied to userspace.
Signed-off-by: Matteo Semenato
---
drivers/staging/comedi/comedi_fops.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers
le are ignoring good patches but you
> send the first email 4 minutes before sending the second email.
This patch is a RESEND because i forgot to remove PATCH 2/2 from the subject.
> On Sat, Mar 07, 2015 at 04:33:27PM +0100, Matteo Semenzato wrote:
> > From: Matteo Semenzato
>
> No
From: Matteo Semenzato
Return -ENOMEM if allocating brd->flipbuf fails.
Signed-off-by: Matteo Semenzato
---
drivers/staging/dgnc/dgnc_driver.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/staging/dgnc/dgnc_driver.c
b/drivers/staging/dgnc/dgnc_driver.c
index f177
Il giorno sab, 07/03/2015 alle 11.28 -0600, Larry Finger ha scritto:
> Have you actually tested this patch, or did you find this with some static
> checker?
>
> Larry
The double unlock was found using smatch.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body
From: Matteo Semenzato
The rtw_cmd_thread semaphore was being unlocked twice.
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/core/rtw_cmd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c
b/drivers/staging/rtl8188eu
From: Matteo Semenzato
The rtw_cmd_thread semaphore was being unlocked twice.
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/core/rtw_cmd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c
b/drivers/staging/rtl8188eu
From: Matteo Semenzato
This patch removes the assignment of a function parameter that has no
effect.
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8192e/rtllib_rx.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/rtl8192e/rtllib_rx.c
b/drivers/staging/rtl8192e
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space prohibited before that ',' (ctx:WxW)
ERROR: space prohibited before that ',' (ctx:WxW)
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/core/rtw_sta_mgt.c | 14 +++---
1 file c
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space prohibited before that ',' (ctx:WxW)
ERROR: space prohibited before that ',' (ctx:WxW)
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 40 +--
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space prohibited before that ',' (ctx:WxW)
ERROR: space prohibited before that ',' (ctx:WxW)
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/core/rtw_mlme.c | 20 ++--
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space prohibited before that ',' (ctx:WxW)
ERROR: space prohibited before that ',' (ctx:WxW)
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/core/rtw_ieee80211.c | 8
1 file changed
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space prohibited before that ','
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/core/rtw_efuse.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space required after that ','
ERROR: trailing whitespace
ERROR: code indent should use tabs where possible
V2: The patch should apply now
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_st77
From: Matteo Semenzato
This patch fixes the following error:
space required after that ';'
V2: Indented and organized the init sequences.
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fbtft_device.c | 145 +++
1 file changed, 111 inserti
From: Matteo Semenzato
This patch fixes the following warnings:
space prohibited before that close parenthesis ')'
space prohibited after that open parenthesis '('
V2: The patch should apply now.
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft
Il giorno lun, 02/03/2015 alle 12.29 +0300, Dan Carpenter ha scritto:
> On Sat, Feb 28, 2015 at 06:37:30PM +0100, Matteo Semenzato wrote:
> > static int hy28b_init_sequence[] = {
> > - -1,0x00e7,0x0010,-1,0x,0x0001,-1,0x0001,0x0100,-1,0x0002,0x0700,
> > - -1,0x0003,0x
On Sun, 1 Mar 2015 10:05:06 -0800, Greg KH wrote:
> What it is being used for? Can it be removed? Is it just there for
> informational purposes or does the code actually do something based on
> the value?
The value is used to create a file called version in the driver's sysfs
directory, the file
Il giorno dom, 01/03/2015 alle 09.26 -0800, Greg KH ha scritto:
> On Sun, Mar 01, 2015 at 06:19:57PM +0100, Matteo Semenzato wrote:
> > From: Matteo Semenzato
> >
> > This patch removes EXTRA_CFLAGS from the Makefile and defines DDG_PART
> > in dgnc_driver.h
>
From: Matteo Semenzato
This patch removes EXTRA_CFLAGS from the Makefile and defines DDG_PART
in dgnc_driver.h
Signed-off-by: Matteo Semenzato
---
drivers/staging/dgnc/Makefile | 2 --
drivers/staging/dgnc/dgnc_driver.h | 1 +
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a
From: Matteo Semenzato
This patch fixes the following warning:
Use of EXTRA_CFLAGS is deprecated, please use `ccflags-y instead.
Signed-off-by: Matteo Semenzato
---
drivers/staging/dgnc/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc/Makefile
From: Matteo Semenzato
This patch fixes the following warning:
Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev,
... then pr_err(... to printk(KERN_ERR ...
Signed-off-by: Matteo Semenzato
---
drivers/staging/dgnc/dgnc_sysfs.c | 4 ++--
1 file changed, 2 insertions(+), 2
From: Matteo Semenzato
This patch fixes the following error:
space required after that ';'
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fbtft_device.c | 145 +++
1 file changed, 111 insertions(+), 34 deletions(-)
diff --git a/drivers/sta
From: Matteo Semenzato
This patch fixes the following errors:
spaces required around that '=' (ctx:VxV)
space required after that ',' (ctx:VxV)
space required after that ';' (ctx:VxV)
Signed-off-by: Matteo Semenzato
---
driver
From: Matteo Semenzato
This patch fixes the following error:
code indent should use tabs where possible
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/flexfb.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/fbtft/flexfb.c b/drivers
From: Matteo Semenzato
This patch fixes the following errors:
spaces required around that '=' (ctx:VxV)
space required after that ',' (ctx:VxV)
space required after that ';' (ctx:VxV)
Signed-off-by: Matteo Semenzato
---
drive
From: Matteo Semenzato
This patch fixes the following error:
space required after that ';'
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fbtft_device.c | 68 ++--
1 file changed, 34 insertions(+), 34 deletions(-)
diff --git a/drivers/sta
From: Matteo Semenzato
This patch fixes the following warning:
do not use C99 // comments
Signed-off-by: Matteo Semenzato
---
drivers/staging/vt6655/mac.c | 82 ++--
1 file changed, 41 insertions(+), 41 deletions(-)
diff --git a/drivers/staging/vt6655
From: Matteo Semenzato
This patch fixes the following error:
ERROR: do not use C99 // comments
Signed-off-by: Matteo Semenzato
---
drivers/staging/vt6655/key.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6655/key.h b/drivers/staging/vt6655/key.h
index
From: Matteo Semenzato
This patch fixes the following error:
ERROR: do not use C99 // comments
Signed-off-by: Matteo Semenzato
---
drivers/staging/vt6655/device_main.c | 35 ++-
1 file changed, 18 insertions(+), 17 deletions(-)
diff --git a/drivers/staging
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: code indent should use tabs where possible
ERROR: space prohibited before that
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c | 4 ++--
drivers/staging/rtl8188eu/hal/odm.c
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space prohibited before that ',' (ctx:WxW)
ERROR: space prohibited before that ',' (ctx:WxW)
ERROR: switch and case should be at the same indent
Signed-off-by: Matteo Semenzato
---
drivers/staging/rtl818
From: Matteo Semenzato
This patch fixes the following warning:
__aligned(size) is preferred over __attribute__((aligned(size)))
Signed-off-by: Matteo Semenzato
---
drivers/staging/emxx_udc/emxx_udc.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/emxx_udc
From: Matteo Semenzato
This patch fixes the following errors:
Code indent should use tabs where possible
Space prohibited before that ','
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fbtft-core.c | 2 +-
drivers/staging/fbtft/fbtft.h | 8
2 files
From: Matteo Semenzato
This patch fixes the following error:
Do not use C99 // comments
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_ssd1331.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/staging/fbtft/fb_ssd1331
From: Matteo Semenzato
This patch fixes the following error:
space prohibited before that ','
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_ili9340.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/fbtft/fb_ili9340.c
From: Matteo Semenzato
This patch fixes the following error:
space prohibited before that ','
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_ra8875.c | 150 +++---
1 file changed, 75 insertions(+), 75 deletions(-)
diff --git a/drive
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space required after that ','
ERROR: trailing whitespace
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_st7735r.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space required after that ','
ERROR: trailing whitespace
Signed-off-by Matteo Semenzato
---
drivers/staging/fbtft/fb_st7735r.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git
From: Matteo Semenzato
This patch fixes the following errors:
ERROR: space required after that ','
ERROR: trailing whitespace
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_upd161704.c | 90 ++--
1 file changed, 45 insertions(+), 45
From: Matteo Semenzato
This patch fixes the following warnings:
space prohibited before that close parenthesis ')'
space prohibited after that open parenthesis '('
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_bd663474.c | 100 ++--
From: Matt
This patch fixes the following warnings:
space prohibited before that close parenthesis ')'
space prohibited after that open parenthesis '('
Signed-off-by: Matteo Semenzato
---
drivers/staging/fbtft/fb_bd663474.c | 100 ++--
45 matches
Mail list logo
