summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2014-04-26iio: st_sensors: announce registered sensorsLinus Walleij
It is pretty helpful to know already from dmesg that a certain device is successfully registered, instead of having to browse sysfs to see if it's actually there. Cc: Lee Jones <lee.jones@linaro.org> Cc: Denis CIOCCA <denis.ciocca@st.com> Signed-off-by: Linus Walleij <linus.walleij@linaro.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-04-23staging: iio: Fix typo in iioMasanari Iida
Correct spelling typo in comment within staging/iio Signed-off-by: Masanari Iida <standby24x7@gmail.com> Acked-by: Randy Dunlap <rdunlap@infradead.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-04-12Staging:iio:tsl2583 Switch from msleep to usleep range per timers-howto.txtMichael Welling
Signed-off-by: Michael Welling <mwelling@ieee.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-04-12Staging:iio:tsl2583 Remove quoted string split across lines warningsMichael Welling
Signed-off-by: Michael Welling <mwelling@ieee.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-04-12Staging:iio:tsl2583 Removes unwanted space before semicolonMichael Welling
Signed-off-by: Michael Welling <mwelling@ieee.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-04-11iio: adc: Fix exynos_adc dependenciesJean Delvare
EXYNOS_ADC shouldn't only depend on OF. It is useless on architectures other than Exynos so it should depend on ARCH_EXYNOS (which implies OF.) Signed-off-by: Jean Delvare <jdelvare@suse.de> Cc: Jonathan Cameron <jic23@kernel.org> Cc: Kukjin Kim <kgene.kim@samsung.com> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-04-07staging:iio:adc:spear rename spear_read_raw to spear_adc_read_rawJonathan Cameron
This brings it inline with all the other prefix usage in the driver. Signed-off-by: Jonathan Cameron <jic23@kernel.org> Acked-by: Hartmut Knaack <knaack.h@gmx.de>
2014-04-07staging:iio:adc:spear rename iodev -> indio_dev to fit with current conventionJonathan Cameron
How we ended up with indio_dev is a mystery, but we have so in the interests of consistency, move this driver over to that convention. Signed-off-by: Jonathan Cameron <jic23@kernel.org> Acked-by: Hartmut Knaack <knaack.h@gmx.de>
2014-04-07staging:iio:adc:spear rename device specific state structure to _stateJonathan Cameron
Also rename instances of it to st. This brings the driver inline with the conventions of IIO and avoids some naming confusion with some IIO structures which also use the _info postfix. Signed-off-by: Jonathan Cameron <jic23@kernel.org> Acked-by: Hartmut Knaack <knaack.h@gmx.de>
2014-04-07staging:iio:adc:spear_adc use info_mask_shared_by_all for samp freqJonathan Cameron
Using the core support makes this element available to in kernel users as well as to userspace under exactly the same interface as before. The intent is to move all sampling frequency control to this approach throughout IIO. Drop unused clk_high and clk_low whilst we are at it. Signed-off-by: Jonathan Cameron <jic23@kernel.org> Acked-by: Stefan Roese <sr@denx.de> Acked-by: Hartmut Knaack <knaack.h@gmx.de>
2014-04-07staging:iio:adc:spear_adc drop initialization of unused scan_typeJonathan Cameron
As the driver does not support the buffered interfaces of IIO this is not used. Signed-off-by: Jonathan Cameron <jic23@kernel.org> Acked-by: Stefan Roese <sr@denx.de>
2014-04-07staging:iio:adc:spear adc - prefix defines to avoid namespace clashes.Jonathan Cameron
Signed-off-by: Jonathan Cameron <jic23@kernel.org> Acked-by: Stefan Roese <sr@denx.de> Acked-by: Hartmut Knaack <knaack.h@gmx.de>
2014-03-29iio: imu: inv_mpu6050: Add compatibity with MPU6500Srinivas Pandruvada
Adding MPU6500 in target list for this driver. Description: Source Document: MPU-6500 Register Map and Descriptions Revision 2.1 Section 3: Register Map This section describes difference in terms device programmability between MPU6050 and MPU6500. These are different registers, which differs between MPU6050 and MPU6500. Addr Name --------------------- 1E LP_ACCEL_ODR 6C PWR_MGMT_2 77 XA_OFFSET_H 78 XA_OFFSET_L 7A YA_OFFSET_H 7B YA_OFFSET_L 7D ZA_OFFSET_H 7E ZA_OFFSET_L But the current MPU6050 driver doesn't use registers which are different except PWR_MGMT_2. The difference is support of "LP_WAKE_CTRL" at bit6-7 in MPU6050 mode. In MPU6500 they are not defined. In current mpu6050 driver, only values used for this register are for standby mode for gyro and accelerometer. In both case frequency of wakeups is set to default and not using bit 6-7. So this driver van as well support MPU6500. In addition MPU6500 can run MPU6050 mode by changing device trim settings. So changing config comments to allow MPU6500 to use this driver. When the driver is enhanced to support more functions, i2c driver data INV_MPU6500 or "WHO_AM_I" register can be used to add additional functionality. Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-29iio: ak8975: Added ACPI enumerationSrinivas Pandruvada
Added capability so that this device can be enumerated via ACPI. Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-22iio: ak8975 : Add AK8963 compatibility mode supportSrinivas Pandruvada
AK8963 and AK8975 use same register definitions, except the range of X,Y,Z values. Added support of 8963 based on i2c_device_id. Unfortunately there is no way to detect the type via registers, both device registers return 0x48 as id of chipset. Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-16iio:gyro:itg3200 - drop unreachable return ret. Highlighted by smatchJonathan Cameron
CHECK drivers/iio/gyro/itg3200_core.c drivers/iio/gyro/itg3200_core.c:114 itg3200_read_raw() info: ignoring unreachable code. Signed-off-by: Jonathan Cameron <jic23@kernel.org> Cc: Neil Brown <neilb@suse.de> Cc: Dan Carpenter <dan.carpenter@oracle.com>
2014-03-16iio:core: Fix bug in length of event info_mask and catch unhandled bits set ↵Jonathan Cameron
in masks. The unhandled bits case was highlighted by smatch: CHECK drivers/iio/industrialio-core.c drivers/iio/industrialio-core.c:719 iio_device_add_info_mask_type() error: buffer overflow 'iio_chan_info_postfix' 17 <= 31 CC [M] drivers/iio/industrialio-core.o CHECK drivers/iio/industrialio-event.c drivers/iio/industrialio-event.c:327 iio_device_add_event() error: buffer overflow 'iio_ev_info_text' 3 <= 3 The incorrect limit for the for_each_set_bit loop was noticed whilst fixing this other case. Note that as we only have 3 possible entries a the moment and the value was set to 4, the bug would not have any effect currently. It will bite fairly soon though, so best fix it now. Signed-off-by: Jonathan Cameron <jic23@kernel.org> Cc: Lars-Peter Clausen <lars@metafoo.de> Cc: Dan Carpenter <dan.carpenter@oracle.com>
2014-03-16Add support for Microchip Technology's MCP3426/7/8 ADCAngelo Compagnucci
This patch extends previous mcp3422 driver to support missing members of the family, mcp3426/7/8. Signed-off-by: Angelo Compagnucci <angelo.compagnucci@gmail.com> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-16staging:iio: Move ad799x driver out of stagingLars-Peter Clausen
The driver is now at a reasonable quality level. Move it out of staging. Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Acked-by: Hartmut Knaack <knaack.h@gmx.de> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-16staging:iio:ad799x: Move ring functions to the main fileLars-Peter Clausen
The ad799x_ring.c file is pretty much only one function these days. No need to keep it in a separate file. Since there is then only one user of the header left also move everything from the header to the main file. Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Acked-by: Hartmut Knaack <knaack.h@gmx.de> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-16iio: inkern: add iio_read_channel_average_rawSebastian Reichel
Add iio_read_channel_average_raw to support reading averaged raw values in consumer drivers. Signed-off-by: Sebastian Reichel <sre@debian.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-16iio: documentation: Add ABI documentation for *_mean_rawSebastian Reichel
Add ABI documentation for in_*_mean_raw files, which are already supported and used in the kernel for some time. Signed-off-by: Sebastian Reichel <sre@debian.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-16iio: Add AS3935 lightning sensor supportMatt Ranostay
AS3935 chipset can detect lightning strikes and reports those back as events and the estimated distance to the storm. Signed-off-by: Matt Ranostay <mranostay@gmail.com> Reviewed-by: Marek Vasut <marex@denx.de> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-15iio:as3935: Add DT binding docs for AS3935 driverMatt Ranostay
Document compatible string, required and optional DT properties for AS3935 chipset driver. Signed-off-by: Matt Ranostay <mranostay@gmail.com> Acked-by: Rob Herring <robh@kernel.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-15iio: force snprintf for PAGE_SIZE bufsKees Cook
This is a tiny preventative measure to make sure we can't write beyond PAGE_SIZE on the buffers being used in sysfs for iio. There is currently no way for this to happen, but the change makes this code more robust for the future. Signed-off-by: Kees Cook <keescook@chromium.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
2014-03-13staging: cxt1e1: Removed assignments from if statements.Chi Pham
Assignments removed from if statements. Fixed checkpatch warning such as indentation and negative error returns in adjacent code. Coccinelle was used for this patch. The following script found the match: @simple@ expression E1, E2; statement S1, S2; @@ + E1 = E2; if ( - (E1 = E2) + E1 ) S1 else S2 @left@ expression E0, E1, E2; statement S1, S2; @@ + E1 = E2; if ( - (E1 = E2) + E1 == E0 ) S1 else S2 Signed-off-by: Chi Pham <fempsci@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: lirc: Add fallthrough commentAndreea-Cristina Bernat
This patch adds fallthrough comments for the cases not preceded by break or fallthrough comment Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging:lustre: Fix Sparse Warning for Static Declarations in console.cHimangi Saraogi
This patch fixes the following Sparse warnings in console.c : drivers/staging/lustre/lnet/selftest/console.c:65:1: warning: symbol 'lstcon_node_get' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:118:1: warning: symbol 'lstcon_node_put' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:348:1: warning: symbol 'lstcon_sesrpc_condition' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:377:1: warning: symbol 'lstcon_sesrpc_readent' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:834:1: warning: symbol 'lstcon_batch_find' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1002:1: warning: symbol 'lstcon_batrpc_condition' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1145:1: warning: symbol 'lstcon_testrpc_condition' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1374:1: warning: symbol 'lstcon_test_find' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1389:1: warning: symbol 'lstcon_tsbrpc_readent' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1468:1: warning: symbol 'lstcon_statrpc_readent' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1492:1: warning: symbol 'lstcon_ndlist_stat' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1581:1: warning: symbol 'lstcon_debug_ndlist' was not declared. Should it be static? Signed-off-by: Himangi Saraogi <himangi774@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: rts5208: Remove unnecessary parentheses.Gulsah Kose
Return is not a method and doesn't need parentheses. Removed unnecaasary parentheses. Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging:comedi: Fix sparse warnings of incorrect type in assignmentHimangi Saraogi
This patch fixes the following sparse warnings: drivers/staging/comedi/drivers/mite.c:343:44: warning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers/mite.c:343:44: expected unsigned int [unsigned] [usertype] count drivers/staging/comedi/drivers/mite.c:343:44: got restricted __le32 [usertype] <noident> drivers/staging/comedi/drivers/mite.c:344:43: warning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers/mite.c:344:43: expected unsigned int [unsigned] [usertype] addr drivers/staging/comedi/drivers/mite.c:344:43: got restricted __le32 [usertype] <noident> drivers/staging/comedi/drivers/mite.c:346:43: warning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers/mite.c:346:43: expected unsigned int [unsigned] [usertype] next drivers/staging/comedi/drivers/mite.c:346:43: got restricted __le32 [usertype] <noident> drivers/staging/comedi/drivers/mite.c:351:45: warning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers/mite.c:351:45: expected unsigned int [unsigned] [usertype] next drivers/staging/comedi/drivers/mite.c:351:45: got restricted __le32 [usertype] <noident> Signed-off-by: Himangi Saraogi <himangi774@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: rts5208: Fix line over 80 characters.Gulsah Kose
Fix checkpatch.pl issues with line over 80 characters in rtsx.c Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: rtl8712: Delete space before semicolonAndreea-Cristina Bernat
This patch solves the warning: "space prohibited before semicolon". Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging:rtl8187se: Fix sparse warning in right shiftHimangi Saraogi
This patch fixes the following sparse warning: drivers/staging/rtl8187se/r8180_core.c:1328:40: warning: right shift by bigger than source value by adding parenthesis. Signed-off-by: Himangi Saraogi <himangi774@gmail.com> Acked-by: Bob Copeland <me@bobcopeland.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13/staging/vt6656: Delete extra paranthesis from macro definitionAndreea-Cristina Bernat
This patch solves the error of checkpatch.pl by deleting the extra paranthesis. Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging:rtl8187se:ieee80211: Fix sparse warning of cast to restricted __le16Himangi Saraogi
This patch fixes the following sparse warnings: drivers/staging/rtl8187se/ieee80211/ieee80211_crypt_tkip.c:496:17: warning: cast to restricted __le16 drivers/staging/rtl8187se/ieee80211/ieee80211_crypt_tkip.c:540:12: warning: cast to restricted __le16 drivers/staging/rtl8187se/ieee80211/ieee80211_crypt_tkip.c:586:12: warning: cast to restricted __le16 Signed-off-by: Himangi Saraogi <himangi774@gmail.com> Reviewed-by: Bob Copeland <me@bobcopeland.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13Staging: rtl8188eu: Fix sparse warnings.Elena Oat
This patch fixes following warnings produced by sparse: drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3867:31: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3867:31: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3869:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3869:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3869:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3871:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3871:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3871:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3873:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3873:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3873:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3875:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3875:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3875:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3878:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3878:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3878:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3880:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3880:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3880:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3883:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3883:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3883:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3897:9: warning: cast removes address space of expression While at it, I have also fixed the warnings of lines over 80 characters. Signed-off-by: Elena Oat <oat.elena@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: rtl8712: Remove unnecessary parentheses.Gulsah Kose
Remove unnecessary parentheses from rtl871x_cmd.c Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: unisys: visorutil: Use kzalloc instead of kmalloc with memsetIulia Manda
Fix coccinelle warnings of better allocation by using kzalloc. In these cases, kzalloc is preferred, as kmalloc may fail if it does not find contiguous memory. Signed-off-by: Iulia Manda <iulia.manda21@gmail.com> Acked-by: Luis R. Rodriguez <mcgrof@do-not-panic.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: android: ion: Use ERR_CAST instead of ERR_PTRIulia Manda
Fix the following coccinelle warnings in ion.c: drivers/staging/android/ion/ion.c:511:9-16: WARNING: ERR_CAST can be used with buffer drivers/staging/android/ion/ion.c:218:9-16: WARNING: ERR_CAST can be used with table drivers/staging/android/ion/ion.c:1150:9-16: WARNING: ERR_CAST can be used with dmabuf Signed-off-by: Iulia Manda <iulia.manda21@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: android: ion: Replace seq_printf with seq_putsIulia Manda
It is preferred to use seq_puts instead of seq_printf here, as it suffices string printing. Signed-off-by: Iulia Manda <iulia.manda21@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: android: ion: Fix quoted string split across linesIulia Manda
Join strings from two separated lines, even if this makes line longer than 80 characters. Signed-off-by: Iulia Manda <iulia.manda21@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: rtl8188eu: explicitly convert from __le16 to integerKristina Martšenko
BA_para_set (of type __le16) is implicitly cast into an integer for bitwise operations, after which the result is converted back into a __le16. Make the initial cast explicit to make the code clearer. In addition, refactor the code for readability. This also removes five sparse warnings of the following type: drivers/staging/rtl8188eu//core/rtw_mlme_ext.c:6114:70: warning: restricted __le16 degrades to integer Signed-off-by: Kristina Martšenko <kristina.martsenko@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging/media/solo6x10: Fix coding-style errorAndreea-Cristina Bernat
This patch fixes the error "space required after that ';'" for the file /drivers/staging/media/solo6x10/solo6x10-tx28.c Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: vt6655 : Fix placement of open braceAshley Smith
This patch fixes the checkpatch warning "that open brace { should be on the previous line" for a non-function statement block. Signed-off-by: Ashley Smith <ashley@eclipso.ch> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13Staging: dgnc: fix indentation in dgnc_mgmt.cIulia Manda
Align test parts in if statement on separate lines. Signed-off-by: Iulia Manda <iulia.manda21@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging:cxt1e1:remove unused code with #if 0Himangi Saraogi
The functions musycc_del_chan and musycc_del_chan_stats are not being used at the moment and may not be required. Remove them for now (with #if 0) till it is decided whether it is really required. Signed-off-by: Himangi Saraogi <himangi774@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13Staging: rtl8188eu: Fix warning of unchecked sscanf return value.Elena Oat
This patch fixes the warnings produced by checkpatch "unchecked sscanf return value" in file ioctl_linux.c. Signed-off-by: Elena Oat <oat.elena@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: rtl8187se: Fix braces in ieee80211/ieee80211_crypt_tkip.cAna Rey
This patch fixes braces errors as the codingStyle of the kernel recommends. Thus, this patch fixes these errors and warning messages found by checkpatch.pl: WARNING: braces {} are not necessary for single statement blocks ERROR: that open brace { should be on the previous line Signed-off-by: Ana Rey <anarey@gmail.com> Reviewed-by: Bob Copeland <me@bobcopeland.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: rtl8187se: Fix whitespaces in ieee80211/ieee80211_crypt_tkip.cAna Rey
This patch fixes whitespaces as the codingStyle of the kernel recommends. Thus, this patch fixes these errors found by checkpatch.pl: ERROR: space required before the open parenthesis '(' ERROR: space required before that '*' (ctx:OxV) ERROR: spaces required around that '=' (ctx:VxW) Signed-off-by: Ana Rey <anarey@gmail.com> Reviewed-by: Bob Copeland <me@bobcopeland.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
2014-03-13staging: rtl8187se: Deleted rtl8225z2_rf_set_mode functionAna Rey
After the next warning messages by sparse tools: drivers/staging/rtl8187se/r8180_rtl8225z2.c:609:6: warning: symbol 'rtl8225z2_rf_set_mode' was not declared. Should it be static? drivers/staging/rtl8187se/r8180_rtl8225z2.c:609:13: warning: ‘rtl8225z2_rf_set_mode’ defined but not used [-Wunused-function] static void rtl8225z2_rf_set_mode(struct net_device *dev) And the next search: ~/opw/dev/staging/drivers/staging/rtl8187se (staging-next) $ git grep rtl8225z2_rf_set_mode r8180_rtl8225z2.c:void rtl8225z2_rf_set_mode(struct net_device *dev) I decided to delete rtl8225z2_rf_set_mode function. Signed-off-by: Ana Rey <anarey@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
generated by cgit v1.2.3 (git 2.0.4) at 2024-11-15 00:22:13 +0000