Fix a pointer check to use NULL instead of 0
Warning:
drivers/staging/goldfish/goldfish_audio.c:300:34: warning: Using plain
integer as NULL pointer
Signed-off-by: Robin Schroer
---
drivers/staging/goldfish/goldfish_audio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dri
Hi Martyn,
Thanks for your patience with me. I have a couple of questions for you:
0. I put the SBC with the right settings for Geographical addressing.
I did 2 tests by setting the board in each of the 2 slots available on
my rack and the geo address was detected as 0 in both the cases. This
m
A 'Missing a blank line after declarations' warning fixed by inserting
a blank line after struct pointer declaration.
Signed-off-by: Sam Asadi
---
drivers/staging/comedi/drivers/amplc_pc236.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/comedi/drivers/amplc_pc236.c
b/d
From: Wei Yongjun
Fix to return -ENOMEM from the kalloc error handling
case instead of 0.
Signed-off-by: Wei Yongjun
---
drivers/net/hyperv/netvsc.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/net/hyperv/netvsc.c b/drivers/net/hyperv/netvsc.c
index 4ed38ea..d
Put extern declarations in cxt1e1_common.h to reduce sparse warnings for
linux.c:
drivers/staging/cxt1e1/linux.c:86:13: warning:
symbol 'cxt1e1_error_flag' was not declared. Should it be static?
drivers/staging/cxt1e1/linux.c:91:13: warning:
symbol 'cxt1e1_max_mru' was not declared. Should it
Changed names of ambiguous sounding variable names as follows
error_flag -> cxt1e1_error_flag
max_mtu_default -> cxt1e1_max_mtu_default
max_txdesc_used -> cxt1e1_max_txdesc_used
max_txdesc_default -> cxt1e1_max_txdesc_default
max_rxdesc_used -> cxt1e1_max_rxdesc_used
max
This patch set:
changes ambiguous global names, such as "error_flag" to "cxt1e1_error_flag"
puts globals in header file to reduce externs used in c files as well as sparse
warnings
Jeff Oczek (2):
staging: cxt1e1: Prefix ambiguous variable names with 'cxt1e1_' for
clarity
staging: cxt1e1:
On Tue, Jul 22, 2014 at 09:56:29AM -0400, Benjamin Romer wrote:
> Move the /proc/visorchipset/parahotplug interface to sysfs under
> /sys/devices/platform/visorchipset/guest/parahotplug.
>
> The parahotplug interface is used to deal with recovery situations on s-Par
> guest partitions. The command
On Tue, Jul 22, 2014 at 09:56:28AM -0400, Benjamin Romer wrote:
> Move the chipsetready proc entry to sysfs under a new directory guest. This
> entry is used by Unisys application software on the guest to acknowledge
> completion of specific events for integration purposes, but these
> acknowledgem
On Tue, Jul 22, 2014 at 09:56:28AM -0400, Benjamin Romer wrote:
> Move the chipsetready proc entry to sysfs under a new directory guest. This
> entry is used by Unisys application software on the guest to acknowledge
> completion of specific events for integration purposes, but these
> acknowledgem
On Tue, Jul 22, 2014 at 09:56:32AM -0400, Benjamin Romer wrote:
> This patch adds a documentation file for all of the interfaces that were moved
> to sysfs by the other patches in this set.
>
> Signed-off-by: Benjamin Romer
> ---
> v2: whitespace errors were corrected.
>
> .../Documentation/ABI
On Tue, Jul 22, 2014 at 09:56:27AM -0400, Benjamin Romer wrote:
> The installer entry in /proc/visorchipset/installer was composed of three
> separate fields as one entry. This patch removes the proc entry and associated
> functions, and creates new fields with distinct entries under sysfs in the
>
On Tue, Jul 22, 2014 at 10:08:26PM +0200, Guillaume Clement wrote:
> PortOffset was an unsigned long, but used as an pointer to io
> memory. Sometimes it was not properly cast before use, which caused
> many warning by sparse.
>
> By updating its type to void __iomem *, and reflecting the changes
On Tue, Jul 22, 2014 at 04:56:59PM -0400, Jeff Oczek wrote:
> On Tue, Jul 22, 2014 at 01:17:22PM -0700, Greg KH wrote:
> > On Tue, Jul 22, 2014 at 04:07:51PM -0400, Jeff Oczek wrote:
> > > Put extern declarations in cxt1e1_common.h to reduce sparse warnings for
> > > linux.c:
> >
> > I know you d
On Tue, Jul 22, 2014 at 01:17:22PM -0700, Greg KH wrote:
> On Tue, Jul 22, 2014 at 04:07:51PM -0400, Jeff Oczek wrote:
> > Put extern declarations in cxt1e1_common.h to reduce sparse warnings for
> > linux.c:
>
> I know you didn't name this variable, but wow, that's a horrid name for
> a global v
Some variables are used only in the context of their .c file, which
gives warnings with sparse.
Signed-off-by: Guillaume Clement
---
drivers/staging/vt6655/baseband.c| 14 +++---
drivers/staging/vt6655/bssdb.c | 4 ++--
drivers/staging/vt6655/card.c| 6 +++---
drivers
On Tue, Jul 22, 2014 at 04:07:51PM -0400, Jeff Oczek wrote:
> Put extern declarations in cxt1e1_common.h to reduce sparse warnings for
> linux.c:
> drivers/staging/cxt1e1/linux.c:86:13: warning:
> symbol 'error_flag' was not declared. Should it be static?
> drivers/staging/cxt1e1/linux.c:91:13:
PortOffset was an unsigned long, but used as an pointer to io
memory. Sometimes it was not properly cast before use, which caused
many warning by sparse.
By updating its type to void __iomem *, and reflecting the changes
where it is needed, this removes most of those warnings.
Signed-off-by: Guil
The IEEE11hbMgrRxAction is not exported and never used. Deleting it
allows to delete other functions that were only used by
IEEE11hbMgrRxAction.
This allows to fix several warnings reported by sparse.
Signed-off-by: Guillaume Clement
---
drivers/staging/vt6655/IEEE11h.c | 175 --
There is one function in aes_ccmp.c which is exported, but sparse sees
it unexported because it doesn't include the header that exports it.
Signed-off-by: Guillaume Clement
---
drivers/staging/vt6655/aes_ccmp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/vt6655/aes_ccmp.c
Sparse reports that MimeThread is not used. Actually, it can be used
if THREAD is defined. By enclosing the MimeThread function into the
same #ifdef as the caller of MimeThread, this fixes the sparse
warnings.
Signed-off-by: Guillaume Clement
---
drivers/staging/vt6655/device_main.c | 6 +-
Those patches fix warnings reported by sparse on the vt6655
driver. This will allow to more easily find real errors with sparse
later if one appears.
Note that checkpatch does return an error for patch 1 and 2 (line over
80 characters) but the line length was not increased by this patch. If
neede
Put extern declarations in cxt1e1_common.h to reduce sparse warnings for
linux.c:
drivers/staging/cxt1e1/linux.c:86:13: warning:
symbol 'error_flag' was not declared. Should it be static?
drivers/staging/cxt1e1/linux.c:91:13: warning:
symbol 'cxt1e1_max_mru' was not declared. Should it be stat
Replace use of NLMSG_SPACE(0) with NLMSG_HDRLEN as they are equivalent
and NLMSG_SPACE seems to be deprecated.
Signed-off-by: Himangi Saraogi
---
To send to: Greg Kroah-Hartman
,de...@driverdev.osuosl.org,linux-ker...@vger.kernel.org
drivers/staging/gdm724x/netlink_k.c | 2 +-
1 file changed,
On 21/07/2014 17:16, Bjorn Helgaas wrote:
> [+cc Jingoo]
>
> On Fri, Jul 18, 2014 at 12:50 PM, James Bottomley
> > Anyway, it's PCI code ... let the PCI maintainer decide. However, if he
> > does want this do it as one big bang patch via either the PCI or Trivial
> > tree (latter because Jiří has
This patch fixes the following checkpatch error:
ERROR: do not use C99 // comments
CC: Lidza Louina
Signed-off-by: Seunghun Lee
---
drivers/staging/dgnc/dgnc_driver.h | 10 +-
drivers/staging/dgnc/digi.h| 36 ++--
2 files changed, 23 insertio
This patch series modifies the hmc5843 driver to support the hmc5983 i2c and
spi interfaces.
v2:
* Reverted the changed order of iio unregister and setting hmc mode to
sleep
v3:
* Fixed bug introduced in the first patch
* Readded few comment lines which have gone missing
This patch adds support for the hmc5983 spi interface.
This chip is almost identical to the hmc5883. The difference being added
temperature compensation, additional available sample rate (220Hz) and an SPI
interface.
Signed-off-by: Josef Gajdusek
---
drivers/staging/iio/magnetometer/Kconfig
This patch adds support for the hmc5983 i2c interface.
This chip is almost identical to the hmc5883. The difference being added
temperature compensation, additional available sample rate (220Hz) and an SPI
interface.
Signed-off-by: Josef Gajdusek
---
.../devicetree/bindings/iio/magnetometer/hmc5
Changed structure of struct hmc5843_chip_info to include length of translation
arrays. Code previously using #defined constant has been changed accordingly.
This allows to integrate devices which do have different amounts of available
rates/scales.
Signed-off-by: Josef Gajdusek
---
drivers/stagi
This patch splits hmc5843.c to multiple files - the interface-agnostic
hmc5843_core.c, i2c specific hmc5843_i2c.c and header file hmc5843.h. This is
another step to add support of SPI-enabled hmc5983.
Signed-off-by: Josef Gajdusek
---
drivers/staging/iio/magnetometer/Kconfig | 14 +-
This patch changes hmc5843.c to use regmap. This provides transparent caching
to the code as well as abstraction necessary to add support for SPI-based
hmc5983.
Signed-off-by: Josef Gajdusek
---
drivers/staging/iio/magnetometer/Kconfig | 1 +
drivers/staging/iio/magnetometer/hmc5843.c | 140
This patch fix spelling typo in comments within rtw_xmit.c
Signed-off-by: Masanari Iida
---
drivers/staging/rtl8723au/core/rtw_xmit.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c
b/drivers/staging/rtl8723au/core/rtw_x
This patch fixes formatting and cleans up unneeded comments
Signed-off-by: Erik Arfvidson
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/virthba/virthba.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/virthba/virthba.c
b/drivers/staging/
This patch fix spelling typo in comments within
rtl8723au/core.
Signed-off-by: Masanari Iida
---
drivers/staging/rtl8723au/core/rtw_cmd.c | 4 ++--
drivers/staging/rtl8723au/core/rtw_efuse.c | 6 +++---
drivers/staging/rtl8723au/core/rtw_ieee80211.c | 2 +-
drivers/staging/rtl8723au/co
This patch fix spelling typo in comments within rtw_ap.c
Signed-off-by: Masanari Iida
---
drivers/staging/rtl8723au/core/rtw_ap.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_ap.c
b/drivers/staging/rtl8723au/core/rtw_a
This patch fix spelling typo in comments within rtw_mlme.c
Signed-off-by: Masanari Iida
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c
b/drivers/staging/rtl872
This patch fix spelling typo in comments within rtw_mlme_ext.c
Signed-off-by: Masanari Iida
---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
b/drivers/
Debugging information for the guest's channels was being exposed in proc.
Remove the code that creates these entries, which are no longer needed.
Signed-off-by: Benjamin Romer
---
v2: patch location changed due to prior patches being revised.
.../unisys/visorchipset/visorchipset_main.c|
The installer entry in /proc/visorchipset/installer was composed of three
separate fields as one entry. This patch removes the proc entry and associated
functions, and creates new fields with distinct entries under sysfs in the
visorchipset/install directory. The fields are:
textid: used t
This patch adds a documentation file for all of the interfaces that were moved
to sysfs by the other patches in this set.
Signed-off-by: Benjamin Romer
---
v2: whitespace errors were corrected.
.../Documentation/ABI/sysfs-platform-visorchipset | 74 ++
1 file changed, 74 in
Move the /proc/visorchipset/parahotplug interface to sysfs under
/sys/devices/platform/visorchipset/guest/parahotplug.
The parahotplug interface is used to deal with recovery situations on s-Par
guest partitions. The command service partition will send a message to a guest
when a device that guest
Move the chipsetready proc entry to sysfs under a new directory guest. This
entry is used by Unisys application software on the guest to acknowledge
completion of specific events for integration purposes, but these
acknowledgements are not required for the guest to operate correctly.
The store fun
Move the proc entry for controlling the toolaction field to sysfs. The field
appears in /sys/devices/platform/visorchipset/install/toolaction.
This field is used to tell s-Par which type of recovery tool action to perform
on the next guest boot-up. The meaning of the value is dependent on the type
This patch set moves several interfaces in the visorchipset module from procfs
to sysfs, and removes the remaining proc interfaces from the module. It includes
documentation for the new interfaces in Documentation/ABI, and some code cleanup
in the new sysfs handler functions.
In version 2 of the s
Move the proc entry controlling the boottotool flag to procfs. The field
appears in /sys/devices/platform/visorchipset/install/boottotool.
The boottotool flag controls s-Par behavior on the next boot of this guest.
Setting the flag will cause the guest to boot from the utility and installation
ima
Remove remnant code left over from the diagdump proc entry.
Signed-off-by: Benjamin Romer
---
v2: patch locations changed due to prior patches being revised.
drivers/staging/unisys/visorchipset/visorchipset_main.c | 8 +---
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/driver
On Sat, Jul 19, 2014 at 11:55:39AM +0200, LABBE Corentin wrote:
> Hello
>
> The drivers/staging/sep/sep_main.c have lots of checkpatch issue.
> This patch series solves them and since there are many, I have splitted
> corrections in 4 patch.
>
> Note that this work is done for the Eudyptulla Cha
On Mon, Jul 21, 2014 at 11:36:02PM +0200, Fabio Falzoi wrote:
> @@ -3664,7 +3672,7 @@ static int mg_set_tpc_para_sub(struct rtsx_chip *chip,
> int type,
> int retval;
> u8 buf[6];
>
> - RTSX_DEBUGP("--%s--\n", __func__);
> + dev_dbg(rtsx_dev(chip), "--%s--\n", __func__);
All
Removed redundant comments from aes_ccmp.h header file
Signed-off-by: Igor Bezukh
---
drivers/staging/vt6655/aes_ccmp.h |9 -
1 file changed, 9 deletions(-)
diff --git a/drivers/staging/vt6655/aes_ccmp.h
b/drivers/staging/vt6655/aes_ccmp.h
index cc02e64..fe0c506 100644
--- a/driver
Removed redundant comments and newlines from 80211mgr.h header file.
Signed-off-by: Igor Bezukh
---
drivers/staging/vt6655/80211mgr.h | 38 -
1 file changed, 38 deletions(-)
diff --git a/drivers/staging/vt6655/80211mgr.h
b/drivers/staging/vt6655/80211mgr.h
Removed redundant comments from 80211hdr.h header file
Signed-off-by: Igor Bezukh
---
drivers/staging/vt6655/80211hdr.h |7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/staging/vt6655/80211hdr.h
b/drivers/staging/vt6655/80211hdr.h
index ba155cd..e05d135 100644
--- a/drivers/
On 07/22/2014 06:32 PM, Arnd Bergmann wrote:
> On Sunday 20 July 2014 17:45:40 Chen Gang wrote:
>>>
>>> Next, I shall:
>>>
>>> - Remove HAS_IOMEM and NO_IOMEM from kernel, firstly.
>>>
>>> - Try to make dummy IOMEM functions for score architecture.
>>>
>>> - Continue discussing with UML for it
On Sunday 20 July 2014 17:45:40 Chen Gang wrote:
> >
> > Next, I shall:
> >
> > - Remove HAS_IOMEM and NO_IOMEM from kernel, firstly.
> >
> > - Try to make dummy IOMEM functions for score architecture.
> >
> > - Continue discussing with UML for it.
> >
>
> Oh, sorry, I forgot, after remove
On 18/07/14 16:33, Masanari Iida wrote:
This patch add missing CR in messages within vme_pio2_core.c
Signed-off-by: Masanari Iida
Acked-by: Martyn Welch
---
drivers/staging/vme/devices/vme_pio2_core.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/dr
On 18/07/14 16:33, Masanari Iida wrote:
This patch fix spelling typo in printk message.
Signed-off-by: Masanari Iida
Acked-by: Martyn Welch
---
drivers/staging/vme/devices/vme_pio2_gpio.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/vme/device
On 18/07/14 16:33, Masanari Iida wrote:
This patch adds missing CR in dev_err message.
Signed-off-by: Masanari Iida
Acked-by: Martyn Welch
---
drivers/staging/vme/devices/vme_pio2_gpio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vme/devices/vm
On 2014-07-21 19:48, H Hartley Sweeten wrote:
This parameter is never used by any of the comedi drivers that provide a
(*buf_change) callback. If the 'new_size' is needed in the callback it can
be found from the 's->async->prealloc_bufsz' as done in the ni_pcidio driver.
Remove the unused parame
On 2014-07-21 19:01, H Hartley Sweeten wrote:
The subdevice lock/unlock callbacks have been disbled ever since the code
was merged. It's also broken due to the core member names in the struct
not matching the calls in the core. Just remove it.
H Hartley Sweeten (2):
staging: comedi: comedi_fo
On Tue, Jul 22, 2014 at 12:26:42PM +0530, Kiran Padwal wrote:
> This patch fixes the following checkpatch.pl warnings:
> WARNING: "Prefer ether_addr_copy() over memcpy() if the Ethernet addresses
> are __aligned(2)".
Is that true here?
Have you tested this on the hardware to ensure it works prop
Checkpatch fix - Add missing blank line after variable declaration
Signed-off-by: Kiran Padwal
---
drivers/staging/gdm724x/gdm_usb.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/gdm724x/gdm_usb.c
b/drivers/staging/gdm724x/gdm_usb.c
index ea89d53..0c1b2de 100644
--- a/d
This patch fixes the following checkpatch.pl warnings:
WARNING: "Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are
__aligned(2)".
Signed-off-by: Kiran Padwal
---
drivers/staging/gdm724x/gdm_lte.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a
62 matches
Mail list logo