aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2011-05-18staging/keucr: fix transport other code styleCho, Yu-Chen1-2/+6
fix keucr transport.c other coding style but not from checkpatch.pl. replace ternary conditional "?:" with if/else Signed-off-by: Cho, Yu-Chen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-18staging/keucr: fix keucr transport coding styleCho, Yu-Chen2-208/+258
fix keucr transport.c transport.h coding style Signed-off-by: Cho, Yu-Chen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-18staging/keucr: fix keucr smil.h coding styleCho, Yu-Chen1-141/+150
fix keucr smil.h coding style Signed-off-by: Cho, Yu-Chen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-18staging/keucr: fix keucr smilecc.c coding styleCho, Yu-Chen1-144/+149
fix keucr smilecc.c coding style Signed-off-by: Cho, Yu-Chen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-18staging/keucr: fix keucr scsiglue coding styleCho, Yu-Chen1-64/+93
fix keucr scsiglue.c coding style Signed-off-by: Cho, Yu-Chen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-18staging/keucr: fix keucr msscsi coding styleCho, Yu-Chen1-102/+117
Fix keucr msscsi.c coding style. Remove externs ,and move MS_SCSIIrp to end, because there are not necessary to add extern for MS_SCSIIrp function. Signed-off-by: Cho, Yu-Chen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-18staging/keucr: fix keucr ms coding styleCho, Yu-Chen2-465/+566
fix keucr ms.c and ms.h coding style Signed-off-by: Cho, Yu-Chen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: vmbus_drv: Get rid of util synchronixation based on channel ↵K. Y. Srinivasan3-20/+0
counting Now that we have dealt with this issue differently, get rid of the old mechanism. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: Re-implement the synchronization for util channelsK. Y. Srinivasan2-26/+37
The util module expects that the util channels are fully initialized when the module loads. To deal with the race condition which can result in a NULL pointer dereferencing if the util module were to load before all the util channels are fully initialized, in commit: commit: 8b5d6d3bd3e34e4cc67d875c8c88007c1c9aa96 Author: Haiyang Zhang <[email protected]> Date: Fri May 28 23:22:44 2010 +000 code was introduced in the vmbus driver to ensure that all the util channels were fully initialized before returning from the load of the vmbus driver. This solution has several problems: if for whatever reason, any util channel were to fail to initialize, vmbus driver would wait indefinitely. We deal with this synchronization issue very differently in this patch. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: vmbus_drv: Introduce state in struct vmbus_channel to track ↵K. Y. Srinivasan2-0/+7
util services In preparation for getting rid of util channel synchronization based on counting util channels, introduce state in struct vmbus_channel to track util services. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: comedi: addi-data: Get rid of redundant ps_BoardInfoIan Abbott4-15/+10
The ps_BoardInfo pointer in the device private data is redundant as we can just use the this_board macro to access the same data, as is done elsewhere in the code. Get rid of the pointer and change the code to use the this_board macro instead. Signed-off-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: comedi: addi-data: Don't overwrite read-only dataIan Abbott5-52/+102
The drivers for ADDI-DATA cards can override some static parameters for the board type using information read from EEPROM. Unfortunately, they currently write the parameters from the EEPROM back to the shared, read-only board data! The problem has been masked during compilation by type-casting away the const-ness of the data. This patch changes the code to use an area in the private data for the board instance to hold the parameters read from EEPROM (after initializing the parameters from the static board data). It also changes the type-casts to the read-only data to preserve the const qualifier. Signed-off-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: comedi: adv_pci_dio: Add a counter subdevice to PCI-1751Ian Abbott1-1/+2
The Advantech PCI-1751 has a 8254 counter chip on board. Add it to the device as a counter subdevice. Apparently the counter can generate interrupts although the driver does not currently use this capability. Original patch by Ivan Russkih (Иван Русских) <vanekrus at gmail dot com>. Signed-off-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17drivers/staging/line6/driver.c: Drop unneeded put functions.Julia Lawall1-2/+0
This seems to be the result of patches ab366c1a and 1027f476 crossing each other. Patch ab366c1a adds calls to usb_put_intf and usb_put_dev at the end of the function line6_probe, in the error handling code, while patch 1027f476 moves the calls to the corresponding get function from the beginning to the end of line6_probe, making the calls to put in the error handling code unnecessary. // </smpl> Signed-off-by: Julia Lawall <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17ath6kl: iw dev wlan0 link implementationNaveen Singh1-0/+146
implementing the cfg ops that gets called when iw dev wlan0 link is issued by user. The ops that needs to be implemented is get_station. kvalo: check the mac address, remove signal_pending(), use ARRAY_SIZE() and fix style issues Signed-off-by: Naveen Singh <[email protected]> Signed-off-by: Kalle Valo <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17drivers/staging/ath6kl/os/linux/cfg80211.c: Add missing call to cfg80211_put_bssJulia Lawall1-0/+1
A call to cfg80211_get_bss hould be accompanied by a call to cfg80211_put_bss in error-handling code. A simplified version of the semantic match that finds this problem is: (http://coccinelle.lip6.fr/) // <smpl> @r exists@ local idexpression struct cfg80211_bss * x; expression ra,rr; position p1,p2; @@ x = cfg80211_get_bss@p1(...) ... when != x = rr when != cfg80211_put_bss(x,...) when != if (...) { ... cfg80211_put_bss(x,...) ...} if(...) { ... when != x = ra when forall when != cfg80211_put_bss(x,...) \(return <+...x...+>; \| return@p2...; \) } @script:python@ p1 << r.p1; p2 << r.p2; @@ cocci.print_main("cfg80211_get_bss",p1) cocci.print_secs("return",p2) // </smpl> Signed-off-by: Julia Lawall <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: brcm80211: use correct pointer types in bcm_pktq_flushClemens Noss2-3/+2
to clarify the intention: On Sun, May 15, 2011 at 09:54:26PM +0200, Arend van Spriel wrote: > On 05/15/2011 08:10 PM, Hauke Mehrtens wrote: [...] >> Why don't you use the correct pointer type here instead of casting then >> around and the parameter names are meaningless too. >> >> static bool cb_del_ampdu_pkt(struct sk_buff *mpdu, struct >> cb_del_ampdu_pars *ampdu_pars) [...] > Actually this is a generic callback mechanism in which an additional > parameter can be passed, which can be of *any* type hence a void pointer > seems justified here and a meaningless name is used. I do agree that the > txi parameter will always be a struct sk_buff and should be indicated as > such. Reported-by: Hauke Mehrtens <[email protected]> Signed-off-by: Clemens Noss <[email protected]> Acked-by: Roland Vossen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: brcm80211: fix cast to pointer from integerClemens Noss5-10/+10
bcm_pktq_flush and related functions only ever get 0 or a pointer for arg, so make it a pointer. This might fix a crash on 64bit. Signed-off-by: Clemens Noss <[email protected]> Acked-by: Roland Vossen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: echo: Trivial indentation changesRahul Tank1-11/+15
--bcaec50161b317392804a33628dd Content-Type: text/plain; charset=ISO-8859-1 Hi Greg, Please find patch attached . Thanks, Rahul Hi Greg,<br><br> Please find patch attached .<br><br>Thanks,<br> Rahul<br> From ba1dbafaef1a87c1338c5bb0e6bba3be06071c90 Mon Sep 17 00:00:00 2001 From: Rahul Tank <[email protected]> Date: Sat, 14 May 2011 11:28:25 +0530 Subject: [PATCH] Trivial indentation cleanup Signed-off-by: Rahul Tank <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: modify project gcc flagsmatt mooney1-1/+1
Add -Werror, remove -Wstrict-prototypes, and change -W to -Wextra. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: move common cflags to configure.acmatt mooney3-3/+3
There are a standard set of cflags that are used in each makefile so let's set those in EXTRA_CFLAGS and enforce them for the entire project. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: remove gcc warningsmatt mooney3-3/+5
The warnings were for unused parameters, so __attribute__((unused)) has been added until it can be determined they are truly unneeded. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: remove revision $Id$matt mooney5-9/+2
This is git not CVS! Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: set libusbip version in configure.acmatt mooney2-1/+6
Move libusbip version setting to configure.ac so that version numbers can be found in a single location. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: cleanup makefilesmatt mooney3-30/+18
Remove unneeded comments; change deprecated flag INCLUDE to AM_CPPFLAGS and put -D option in *_CPPFLAGS; and use "simply expanded variables" in assignments. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: configure.ac: major overhaulmatt mooney1-70/+74
All parameters have been quoted; and autoscan was rerun so new headers, types, and functions were added. The deprecated macros AM_CONFIG_HEADER and AM_PROG_LIBTOOL were changed to AC_CONFIG_HEADERS and LT_INIT, respectively. The AS_HELP_STRING macro is used to avoid arbitrary spacing for proper help menu alignment, and AS_CASE to avoid quoting issues. And finally, the macros were realigned to allow mere mortals the ability to read them. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: update module name referencesmatt mooney5-13/+13
Change all references to the kernel modules to correspond with the new names. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: whitespace cleanupmatt mooney8-40/+40
Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: remove unused filesmatt mooney2-1/+0
Remove ChangeLog and NEWS. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: rename source directoriesmatt mooney20-5/+6
Rename cmd/ to src/ and lib/ to libsrc/. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: userspace: move manpages to separate directorymatt mooney5-2/+2
Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17staging: usbip: add userspace codeTakahiro Hirofuchi32-0/+19318
Add userspace code from svn trunk revision 175. Located at https://usbip.svn.sourceforge.net/svnroot/usbip/linux/trunk/src. Signed-off-by: matt mooney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Make netvsc_drv_exit() the module exit functionK. Y. Srinivasan1-7/+2
Make netvsc_drv_exit() the module exit function. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Make netvsc_drv_init the module init functionK. Y. Srinivasan1-7/+2
Now make netvsc_drv_init the module init function. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Move the dmi_check code to netvsc_drv_init()K. Y. Srinivasan1-5/+6
In preparation to eliminating netvsc_init(), move the dmi_check code to netvsc_drv_init(). Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Change the signature of netvsc_drv_init()K. Y. Srinivasan1-3/+3
Change the signature of netvsc_drv_init(). Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Move the dmi table declaration to earlier in the fileK. Y. Srinivasan1-14/+15
Move the dmi table declaration to earlier in the file. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Get rid of the used type struct netvsc_driverK. Y. Srinivasan1-7/+0
Now, get rid of the unused type struct netvsc_driver. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Get rid of the unused function drv_to_netvscdrv()K. Y. Srinivasan1-7/+0
Get rid of the unused function drv_to_netvscdrv(). Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Get rid of the empty function rndis_filter_init()K. Y. Srinivasan3-11/+0
Get rid of the empty function rndis_filter_init(). Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Get rid of some unnecessary/dead code from ↵K. Y. Srinivasan1-16/+0
rndis_filter.c After the cleanup that has been done, some code in rndis_filter.c is no longer needed; get rid of it. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Make netvsc_drv an instance of struct hv_driverK. Y. Srinivasan1-8/+7
In preparation to getting rid of struct netvsc_driver, make the variable netvsc_drv an instance of struct hv_driver. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Get rid of the unnecessary state from struct ↵K. Y. Srinivasan1-2/+0
netvsc_driver Now get rid of unused state (ring_buf_size) from struct netvsc_driver. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Get rid of the unnecessary assignment of ring_buf_sizeK. Y. Srinivasan1-2/+0
Since we pass ring size information differently, this assignment is not needed. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Get the ring size information from struct ↵K. Y. Srinivasan2-4/+5
netvsc_device_info Get the ring size information from struct netvsc_device_info. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Add ring_size element to struct netvsc_device_infoK. Y. Srinivasan1-0/+1
Add ring_size element to struct netvsc_device_info. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Get rid of unused state in struct netvsc_driverK. Y. Srinivasan1-2/+0
Now, get rid of unused state (req_ext_size) from struct netvsc_driver. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Get rid of an unnecessary assignment in rndis_filter.cK. Y. Srinivasan1-1/+0
Now that we compute the size of struct rndis_filter_packet where it is needed, get rid of the unnecessary assignment. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Directly get the size of rndis_filter_packetK. Y. Srinivasan1-3/+1
Directly get the size of rndis_filter_packet. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2011-05-17Staging: hv: netvsc_drv: Move the definition of struct rndis_filter_packet ↵K. Y. Srinivasan2-8/+7
to hyperv_net.h Move the definition of struct rndis_filter_packet to hyperv_net.h. Signed-off-by: K. Y. Srinivasan <[email protected]> Signed-off-by: Haiyang Zhang <[email protected]> Signed-off-by: Abhishek Kane <[email protected]> Signed-off-by: Hank Janssen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>