- 12 Feb, 2017 40 commits
-
-
simran singhal authored
Checkpatch recommended changes. Signed-off-by:
simran singhal <singhalsimran0@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
simran singhal authored
This patch modifies the assignments into single assignments. Signed-off-by:
-
simran singhal authored
This patch fixes the checkpatch warning by removing unnecessary space after a cast. CHECK: No space is necessary after a cast Signed-off-by:
-
simran singhal authored
This patch fixes the checkpatch warning by removing multiple blank lines. CHECK: Please don't use multiple blank lines Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments. Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
-
Larry Finger authored
Sparse reports the following: CHECK drivers/staging/rtl8712/rtl871x_mlme.c drivers/staging/rtl8712/rtl871x_mlme.c:1653:46: warning: incorrect type in assignment (different base types) drivers/staging/rtl8712/rtl871x_mlme.c:1653:46: expected unsigned int [unsigned] [usertype] DSConfig drivers/staging/rtl8712/rtl871x_mlme.c:1653:46: got restricted __le32 [usertype] <noident> drivers/staging/rtl8712/rtl871x_mlme.c:1656:56: warning: incorrect type in assignment (different base types) drivers/staging/rtl8712/rtl871x_mlme.c:1656:56: expected unsigned int [unsigned] [usertype] ATIMWindow drivers/staging/rtl8712/rtl871x_mlme.c:1656:56: got restricted __le32 [usertype] <noident> drivers/staging/rtl8712/rtl871x_mlme.c:1712:35: warning: incorrect type in assignment (different base types) drivers/staging/rtl8712/rtl871x_mlme.c:1712:35: expected restricted __le16 [addressable] [usertype] cap_info drivers/staging/rtl8712/rtl871x_mlme.c:1712:35: got int Signed-off-by:
Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by:
-
Larry Finger authored
Sparse reports the following: CHECK drivers/staging/rtl8712/rtl871x_ioctl_linux.c drivers/staging/rtl8712/rtl871x_ioctl_linux.c:1422:46: warning: restricted __le16 degrades to integer drivers/staging/rtl8712/rtl871x_ioctl_linux.c:1424:46: warning: restricted __le16 degrades to integer Signed-off-by:
-
Larry Finger authored
Sparse reports the following: CHECK drivers/staging/rtl8712/rtl871x_recv.c drivers/staging/rtl8712/rtl871x_recv.c:657:21: warning: incorrect type in assignment (different base types) drivers/staging/rtl8712/rtl871x_recv.c:657:21: expected unsigned short [unsigned] [assigned] [usertype] len drivers/staging/rtl8712/rtl871x_recv.c:657:21: got restricted __be16 [usertype] <noident> Signed-off-by:
-
Larry Finger authored
Sparse reports the following: CHECK drivers/staging/rtl8712/rtl871x_xmit.c drivers/staging/rtl8712/rtl871x_xmit.c:350:44: warning: restricted __le32 degrades to integer drivers/staging/rtl8712/rtl871x_xmit.c:491:23: warning: incorrect type in initializer (different base types) drivers/staging/rtl8712/rtl871x_xmit.c:491:23: expected unsigned short [usertype] *fctrl drivers/staging/rtl8712/rtl871x_xmit.c:491:23: got restricted __le16 *<noident> drivers/staging/rtl8712/rtl871x_xmit.c:580:36: warning: incorrect type in assignment (different base types) drivers/staging/rtl8712/rtl871x_xmit.c:580:36: expected unsigned short [unsigned] [short] [usertype] <noident> drivers/staging/rtl8712/rtl871x_xmit.c:580:36: got restricted __be16 [usertype] <noident> Signed-off-by:
-
Larry Finger authored
Although the driver works on big-endian hardware, Sparse generates a lot of warnings. Many of these are the result of incorrect coding of these macros. Signed-off-by:
-
Larry Finger authored
The headers describing a number of network packets do not have the correct endian settings for several types of data. Signed-off-by:
-
Larry Finger authored
Sparse reports the following: CHECK drivers/staging/rtl8712/rtl8712_xmit.c drivers/staging/rtl8712/rtl8712_xmit.c:564:42: warning: cast from restricted __le32 drivers/staging/rtl8712/rtl8712_xmit.c:569:42: warning: cast from restricted __le32 drivers/staging/rtl8712/rtl8712_xmit.c:571:42: warning: cast from restricted __le32 Each of these cases is transferring a quantity that is little-endian. There is no need for conversion. Signed-off-by:
-
Jess Frazelle authored
Marked msi_domain_ops structs as __ro_after_init when called only during init. This protects the data structure from accidental corruption. Suggested-by:
Kees Cook <keescook@chromium.org> Signed-off-by:
Jess Frazelle <me@jessfraz.com> Signed-off-by:
-
Shiva Kerdel authored
Removing unnecessary blank lines around braces to solve CHECKS. Signed-off-by:
Shiva Kerdel <shiva@exdev.nl> Signed-off-by:
-
Youngdo, Lee authored
Replaced sizeof(struct foo) into sizeof(*ptr), found by checkpatch.pl. Signed-off-by:
Youngdo, Lee <oss.ydlee@gmail.com> Signed-off-by:
-
Gioh Kim authored
Fix "Octal permissions are preffered than symbolic ones" issues. Signed-off-by:
Gioh Kim <gi-oh.kim@profitbricks.com> Signed-off-by:
-
Gioh Kim authored
This patch fixes only obvious lines. There are still more issues. Signed-off-by:
-
Nathan Howard authored
Fix checkpatch.pl issue of the form: "CHECK: Please don't use multiple blank lines". Signed-off-by:
Nathan Howard <adanhawthorn@gmail.com> Signed-off-by:
-
Nathan Howard authored
Fix checkpatch.pl issue of the form: "CHECK: Alignment should match open parenthesis". Signed-off-by:
-
Derek Robson authored
Change permissions to octal style. Found using checkpatch Signed-off-by:
-
Alexander Alemayhu authored
Fixes the following sparse output: drivers/staging/wlan-ng/p80211conv.c:132:25: warning: cast to restricted __be16 drivers/staging/wlan-ng/p80211conv.c:132:25: warning: cast to restricted __be16 drivers/staging/wlan-ng/p80211conv.c:132:25: warning: cast to restricted __be16 drivers/staging/wlan-ng/p80211conv.c:132:25: warning: cast to restricted __be16 drivers/staging/wlan-ng/p80211conv.c:154:38: warning: incorrect type in assignment (different base types) drivers/staging/wlan-ng/p80211conv.c:154:38: expected unsigned short [unsigned] [usertype] type drivers/staging/wlan-ng/p80211conv.c:154:38: got restricted __be16 [usertype] <noident> drivers/staging/wlan-ng/p80211conv.c:390:42: warning: cast to restricted __le16 drivers/staging/wlan-ng/p80211conv.c:413:29: warning: incorrect type in assignment (different base types) drivers/staging/wlan-ng/p80211conv.c:413:29: expected unsigned short [unsigned] [usertype] type drivers/staging/wlan-ng/p80211conv.c:413:29: got restricted __be16 [usertype] <noident> drivers/staging/wlan-ng/p80211conv.c:481:29: warning: incorrect type in assignment (different base types) drivers/staging/wlan-ng/p80211conv.c:481:29: expected unsigned short [unsigned] [usertype] type drivers/staging/wlan-ng/p80211conv.c:481:29: got restricted __be16 [usertype] <noident> Signed-off-by:
Alexander Alemayhu <alexander@alemayhu.com> Signed-off-by:
-
Jacob Zachariah authored
Fix a checkpatch.pl WARNING: line over 80 characters Signed-off-by:
Jacob Zachariah <jacob_z@yahoo.com> Signed-off-by:
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: usleep_range is preferred over udelay Signed-off-by:
Saber Rezvani <irsaber@gmail.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: usleep_range is preferred over udelay Signed-off-by:
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: usleep_range is preferred over udelay Signed-off-by:
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: usleep_range is preferred over udelay Signed-off-by:
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: usleep_range is preferred over udelay Signed-off-by:
-
AbdAllah-MEZITI authored
Trailing statements should be on next line. Signed-off-by:
AbdAllah-MEZITI <abdallah.meziti.pro@gmail.com> Signed-off-by:
-
William Blough authored
The local socket address is defined as a __be32 value. However, if the local ip address is not given, then the constant INADDR_ANY is used. This patch converts INADDR_ANY to __be32 for type consistency. Fixes the following sparse warnings: drivers/staging/lustre/lnet/lnet/lib-socket.c:396:41: warning: incorrect type in assignment (different base types) drivers/staging/lustre/lnet/lnet/lib-socket.c:397:56: warning: restricted __be32 degrades to integer Signed-off-by:
William Blough <devel@blough.us> Signed-off-by:
-
Niu Yawei authored
ll_migrate() should check reply status before trying to read reply buffer, checking if request is NULL doesn't make sense. Signed-off-by:
Niu Yawei <yawei.niu@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8807 Reviewed-on: https://review.whamcloud.com/23666Reviewed-by:
Andreas Dilger <andreas.dilger@intel.com> Reviewed-by:
Lai Siyao <lai.siyao@intel.com> Reviewed-by:
John L. Hammond <john.hammond@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
-
wang di authored
Do not migrate root inode. Signed-off-by:
wang di <di.wang@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-7577 Reviewed-on: http://review.whamcloud.com/17669Reviewed-by:
-
