- 14 Sep, 2014 3 commits
-
-
Artemiy Volkov authored
This patch fixes the following sparse warnings: drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb.c:284:51: warning: cast removes address space of expression drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb.c:284:39: warning: incorrect type in assignment (different address spaces) drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb.c:284:39: expected void [noderef] <asn:1>*iov_base drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb.c:284:39: got void *<noident> by simplifying an expression containing non-__user-attributed type names. Signed-off-by:
Artemiy Volkov <artemiyv@acm.org> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Greg Donald authored
Cleanup obdo_cmp_md() src/dst comparisons using bitwise OR assignments Suggested-by:
Joe Perches <joe@perches.com> Signed-off-by:
Greg Donald <gdonald@gmail.com> Signed-off-by:
-
Nick Hudson authored
I'm seeing problems with a d-link dwcl-g122 wifi dongle that someone sent me. There are reports of other wifi dongles with the same/similar problem. The devices appear to be NAKing to the point of confusing the dwc2 driver completely. The attached patch helps with my d-link dwl-g122 - it's adapted from the Raspberry Pi dwc_otg driver, which is a modified version of the Synopsys vendor driver. The error recovery is still valid after the patch, I think. Cc: Dom Cobley <popcornmix@gmail.com> Signed-off-by:
Nick Hudson <skrll@netbsd.org> Signed-off-by:
Paul Zimmerman <paulz@synopsys.com> Signed-off-by:
-
- 11 Sep, 2014 37 commits
-
-
Tobias Klauser authored
ethtool (and SIOCETHTOOL in particular) is part of Linux since the pre-git era, it thus makes no sense no sense to make the include of linux/ethtool.h conditional. Also remove the unused define DEVICE_ETHTOOL_IOCTL_SUPPORT. Signed-off-by:
Tobias Klauser <tklauser@distanz.ch> Signed-off-by:
-
Tobias Klauser authored
Instead of using an own copy of struct net_device_stats in struct vnt_private, use stats from struct net_device. Also remove the thus unnecessary device_get_stats(), as it would now just return netdev->stats, which is the default in dev_get_stats(). Signed-off-by:
-
Tobias Klauser authored
The pci_state member of struct vnt_private is used nowhere in the code, so remove it. Supposedly it was used to save the PCI configuration space which is now done using pci_save_state(). Signed-off-by:
-
Tobenna P. Igwe authored
This patch fixes the following sparse warnings: CHECK drivers/staging/rtl8188eu/hal/fw.c drivers/staging/rtl8188eu/hal/fw.c:219:13: warning: cast to restricted __le16 drivers/staging/rtl8188eu/hal/fw.c:219:13: warning: cast to restricted __le16 drivers/staging/rtl8188eu/hal/fw.c:219:13: warning: cast to restricted __le16 drivers/staging/rtl8188eu/hal/fw.c:219:13: warning: cast to restricted __le16 by changing the signature from 'u16' to '__le16'. Signed-off-by:
Tobenna P. Igwe <ptigwe@gmail.com> Signed-off-by:
-
Vincenzo Scotti authored
Signed-off-by:
Vincenzo Scotti <vinc94@gmail.com> Signed-off-by:
-
Vincenzo Scotti authored
Signed-off-by:
-
Vincenzo Scotti authored
Signed-off-by:
-
Domagoj Trsan authored
Signed-off-by:
Domagoj Trsan <domagoj.trsan@gmail.com> Signed-off-by:
-
Domagoj Trsan authored
Signed-off-by:
-
Domagoj Trsan authored
Signed-off-by:
-
Domagoj Trsan authored
Signed-off-by:
-
Sudip Mukherjee authored
fixed sparse warning : context imbalance in 'resume_device' unexpected unlock this patch will generate warning from checkpatch for lines over 80 character , but since those are user-visible strings so it was not modified. Signed-off-by:Sudip Mukherjee <sudip@vectorindia.org> Acked-by:
Benjamin Romer <benjamin.romer@unisys.com> Tested-by:
-
Nitin Kuppelur authored
This is a patch to the octeon-hcd.c file that fixes removes dummy label i.e. label followed by return of void function Signed-off-by:
Nitin Kuppelur <nitinkuppelur@gmail.com> Signed-off-by:
-
Sanjeev Sharma authored
This is a patch to the r8190_rtl8256.c file that fixes commenting style Error Signed-off-by:
Sanjeev Sharma <Sanjeev_Sharma@mentor.com> Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl 'code indent should use tabs where possible' errors Signed-off-by:
-
John L. Hammond authored
Move the definition of struct lvfs_run_ctxt to the one file that needed it (lustre/include/obd.h). Remove the then unneeded headers lustre/include/lvfs.h and lustre/include/linux/lvfs.h. Signed-off-by:
John L. Hammond <john.hammond@intel.com> Signed-off-by:
-
John L. Hammond authored
The upcall cache is unused so remove include/linux/libcfs/lucache.h and lustre/libcfs/upcall_cache.c. Signed-off-by:
-
John L. Hammond authored
Remove the unused "lvfs" functions obd_lvfs_fid2dentry(), ll_lookup_one_len(), l_dput(), lustre_rename(), push_ctxt(), and pop_ctxt(). Remove the unused members of struct lvfs_run_ctxt. Signed-off-by:
-
John L. Hammond authored
The definitions in lustre/include/linux/lvfs_linux.h are not used so remove that header. Signed-off-by:
-
James Simmons authored
Remove include/linux/libcfs/linux/portals_compat25.h. . remove some unused/unnecessary macros such as smp_num_cpus / SIGNAL_MASK_ASSERT etc. . replace some macros with direct kernel API calls such as RECALC_SIGPENDING/CLEAR_SIGPENDING/CURRENT_SECONDS, cfs_wait_event_interruptible/_exclusive etc. Signed-off-by:
Liu Xuezhao <xuezhao.liu@emc.com> Signed-off-by:
Peng Tao <tao.peng@emc.com> Signed-off-by:
James Simmons <uja.ornl@gmail.com> Signed-off-by:
Andreas Dilger <andreas.dilger@intel.com> Reviewed-by:
Keith Mannthey <keith.mannthey@intel.com> Signed-off-by:
-
John L. Hammond authored
Expand the two uses of ll_inode_blksize() and remove the then unnecessary header lustre/include/linux/lustre_common.h. Signed-off-by:
Bob Glossman <bob.glossman@intel.com> Reviewed-by:
Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
-
James Simmons authored
The lvfs layer is obsolete and unused by Lustre clients. So remove it and integrate the last of the procfs code into the obdclass layer. Signed-off-by:
Mikhail Pershin <mike.pershin@intel.com> Signed-off-by:
Jian Yu <jian.yu@intel.com> Reviewed-by:
-
Hugues Morisset authored
Signed-off-by:
Hugues Morisset <morisset.hugues@gmail.com> Signed-off-by:
-
Hugues Morisset authored
Signed-off-by:
-
Hugues Morisset authored
Signed-off-by:
-
Julia Lawall authored
The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Oleg Drokin authored
This function is no longer used anywhere. Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Signed-off-by:
-
Mikhail Pershin authored
OUT service is used to server both MDS-MDS updates and MDS-OST therefore services on MDT and OST are set to use the same request and reply portals to be fully unified and able to serve any type of requests. Signed-off-by:
wangdi <di.wang@intel.com> Reviewed-by:
Fan Yong <fan.yong@intel.com> Reviewed-by:
-
Greg Donald authored
Fix checkpatch.pl 'code indent should use tabs where possible' errors Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl "space prohibited after that open parenthesis '('" errors Signed-off-by: -
Ian Abbott authored
The `comedi_board(dev)` inline function calls just return `dev->board_ptr`. Expand the inline function calls. Signed-off-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Ian Abbott authored
The `comedi_board(dev)` inline function calls just return `dev->board_ptr`. Expand the inline function calls. Signed-off-by:
-
Chase Southwood authored
Returns at the end of void functions are useless. Remove this one. Signed-off-by:
Chase Southwood <chase.southwood@gmail.com> Reviewed-by:
-
Chase Southwood authored
The statement nested in an else after a return may be brought out one indent level, the else is useless. Signed-off-by:
-
Chase Southwood authored
Signed-off-by:
-
Chase Southwood authored
This patch fixes improper comment indentation, removes dead code and obsolete comments, and conforms remaining comments to a consistent kernel commenting style. Signed-off-by:
-
H Hartley Sweeten authored
This driver currently allows a cmd->stop_src == TRIG_COUNT with a cmd->stop_arg of 0 for both the analog input and output async commands. The (*do_cmd) for both subdevices sets up and starts the command without handling the "empty acquisition". This results in the interrupt functions trying to transfer 0 data samples. Validate that the cmd->stop_arg is >= 1 in the (*do_cmdtest) to prevent the "empty acquisition". Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
-
