This patch is to the rtl871x_ioctl.h that fixes up following warning
reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_ioctl.h | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/d
This patch is to the rtl871x_ioctl_set.c file that fixes up following
warning reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_ioctl_set.c | 12
1 file changed, 8 insertions(+), 4 deletion
This patch is to the rtl871x_io.h file that fixes up following warning
reported by checkpatch.pl :
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_io.h | 15 +--
1 file changed, 5 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/rtl8712/rtl871x_io.h
b/drive
Hi,
smatch found a possible buffer overflow in staging/wlan-ng, the first patch
of this series fixes this issue. The remaining two patches clean up
redundant code.
Cheers
Tillmann
Tillmann Heidsieck (3):
staging: wlan-ng fix buffer overflow in firmware handling
staging: wlan-ng remove redund
We exit the above loop either if curroff >= (HFA384x_PDA_LEN_MAX / 2 - 1)
or if we found the END marker in the element beyond the current one. The
first case is checked for in the preceding if statement, therefore the
second if statement is redundant and can be removed.
Signed-off-by: Tillmann Hei
The result variable is not set anywhere beyond its initialization,
therefore it can be remove.
Signed-off-by: Tillmann Heidsieck
---
drivers/staging/wlan-ng/prism2fw.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/prism2fw.c
b/drivers/staging/wlan
We test for an END marker in the element beyond the current one, this
effectively limits the size of the array to be HFA384x_PDA_LEN_MAX/2 - 1
not HFA384x_PDR_END_OF_PDA/2. This patch fixes a possible buffer
overflow in case there was no END marker.
Signed-off-by: Tillmann Heidsieck
---
drivers/
This patch is to the rtl871x_led.h that fixes up following warnings
reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_led.h | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff -
This patch is to the rtl871x_mlme.c file that fixes up following warning
reported by checkpatch.pl:
- Block comments use * on subsequent lines
- Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_mlme.c | 45
This patch is to the rtl871x_mlme.h file that fixes up following warning
reported by checkpatch.pl :
-Block comments use * on subsequent lines
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_mlme.h | 28 -
This patch is to the rtl871x_mp.c file that fixes up following warning
reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit vara
---
drivers/staging/rtl8712/rtl871x_mp.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/driver
This is a patch to the rtl871x_mp.h that fixes up following warnings
reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_mp.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/
This patch is for the rtl871x_mp_ioctl.h that fixes up following warning
reported by checkpatch.pl:
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_mp_ioctl.h | 39 --
1 file changed, 26 insertions(+)
This patch is to the rtl871x_mp_phy_regdef.h file that fixes up
following warning reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_mp_phy_regdef.h | 33 -
1 file changed, 22 in
On Wednesday, September 23, 2015 11:36 AM, Ian Abbott wrote:
> Expand the descriptions of the functions and document the return values.
>
> Signed-off-by: Ian Abbott
Thanks,
Reviewed-by: H Hartley Sweeten
___
devel mailing list
de...@linuxdriverproje
This is patch to the rtl871x_pwrctrl.c file that fixes following warning
reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
-Block comments use * on subsequent lines
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_pwrctrl.c | 41 --
This patch is to the rtl871x_recv.c that fixes up following warnings
reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
-Block comments use * on subsequent lines
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_recv.c | 27 ---
This patch is to the rtl871x_recv.h file that fixes up following
warnings reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
-Block comments use * on subsequent lines
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_recv.h | 21 -
1
This is patch to the rtl871x_security.c that fixes up following warning
reported by checkpatch.pl :
-Block comments use * on subsequent lines
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_security.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/s
This series of patches eliminates some sparse warnings concerning
static variables in lustre rtlrpc module where it seemed reasonable.
Symbols that are really local to a source file are made static and
declarations added to a header for symbols that are shared between
source files.
v2: Thank Andre
This patch moves declarations of variables used in several files
across the ptlrpc module from source files to a local header.
Signed-off-by: Anton Gerasimov
---
drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c| 1 -
drivers/staging/lustre/lustre/ptlrpc/nrs.c | 1 -
drivers/st
Include the header with declarations of variables that are shared between
several sources in ptlrpc module into the appropriate source files.
Signed-off-by: Anton Gerasimov
---
drivers/staging/lustre/lustre/ptlrpc/sec_null.c | 2 ++
drivers/staging/lustre/lustre/ptlrpc/wiretest.c | 2 ++
2 files
This patch is to the rtl871x_security.h file that fixes up following
warning reported by checkpatch.pl :
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_security.h | 24
1 file changed, 16 insertions(+),
Declare non-exported functions as static.
Signed-off-by: Anton Gerasimov
---
drivers/staging/lustre/lustre/ptlrpc/pack_generic.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/lustre/lustre/ptlrpc/pack_generic.c
b/drivers/staging/lustre/lustre/ptlrpc
This is a patch to the rtl871x_sta_mgt.c file that fixes up following
warning reported by checkpatch.pl:
-Block comments use a trailing */ on a separate line
Signed-off-by: Punit Vara
---
drivers/staging/rtl8712/rtl871x_sta_mgt.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --g
Add cmd_flags parameter to all dpmng APIs to comply
with 8.0 MC firmware interface. Updated MC version
major number.
---
drivers/staging/fsl-mc/bus/dpmng.c | 14 ++
drivers/staging/fsl-mc/bus/mc-bus.c| 4 ++--
drivers/staging/fsl-mc/include/dpmng.h | 14 ++
3 files
The Management Complex (MC) binary interface added a new "flags"
field to the command header.
Add the definitions for this field in preparation for adding the
new cmd_flags parameter to all MC interface APIs.
Signed-off-by: J. German Rivera
---
drivers/staging/fsl-mc/include/mc-cmd.h | 30 ++
dpcon object minor version number updated to match latest
MC firmware. This change is needed because the dpcon
object binds to the allocator and the current driver match
logic uses object version numbers.
Signed-off-by: J. German Rivera
---
drivers/staging/fsl-mc/include/dpcon-cmd.h | 2 +-
1 fi
Add cmd_flags parameter to all dpbp APIs to comply
with the dpmcp 2.0 MC interface. Updated version
major number. Pass irq args in struct instead of
separate args.
Signed-off-by: J. German Rivera
---
drivers/staging/fsl-mc/bus/dpmcp-cmd.h| 2 +-
drivers/staging/fsl-mc/bus/dpmcp.c|
Add cmd_flags parameter to all dpbp APIs to comply
with the dpbp 2.0 MC interface. Updated MC version
major number. Pass irq args in struct instead of
separate args.
Signed-off-by: J. German Rivera
---
drivers/staging/fsl-mc/bus/dpbp.c | 94 +++
drivers/stagin
The DPAA2 management complex has a versioned binary interface
that has to be kept in sync with the DPAA2 drivers. This patch
uprevs the APIs that build MC commands and parse results.
This uprev is needed to support object interrupts.
MC binary interface changes
-overall version from 6.0 to 8.0
Add cmd_flags parameter to all dprc APIs to comply
with the dprc 4.0 MC interface. Updated MC version
major number. Pass irq args in struct instead of
separate args.
dprc 4.0 uses MC-relative offsets to specify object regions,
instead of physical addresses. So, translate_mc_addr() and
struct fsl_
The kbuild test robot reports the following build error for
i386-randconfig-c0-09230740:
>> ERROR: "ieee80211_hdrlen" [drivers/staging/rtl8188eu/r8188eu.ko] undefined!
Add a dependency on CFG80211 to fix it.
Signed-off-by: Jakub Sitnicki
---
Same issue has also been reported by Jim Davis for
r
On Wed, Sep 23, 2015 at 12:10:13PM -0500, atull wrote:
> On Tue, 22 Sep 2015, Josh Cartwright wrote:
[..]
> > > +struct fpga_manager *of_fpga_mgr_get(struct device_node *node)
> > > +{
> > > + struct fpga_manager *mgr;
> > > + struct device *dev;
> > > +
> > > + if (!node)
> > > + return ER
On Fri, Jun 26, 2015 at 01:36:14PM -0300, Gaston Gonzalez wrote:
> On Thu, Jun 25, 2015 at 02:06:44PM +0200, Arnd Bergmann wrote:
> > On Wednesday 24 June 2015 13:34:58 Gaston Gonzalez wrote:
> > > On Tue, Jun 23, 2015 at 12:13:47PM +0200, Arnd Bergmann wrote:
> > > > On Sunday 21 June 2015 19:12:0
On Sep 23, 2015, at 3:13 PM, Arnd Bergmann wrote:
> The last request time is stored as an 'unsigned long', which is
> good enough until 2106, but it is then converted to 'long' in
> some places, which overflows in 2038.
>
> This changes the type to time64_t to avoid those problems.
Hm…
All this
On Sep 23, 2015, at 3:13 PM, Arnd Bergmann wrote:
> The lustre tracefile has a timestamp defined as
>
> __u32 ph_sec;
> __u64 ph_usec;
>
> which seems completely backwards, as the microsecond portion of
> a time stamp will always fit into a __u32 value, while the second
> portion wi
On Wed, Sep 23, 2015 at 11:22:09PM +0530, punit vara wrote:
> On Wed, Sep 23, 2015 at 10:25 PM, punit vara wrote:
> > Thank you very much sir I will definitely follow that. I have seen so
> > many people fixing patches over here. Now I know process of submitting
> > patches.I am able to fix severa
On Wed, Sep 23, 2015 at 09:13:42PM +0200, Arnd Bergmann wrote:
> This time is only printed in procfs, and can be easily converted
> to 64-bit to avoid overflowing on 32-bit systems in 2038.
>
> Signed-off-by: Arnd Bergmann
> ---
This introduces new build warnings:
drivers/staging/lustre/lustre/o
On Thu, Sep 24, 2015 at 01:22:33AM +0530, Punit Vara wrote:
> This patch is to the rtl871x_io.h file that fixes up following warning
> reported by checkpatch.pl :
>
> Signed-off-by: Punit Vara
> ---
You missed copying the warning after mentioning "following warning" in
your commit message.
regar
On Thu, Sep 24, 2015 at 02:38:26AM +0530, Punit Vara wrote:
> This is a patch to the rtl871x_sta_mgt.c file that fixes up following
> warning reported by checkpatch.pl:
>
> -Block comments use a trailing */ on a separate line
>
> Signed-off-by: Punit Vara
> ---
When you are sending multiple patc
This series of patches addresses several checkpatch.pl issues in
ieee80211_softmac_wx.c.
Before: total: 14 errors, 24 warnings, 59 checks, 602 lines checked
After: total: 0 errors, 20 warnings, 59 checks, 599 lines checked
Mike Dupuis (6):
Staging: rtl8192u: space required after ','
Staging:
This is a patch to add spaces around = operators in several instances
in ieee80211_softmac_wx.c
Signed-off-by: Mike Dupuis
---
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac_wx.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee802
This is a patch to correct two flavors of brace styling issues in
ieee80211_softmac_wx.c.
Signed-off-by: Mike Dupuis
---
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac_wx.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee8
This is a patch to add blank lines after declaration ins several
instances in iee80211_softmac_wx.c.
Signed-off-by: Mike Dupuis
---
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac_wx.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softma
This is a fix to add spaces after commas in ieee80211_softmac_wx.c.
Signed-off-by: Mike Dupuis
---
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac_wx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac_wx.c
b/drivers/st
This is a patch to move opening braces to the lines on which their
corresponding if-statements exist in ieee80211_softmac_wx.c.
Signed-off-by: Mike Dupuis
---
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac_wx.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/dr
This is a patch to add missing spaces around == and || operators in one
instance in ieee80211_softmac_wx.c.
Signed-off-by: Mike Dupuis
---
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac_wx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/ieee80
101 - 148 of 148 matches
Mail list logo
