- 31 May, 2015 40 commits
-
-
Malcolm Priestley authored
pDevice -> priv Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Malcolm Priestley authored
Page 1 is fully proctected by lock there is no need to check for it. Page 0 is selected at other times. Remove byOrgPageSel and its calls from function. Signed-off-by:
-
Malcolm Priestley authored
Introduce vnt_interrupt to handle interrupt and use workqueue to queue and queue on vif. Convert device_intr to void call vnt_interrupt_process from vnt_interrupt_work providing vif is valid. This removes troublesome heavy code from the interupt handler and allows to remove atomic from other areas of driver. Signed-off-by:
-
Malcolm Priestley authored
Remove from makefile and dead variables Signed-off-by:
-
Malcolm Priestley authored
This function does not provide any data to users. Signed-off-by:
-
Malcolm Priestley authored
This function is nolonger of any future use. Signed-off-by:
-
Malcolm Priestley authored
Collect low level stats from mib counter for mac80211 call. Replacing the unused function STAvUpdate802_11Counter. Signed-off-by:
-
Niranjan Dighe authored
Replace kzalloc followed by memcpy by kmemdup. This patch was generated by 'make coccicheck' Signed-off-by:
Niranjan Dighe <ndighe@visteon.com> Signed-off-by:
-
Greg Kroah-Hartman authored
lustre_ext_acl_xattr_reduce_space() declares old_size, but never uses it and gcc 5.x likes to complain about this, so remove it as it's not used at all. Cc: Oleg Drokin <oleg.drokin@intel.com> Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by: -
Dmitry Eremin authored
Signed-off-by:
Dmitry Eremin <dmiter4ever@gmail.com> Signed-off-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
-
Oleg Drokin authored
This nid_stats tracking only makes sense on the server side, on the client there are no other clients to keep track of anyway. Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Signed-off-by:
-
Oleg Drokin authored
This code only made sense on servers. Signed-off-by:
-
Oleg Drokin authored
Signed-off-by:
-
Dmitry Eremin authored
Signed-off-by:
-
Oleg Drokin authored
This moves all remaining procfs handling in llite layer to debugfs. Signed-off-by:
Dmitry Eremin <dmitry.eremin@intel.com> Signed-off-by:
-
Nicholas Mc Guire authored
API consolidation with coccinelle found: ./drivers/staging/panel/panel.c:782:2-18: consolidation with schedule_timeout_*() recommended This is a 1:1 conversion with respect to schedule_timeout() to the schedule_timeout_interruptible() helper only - so only an API consolidation to improve readability. The timeout was being passed as (ms * HZ + 999) / 1000 but that simply looks wrong - rather than "manual" converting to jiffies, msecs_to_jiffies which handles all corner-cases correctly, was used. Patch was compile tested with x86_64_defconfig + CONFIG_STAGING=y, CONFIG_PARPORT=m, CONFIG_PANEL=m Signed-off-by:Nicholas Mc Guire <hofrat@osadl.org> Acked-by:
Willy Tarreau <w@1wt.eu> Signed-off-by:
-
Nicholas Mc Guire authored
API consolidation with coccinelle found: ./drivers/staging/comedi/drivers/me_daq.c:177:1-17: consolidation with schedule_timeout_*() recommended This is a 1:1 conversion of the current calls to an available helper only - so only an API consolidation to improve readability. Signed-off-by:Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Masanari Iida authored
This patch fix a spelling typo in printk within sm750.c Signed-off-by:
Masanari Iida <standby24x7@gmail.com> Signed-off-by:
-
Michel von Czettritz authored
As suggested by Sudip this patch removes the redundant 1360x768@60 option from the fb_videomode struct array. The removed option and the option one it differ in a typo and a flag. Signed-off-by:
Michel von Czettritz <michel.von.czettritz@gmail.com> Signed-off-by:
-
Isaac Assegai authored
Replace spaces at the start of lines with tabs to rectify the following warning: WARNING: please, no spaces at the start of a line Signed-off-by:
Isaac Assegai <isaac.a.travers@gmail.com> Signed-off-by:
-
Jakub Sitnicki authored
Remove #ifdef's enclosed by an #ifdef test for the same macro to improve readability. No code changes: md5, CONFIG_88EU_AP_MODE=y: b819a33f65133607ebc33b8999ee3a79 r8188eu.o.before b819a33f65133607ebc33b8999ee3a79 r8188eu.o.after md5, CONFIG_88EU_AP_MODE=n: 94c84035d59285408b866a57b442276d r8188eu.o.before 94c84035d59285408b866a57b442276d r8188eu.o.after Signed-off-by:Jakub Sitnicki <jsitnicki@gmail.com> Signed-off-by:
-
Luca Ceresoli authored
"If the name of a function is an action or an imperative command, the function should return an error-code integer." (Documentation/CodingStyle) Several action-like functions in this driver return a boolean: _SUCCESS = 1 on success, _FAIL = 0 on error, defined in drivers/staging/rtl8188eu/include/osdep_service.h. Change rtw_start_drv_threads() to return a proper 0-or-error value. Signed-off-by:
Luca Ceresoli <luca@lucaceresoli.net> Cc: Greg Kroah-Hartman <gregkh@linux.com> Cc: Larry Finger <Larry.Finger@lwfinger.net> Cc: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Luca Ceresoli authored
Also remove them from .h files. Signed-off-by:
-
Luca Ceresoli authored
The loadparam() function cannot fail, it's called only once and its return value is ignored there. Signed-off-by:
-
Luca Ceresoli authored
Fixes checkpatch warnings: WARNING: Missing a blank line after declarations Signed-off-by:
-
Luca Ceresoli authored
Even though these are not reported by checkpatch, they are coding style issues. Signed-off-by:
-
Shailendra Verma authored
Fix spelling error in comment in function tegra_nvec_remove. Signed-off-by:
Shailendra Verma <shailendra.capricorn@gmail.com> Acked-by:
Marc Dietrich <marvin24@gmx.de> Signed-off-by:
-
Luca Ceresoli authored
The loadparam() function cannot fail, it's called only once and its return value is ignored there. Signed-off-by:
-
Luca Ceresoli authored
Signed-off-by:
-
Luca Ceresoli authored
"step 2" does mean much as there is no "step 1" stated anywhere... Signed-off-by:
-
Haggai Eran authored
With an RTL8191SU USB adaptor, sometimes the hints for a fragmented packet are set, but the packet length is too large. Allocate enough space to prevent memory corruption and a resulting kernel panic [1]. [1] http://www.spinics.net/lists/linux-wireless/msg136546.html Cc: <stable@vger.kernel.org> Signed-off-by:
Haggai Eran <haggai.eran@gmail.com> ACKed-by:
Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by:
-
Sudip Mukherjee authored
del_timer_sync() is not to be called in the interrupt context unless the timer is irqsafe. but most of the functions where commits 6501c8e7 and 382d020f touched were called in interrupt context. And as a result the WARN_ON was getting triggered. Changed to del_timer() in places which were called from interrupt. Fixes: 382d020f ("Staging: rtl8712: Eliminate use of _cancel_timer" Fixes: 6501c8e7 ("Staging: rtl8712: Eliminate use of _cancel_timer_ex") Fixes: https://bugzilla.kernel.org/show_bug.cgi?id=97711Reported-by:
Arek Rusniak <arek.rusi@gmail.com> Tested-by:
Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by:
-
Vladimirs Ambrosovs authored
This patch fixes the leak, which was present in fwserial driver in the init function. In case the tty driver allocation failed the function returned error, leaving debugfs entry in the filesystem. To fix the issue additional error label was added, so that the code will jump to it in case of allocation failure, and free debugfs entries. Signed-off-by:
Vladimirs Ambrosovs <rodriguez.twister@gmail.com> Reviewed-by:
Peter Hurley <peter@hurleysoftware.com> Signed-off-by:
-
Jagan Teki authored
Fixes Warning encounter this by applying checkpatch.pl against this file: Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are __aligned(2) pahole output for respective structures: - addr->sa_data struct sockaddr { sa_family_t sa_family; /* 0 2 */ char sa_data[14]; /* 2 14 */ /* size: 16, cachelines: 1, members: 2 */ /* last cacheline: 16 bytes */ }; - pnetdev->dev_addr dev_addr is interface address infor from generic net_device structure which is properly aligned and have some patches with this change as well. "staging: rtl8712: fix Prefer ether_addr_copy() over memcpy()" (sha1: 36e4d882) Signed-off-by:Jagan Teki <jteki@openedev.com> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: Larry Finger <Larry.Finger@lwfinger.net> Cc: Florian Schilhabel <florian.c.schilhabel@googlemail.com> Signed-off-by:
-
Jagan Teki authored
This patch fixes line over 80 characters warninings while running checkpatch.pl - "WARNING: line over 80 characters" Signed-off-by:
-
Sai.Jiang authored
Remove two warnings of missing-blank-line-after-declaration. Signed-off-by:
Sai.Jiang <sai.d.jiang@foxmail.com> Signed-off-by:
-
Dan Carpenter authored
We can't save two different values in "flags" so it means that IRQs are not enabled properly at the end of this function. This isn't a problem in the current code because it's always called with IRQs disabled so we don't want to enable them at the end. This bug is old but it's thanks to David Matlack's recent cleanups that Smatch can detect it. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by:
David Matlack <dmatlack@google.com> Signed-off-by:
-
Masanari Iida authored
This patch fix 2 spelling typos in printk within gdm72xx. Signed-off-by:
-
Jaime Arrocha authored
gdm_endian.c: small changes were done to remove testing for host endianness and in-driver conversion for byte-ordering. The linux/kernel.h functions are used now. gdm_endian.h: removal of code no longer needed with changes in gdm_endian.c. Signed-off-by:
Jaime Arrocha <jarr@kerneldev.net> Signed-off-by:
-
Nicholas Mc Guire authored
schedule_timeout takes a timeout in jiffies but the code currently is passing in a constant POLLING_INTERVAL which makes this timeout HZ dependent, so pass it through msecs_to_jiffies() to fix this up. patch was compile tested for x86_64_defconfig + CONFIG_STAGING=y, CONFIG_RTS5208=m Patch is against 4.0-rc5 (localversion-next is -next-20150527) Signed-off-by:
-
