- Mar 08, 2014
-
-
Himangi Saraogi authored
This patch moves shared private data kzalloc to managed devm_kzalloc. There are no kfree statements associated with this data, so using devm adds the possibility of the data being freed, if the proble function fails (if that is possible) or if the remove function succeeds. Signed-off-by:
Himangi Saraogi <himangi774@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Ebru Akagunduz authored
Fix checkpatch.pl issues with quoted string split across lines in nvec.c Signed-off-by:
Ebru Akagunduz <ebru.akagunduz@gmail.com> Acked-by:
Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warning in crystalhd_lnx.c WARNING: incorrect type in argument Signed-off-by:
Monam Agarwal <monamagarwal123@gmail.com> Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warning in crystalhd_misc.c WARNING: incorrect type in argument Signed-off-by:
-
Ebru Akagunduz authored
Fix checkpatch.pl issues with "foo * bar" should be "foo *bar" in dgap.h Signed-off-by:
-
Ebru Akagunduz authored
Fix checkpatch.pl issues with open brace '{' following struct go on the same line in dgap.h Signed-off-by: -
Ebru Akagunduz authored
This patch fixes the Sparse Warnings "symbol was not declared. Should it be static?" Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warnings: drivers/staging/wlags49_h2/wl_wext.c:3357:28: warning: incorrect type in assignment (different address spaces) drivers/staging/wlags49_h2/wl_wext.c:3357:28: expected void [noderef] <asn:1>*[addressable] [assigned] pointer drivers/staging/wlags49_h2/wl_wext.c:3357:28: got void *<noident> drivers/staging/wlags49_h2/wl_wext.c:3422:47: warning: incorrect type in assignment (different address spaces) drivers/staging/wlags49_h2/wl_wext.c:3422:47: expected void [noderef] <asn:1>*[addressable] [assigned] pointer drivers/staging/wlags49_h2/wl_wext.c:3422:47: got char *[usertype] <noident> drivers/staging/wlags49_h2/wl_wext.c:3781:21: warning: symbol 'wl_priv_args' was not declared. Should it be static? Signed-off-by:
-
Monam Agarwal authored
This patch fixes the following sparse warnings: drivers/staging/wlags49_h2/dhf.c:109:6: warning: symbol 'signature' was not declared. Should it be static? drivers/staging/wlags49_h2/dhf.c:126:25: warning: symbol 'mfi_sup' was not declared. Should it be static? drivers/staging/wlags49_h2/dhf.c:127:25: warning: symbol 'cfi_sup' was not declared. Should it be static? drivers/staging/wlags49_h2/dhf.c:142:17: warning: symbol 'ltv_info' was not declared. Should it be static? drivers/staging/wlags49_h2/dhf.c:173:1: warning: symbol 'check_comp_fw' was not declared. Should it be static? Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warnings: drivers/staging/wlags49_h2/hcf.c:253:26: warning: symbol 'cfg_drv_identity' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:262:24: warning: symbol 'cfg_drv_sup_range' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:274:31: warning: symbol 'cfg_drv_act_ranges_pri' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:291:31: warning: symbol 'cfg_drv_act_ranges_sta' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:336:31: warning: symbol 'cfg_drv_act_ranges_hsi' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:373:24: warning: symbol 'cfg_drv_act_ranges_apf' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:3103:1: warning: symbol 'calc_mic' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:3419:1: warning: symbol 'check_mic' was not declared. Should it be static? Signed-off-by:
-
Monam Agarwal authored
This patch removes the unused function "percent" from wl_util.c Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the checkpatch warnings: WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: -
Himangi Saraogi authored
This patch fixes the following checkpatch errors: ERROR: space prohibited after that open parenthesis '(' ERROR: space prohibited before that close parenthesis ')' Signed-off-by: -
Himangi Saraogi authored
This patch fixes the following warnings: WARNING: that open brace { should be on the previous line WARNING: suspect code indent for conditional statements (16, 16) Signed-off-by: -
Himangi Saraogi authored
This patch fixes the following issue detected by checkpatch.pl. WARNING: Use #include <linux/uaccess.h> instead of <asm/uaccess.h> Signed-off-by:
-
Himangi Saraogi authored
Coding style requires that comments use the standard /* */ style instead of C99 style. The commented #include statements were also removed as they are not required. Signed-off-by:
-
Monam Agarwal authored
This patch fixes sparse warnings "Using plain integer as NULL pointer" in sta_h2.c Signed-off-by:
-
Monam Agarwal authored
Staging: iio/accel: Fix Prefer netdev_warn(netdev, ...then dev_warn(dev, ... then pr_warn(... to printk(KERN_INFO ... in sca3000_ring.c This patch fixes the following checkpatch.pl warning in sca3000_ring.c WARNING: Prefer netdev_info(netdev, ... then dev_info(dev, ... then pr_info(... to printk(KERN_INFO ... Signed-off-by:
-
Monam Agarwal authored
This patch fixes following checkpatch.pl warning WARNING: Prefer pr_warn(... to pr_warning(... Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are __aligned(2)" in file rtw_cmd.c. Pahole shows that the addresses are 2-byte aligned. Signed-off-by:
Elena Oat <oat.elena@gmail.com> Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "suspect code indent for conditional statements" in file rtw_cmd.c. I have removed the conditional statements in two places altogether, as it didn't bring any value. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "space prohibited before semicolon" in file rtw_cmd.c. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "unnecessary parentheses" in file rtw_cmd.c. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "please, no space before tabs" in file rtw_cmd.c. I have also removed couple of lines of code that was commented and probably left from the previous version. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "please, no spaces at the start of a line" in file rtw_cmd.c. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "braces {} are not necessary for single statement blocks" in file rtw_cmd.c. Signed-off-by: -
Monam Agarwal authored
This patch fixes the following sparse warnings: drivers/staging/rtl8188eu/core/rtw_wlan_util.c:1627:25: warning: cast to restricted __le16 by changing the annotations in the struct declaration. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning produced by checkpatch of "printk should include KERN_ facility level". printk was replaced by netdev_info. While at it, I have also fixed the line over 80 characters warning. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning produced by checkpatch "Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are __aligned(2)". The address that is copied is a field of nat25_network_db_entry structure. Definition for this struct is found in rtw_br_ext.h file. Pahole shows that the addresses are 2-byte aligned. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "braces {} are not necessary for any arm of this statement" in file rtw_ioctl_set.c. The braces are removed in three places. Signed-off-by: -
Ebru Akagunduz authored
Fix checkpatch.pl issues with kfree(NULL) is safe this check is probably not required in rtw_ap.c Signed-off-by:
-
Elena Oat authored
This patch fixes the warning produced by checkpatch in the file rtw_cmd.c "kfree(NULL) is safe this check is probably not required". The patch fixes it in two places for this file. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "Please, no space before tab" produced by checkpatch.pl. The modified file: rtw_cmd.c. Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "space prohibited between function name and open parenthesis '('". The modified file: rtw_cmd.c. The patch also removes some strange spaces that were generated probably by some editor. Signed-off-by: -
git://ftp.arm.linux.org.uk/~rmk/linux-armGreg Kroah-Hartman authored
Russell writes: This set of changes reorganises imx-drm's DT bindings by re-using the OF graph parsing code which was located in drivers/media, removing the temporary bindings. The result is that more TODO entries are now removed. While we're not quite done with this yet as there's a few straggling updates to imx-ldb to come, but leaving these out is not detrimental at this point in time - they are more an enhancement. However, this pull has the additional complication that we're sharing seven commits with Mauro's V4L git tree, which move the OF graph parsing code out of drivers/media into drivers/of. Philipp's imx-drm changes depend on these and my previously committed round of imx-drm commits. Hence, the diffstat below is from a test merge with your tree head (17b02809). Mauro merged those seven commits earlier today as a git pull, so both trees will be sharing exactly the same commit IDs. I've given these changes a spin here on both my Hummingboard and Cubox-i4 (one is iMX6Solo, the other is iMX6Quad based), which includes Xorg using the DRM device directly, and I find nothing wrong. The diffstat does look a little scarey - this is because we're having to update the ARM DT files along with this change, and obviously the dependency on the OF graph parsing code.
-
bahar sahin authored
This patch removes unnecessary braces to fix a checkpatch.pl warning. Signed-off-by:
bahar sahin <baharxf@gmail.com> Reviewed-by:
Sarah Sharp <sarah.a.sharp@linux.intel.com> Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the checkpatch.pl warning : Prefer pr_warn(... to pr_warning(... Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the checkpatch warning : space prohibited after that open parenthesis '('. Signed-off-by: -
Ebru Akagunduz authored
Fix checkpatch.pl issues with no space before tabs in p80211mgmt.h Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the following sparse warnings: drivers/staging/wlan-ng/hfa384x_usb.c:3582:14: warning: cast to restricted __le16 drivers/staging/wlan-ng/hfa384x_usb.c:3584:19: warning: cast to restricted __le16 by changing the annotations in the struct declaration. Signed-off-by:
-
