- 19 Jun, 2014 13 commits
-
-
Jes Sorensen authored
Not sure how this happened, but one should never copy the IEs in front of the beacon frame info. This could lead to some nasty corrupted beacon frames hitting the wire if running AP mode - ouf! Signed-off-by:
Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Jes Sorensen authored
This gets rid of a bunch of hard coded offsets and reduces the dependency of the ugly rtw_get_*_from_ie() functions. Signed-off-by:
-
Jes Sorensen authored
We already have the capability info in struct wlan_bssid_ex, no point in searching for it once again. Signed-off-by:
-
Jes Sorensen authored
Pull out the core info beacon_interval, capability, and tsf and update cur_network.network with the info in rtw_add_beacon() instead of relying on it being in ->IEs. This will help later when getting rid of the beacon struct info from ->IEs and only carrying the actual IEs there. Signed-off-by:
-
Jes Sorensen authored
Signed-off-by:
-
Jes Sorensen authored
This is to be more consistent mapping the names in wlan_bssid_ex to those in struct ieee80211_mgmt. Signed-off-by:
-
Jes Sorensen authored
No point in pulling capability info out of the IE array when it's already stored in struct wlan_bssid_ex Signed-off-by:
-
Jes Sorensen authored
Signed-off-by:
-
Jes Sorensen authored
Use the beacon interval we already retreived in collect_bss_info() instead of pulling it out of the saved IE array again. Signed-off-by:
-
Jes Sorensen authored
Signed-off-by:
-
Jes Sorensen authored
There is no point in creating a fake beacon frame to call cfg80211_inform_bss_frame() when we can just call cfg80211_inform_bss() with the parameters we were sticking into the fake frame. Signed-off-by:
-
Jes Sorensen authored
Signed-off-by:
-
Greg Kroah-Hartman authored
Merge tag 'iio-for-3.17a' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio into staging-next Jonathan writes: First round of new drivers, cleanups and functionality for the 3.17 cycle. New drivers * t5403 barometric pressure sensor * kxcjk1013 accelerometer (with a locking followup fix). * ak09911 digital compass Documentation * ABI docs for proximity added (interface has been there a long time but somehow snuck through without being documented) * Move iio-trig-sysfs documentation out of staging (got left behind when the driver moved some time ago). Cleanups * drop the timestamp argument from iio_trigger_poll(_chained) as nothing has been done with it for some time. * ad799x kerneldoc for ad799x_chip brought up to date. * replace a number of reimplementations of the GENMASK macro and use the BIT macro to cleanup a few locations. * bring the iio_event_monitor example program up to date with new device types. * fix some incorrect function prototypes in iio_utils.h example code. * INDIO_RING_TRIGGERED to INDIO_BUFFER_TRIGGERED fix in docs. This got left behind after we renamed it a long time back. * fix error handling in the generic_buffer example program. * small tidy ups in the iio-trig-periodic-rtc driver. * Allow reseting iio-trig-periodic-rtc frequency to 0 (default) after it has changed. * Trivial tidy ups in coding style in iio_simply_dummy
-
- 18 Jun, 2014 27 commits
-
-
Anil Belur authored
fixed: WARNING: suspect code indent for conditional statements (32, 32) Signed-off-by:
Anil Belur <askb23@gmail.com> Signed-off-by:
-
Anil Belur authored
fixed: WARNING: line over 80 characters, used a new variable 'size_index' to store the offset. Replaced "unsigned long" with "loff_t" type for 'cur_index'. Removed the parenthesis around the second part of the if statement following the '||' as its not required. Signed-off-by:
-
Anil Belur authored
fixed: ERROR: inline keyword should sit between storage class and type Signed-off-by:
-
Anil Belur authored
fixed: ERROR: do not initialise statics to 0 or NULL Signed-off-by:
-
Anil Belur authored
fixed: WARNING: Missing a blank line after declarations Signed-off-by:
-
Anil Belur authored
fixed warning: * WARNING: min() should probably be min_t(__u32, desc.ld_tgt_count, LOV_MAX_STRIPE_COUNT) Signed-off-by:
-
Anil Belur authored
fixed "WARNING: labels should not be indented" Signed-off-by:
-
Anil Belur authored
Fixed "ERROR: need consistent spacing around '+' (ctx:WxV)" Signed-off-by:
-
Scott Weir authored
This patch fixes the sparse warnings in drivers/staging/lustre/lustre/ptlrpc/sec_lproc.c: sec_lproc.c:58:6: warning: symbol 'sec_flags2str' was not declared. Should it be static? Signed-off-by:
Scott Weir <sjw0410@gmail.com> Signed-off-by:
-
Vitaly Osipov authored
Joe Perches mentioned on driverdev-devel that memset after LIBCFS_ALLOC is not necessary as it is already done during LIBCFS_ALLOC_POST. This commit removes these unnecessary memsets. Based on the results of running a cocci patch along the lines of: @@ expression E1, E2; @@ LIBCFS_ALLOC (E1,E2); ... - memset(E1,0,E2); Signed-off-by:
Vitaly Osipov <vitaly.osipov@gmail.com> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Marc Fite authored
This is a patch in order to remove 2 lines with spaces at the start of the line. Signed-off-by:
Marc Fite <marc@fite.cat> Signed-off-by:
-
John Church authored
This patch fixes the following sparse warnings for drivers/staging/lustre/lnet/lnet/router.c: router.c:139:1: warning: symbol 'lnet_ni_notify_locked' was not declared. Should it be static? router.c:277:1: warning: symbol 'lnet_add_route_to_rnet' was not declared. Should it be static? Signed-off-by:
John Church <sleeveroller@gmail.com> Signed-off-by:
-
Honggang Li authored
Signed-off-by:
Honggang Li <enjoymindful@gmail.com> Signed-off-by:
-
Honggang Li authored
Signed-off-by:
-
Tair Rzayev authored
Include necessary header files to avoid "should be static" sparse warning Signed-off-by:
Tair Rzayev <tair.rzayev@gmail.com> Signed-off-by:
-
Jeff Oczek authored
Change style in function headers from 'type * ptr' to 'type *ptr' Signed-off-by:
Jeff Oczek <jeffoczek@gmail.com> Signed-off-by:
-
Jeff Oczek authored
Add more descriptive names to function pointers in definition of struct dbll_attrs Signed-off-by:
-
Radek Dostal authored
Style-only modifications to make checkpatch.pl -f a bit happier. Fixes warning: "void function return statements are not generally useful" Signed-off-by:
Radek Dostal <rd@radekdostal.com> Signed-off-by:
-
Radek Dostal authored
Style-only modifications to make checkpatch.pl -f a bit happier. Fixes warning: "Missing a blank line after declarations" Signed-off-by:
-
Mathias Engan authored
Add blank line after declarations in eucr_suspend and eucr_resume as reported by checkpatch. Signed-off-by:
Mathias Engan <mathias.engan@gmail.com> Signed-off-by:
-
Julia Lawall authored
Find_first_zero_bit considers BITS_PER_LONG bits at a time, and thus may return a larger number than the maximum position argument if that position is not a multiple of BITS_PER_LONG. The semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression e1,e2,e3; statement S1,S2; @@ e1 = find_first_zero_bit(e2,e3) ... if (e1 - == + >= e3) S1 else S2 // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Rickard Strandqvist authored
There is a risk that the variables will be used without being initialized. Has also improved error handling, after an email proposal from Dan Carpenter. Signed-off-by:
Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> Signed-off-by:
-
Wolfgang Ocker authored
Fix a style issue reported by checkpatch.pl for the Eudyptula challenge Signed-off-by:
Wolfgang Ocker <weo@weo1.de> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Hoang Tran authored
Using an else following a break or return can unnecessarily indent code blocks. This patch fixes coding style reported by checkpatch.pl, a part of eudyptula challenge. Signed-off-by:
Hoang Tran <hoangtran.gwr@gmail.com> Signed-off-by:
-
H Hartley Sweeten authored
The {read,write}[bwl] macros used to access the registers in the ni_atmio, ni_mio_cs, and ni_pcimio drivers and the included ni_mio_common.c file all rely on a local variable having a specific name. They also require some of the ni_mio_common code to need a __maybe_unused tag on the devpriv local variable. Remove all the macros by converting them into private functions and storing the callbacks in the private data. Signed-off-by:H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
-
H Hartley Sweeten authored
Remove all the unsed members from the private data for the ni mio drivers. The ao0p and ao1p members are only used in the ni_ao_reset() function and the code is commented out. Remove them, and the commented out code, as well. Signed-off-by:
-
H Hartley Sweeten authored
This macro is used to create the private data structure that is used by the ni_atmio, ni_mio_cs, and ni_pcimio drivers. These drivers all include the ni_mio_common.c source to provide most of the driver functionality. The only driver specific information needed to convert the macro into a proper struct definition is the MAX_N_CALDACS define. This define is used to size a couple array members in the struct. The ni_atmio and ni_mio_cs drivers both define MAX_N_CALDACS as 32. The ni_pcimio driver defines it as (16+16+2). The ni_mio_common file only uses this define to sanity check that the struct members are large enough for the number of channels in the calibration subdevice. Move the MAX_N_CALDACS define to ni_stc.h and set it to the largest number of caldacs (34). The ni_atmio and ni_mio_cs drivers also add one additional member to the private data struct before using the NI_PRIVATE_COMMON macro. For the ni_atmio driver, the struct pnp_dev pointer can be saved in the comedi_device as the 'hw_dev'. The (*detach) of this driver can then use to_pnp_dev() to get it back when detaching the pnp device. In the ni_mio_cs driver, the struct pcmia_device pointer is not used so it can simply be removed. The NI_PRIVATE_COMMON macro can then be converted into a proper struct definition. Signed-off-by:
-
