summaryrefslogtreecommitdiffstats
path: root/drivers/staging/cxt1e1
Commit message (Collapse)AuthorAgeFilesLines
* staging: cxt1e1: fix long lines warningSima Baymani2013-11-101-9/+18
| | | | | | | | Break up long lines per checkpatch warnings. Signed-off-by: Sima Baymani <sima.baymani@gmail.com> Reviewed-by: Lisa Nguyen <lisa@xenapiadmin.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: remove typedef comet_tSima Baymani2013-11-105-19/+17
| | | | | | | | | Remove typedef declaration per checkpatch warning. Built and loaded. Signed-off-by: Sima Baymani <sima.baymani@gmail.com> Reviewed-by: Lisa Nguyen <lisa@xenapiadmin.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: comet,h - fix placement of bracesSima Baymani2013-11-101-2/+1
| | | | | | | | Fix bad placement of braces per checkpatch warning. Signed-off-by: Sima Baymani <sima.baymani@gmail.com> Reviewed-by: Lisa Nguyen <lisa@xenapiadmin.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: comet.h - fix long lines warningSima Baymani2013-11-101-12/+24
| | | | | | | | | Fix long lines warnings from checkpatch which were introduced in previous patch in series due to reindentation. Signed-off-by: Sima Baymani <sima.baymani@gmail.com> Reviewed-by: Lisa Nguyen <lisa@xenapiadmin.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: comet.h - fix wrong indentationSima Baymani2013-11-101-291/+291
| | | | | | | | | | | | Reindent needed parts of comet.h per checkpatch warnings - whitespace changes only. This patch introduces long line warnings because of the reindentation. Those warnings are fixed by the next patch in series. Signed-off-by: Sima Baymani <sima.baymani@gmail.com> Reviewed-by: Lisa Nguyen <lisa@xenapiadmin.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging/cxt1e1:Fixes incorrect brace placementDulshani Gunawardhana2013-11-101-10/+5
| | | | | | | | | | This patch fixes the error "This open brace { should be on the above line" detected via checkpatch.pl. Signed-off-by: Dulshani Gunawardhana <dulshani.gunawardhana89@gmail.com> Reviewed-by: Zach Brown <zab@redhat.com> Reviewed-by: Josh Triplett <josh@joshtriplett.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: cxt1e1: remove space between function name and '('Ashvini Varatharaj2013-10-141-1/+1
| | | | | | | | | Fix checkpatch warning: WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by: Ashvini Varatharaj <ashvinivaratharaj@gmail.com> Reviewed-by: Josh Triplett <josh@joshtriplett.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: cxt1e1: moving { to the previous lineAshvini Varatharaj2013-10-141-2/+1
| | | | | | | | | Fix checkpatch error: ERROR: that open brace { should be on the previous line Signed-off-by: Ashvini Varatharaj <ashvinivaratharaj@gmail.com> Reviewed-by: Josh Triplett <josh@joshtriplett.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging:cxt1e1: Fixes whitespace around commas in sbew_ioc.hArchana kumari2013-10-111-12/+12
| | | | | | | | | | Fixes whitepace around commas. Detected via checkpatch.pl. ERROR: space required after that ',' Signed-off-by: Archana kumari <archanakumari959@gmail.com> Reviewed-by: Zach Brown <zab@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging/cxt1e1:Fixes suspect code indent warningDulshani Gunawardhana2013-10-061-4/+3
| | | | | | | | Fixes checkpatch.pl warning "suspect code indent for conditional statements" for comet.c. Signed-off-by: Dulshani Gunawardhana <dulshani.gunawardhana89@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: linux.c: Return negative error codesSachin Kamat2013-09-301-1/+1
| | | | | | | | Return negative error codes as is followed in the rest of the kernel. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: hwprobe.c: Return negative error codesSachin Kamat2013-09-301-5/+5
| | | | | | | | Return negative error codes as is followed in the rest of the kernel. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: musycc.c: Return negative error codesSachin Kamat2013-09-301-2/+2
| | | | | | | | Return negative error codes as is followed in the rest of the kernel. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: pmcc4_drv.c: Use NULL instead of 0Sachin Kamat2013-09-301-8/+8
| | | | | | | Pointers should be assigned NULL instead of 0. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: sbecrc.c: Use NULL instead of 0Sachin Kamat2013-09-301-2/+2
| | | | | | | Pointers should be assigned NULL instead of 0. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: hwprobe.c: Use NULL instead of 0Sachin Kamat2013-09-301-2/+2
| | | | | | | Pointers should be assigned NULL instead of 0. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: sbeproc.c: Use NULL instead of 0Sachin Kamat2013-09-301-1/+1
| | | | | | | Pointers should be assigned NULL instead of 0. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: linux.c: Use NULL instead of 0Sachin Kamat2013-09-301-13/+13
| | | | | | | Pointers should be assigned NULL instead of 0. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: sbeid.c: Use NULL instead of 0Sachin Kamat2013-09-301-1/+1
| | | | | | | Pointers should be assigned NULL instead of 0. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: sbecom_inline_linux.h: Return NULL instead of 0Sachin Kamat2013-09-301-2/+2
| | | | | | | Functions returning pointer should return NULL instead of 0. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: musycc.c: Use NULL instead of 0Sachin Kamat2013-09-301-14/+14
| | | | | | | Pointers should be assigned NULL instead of 0. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: Remove #define SBE_INCLUDE_SYMBOLSShaun Laing2013-08-129-98/+43
| | | | | | | | | | | | | | Removed the unneeded SBE_INCLUDE_SYMBOLS #define, and the associated STATIC #define, and replaced all occurances of STATIC with 'static'. This was in response to sparse warnings of the form "symbol 'XYZ' was not declared. Should it be static?". Removed a function prototype (musycc_init_port) as adding the 'static' declaration produced a new gcc warning. (musycc_init_port is only declared if SBE_WAN256T3_ENABLE is set) Signed-off-by: Shaun Laing <shaun@xresource.ca> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging/cxt1e1:Removing parantheses surrounding return argumentDulshani Gunawardhana2013-05-223-6/+6
| | | | | | | | This patch fixes the error 'return is not a function, parentheses are not required' that is found by using checkpatch.pi Signed-off-by: Dulshani Gunawardhana <dulshani.gunawardhana89@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging/cxt1e1:Fixing foo * bar should be foo *barDulshani Gunawardhana2013-05-2210-104/+104
| | | | | | | This patch fixes the variable naming error foo * bar should be foo *bar. Signed-off-by: Dulshani Gunawardhana <dulshani.gunawardhana89@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging/cxt1e1/comet.c:Modifying comment style and intentationDulshani Gunawardhana2013-05-221-316/+337
| | | | | | | | This patch modifies the commenting style by placing comments above the line of code commented upone. Signed-off-by: Dulshani Gunawardhana <dulshani.gunawardhana89@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging/cxt1e1/comet.c:Fixes indentation errorsDulshani Gunawardhana2013-05-221-256/+256
| | | | | | | | This patch fixes the "Code indent should usually use tabs" and "no spaces at front of line" warnings generated by checkpatch.pi. Signed-off-by: Dulshani Gunawardhana <dulshani.gunawardhana89@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging/cxt1e1/comet.c:Fixes space between function name and parenthesisDulshani Gunawardhana2013-05-221-149/+149
| | | | | | | | This patch corrects the error "Space prohibited between function name and parenthesis", generated by using checkpatch.pi. Signed-off-by: Dulshani Gunawardhana <dulshani.gunawardhana89@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: cxt1e1: Replacing asm/io.h with linux/io.hDulshani Gunawardhana2013-05-213-3/+3
| | | | | | | | This patch fixes the warning "Use #include <linux/io.h> instead of <asm/io.h>" generated by using checkpatch.pi. Signed-off-by: Dulshani Gunawardhana <dulshani.gunawardhana89@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: single_open() leaksAl Viro2013-05-051-1/+1
| | | | | Cc: stable@vger.kernel.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* Merge branch 'for-linus' of ↵Linus Torvalds2013-05-013-302/+175
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs Pull VFS updates from Al Viro, Misc cleanups all over the place, mainly wrt /proc interfaces (switch create_proc_entry to proc_create(), get rid of the deprecated create_proc_read_entry() in favor of using proc_create_data() and seq_file etc). 7kloc removed. * 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs: (204 commits) don't bother with deferred freeing of fdtables proc: Move non-public stuff from linux/proc_fs.h to fs/proc/internal.h proc: Make the PROC_I() and PDE() macros internal to procfs proc: Supply a function to remove a proc entry by PDE take cgroup_open() and cpuset_open() to fs/proc/base.c ppc: Clean up scanlog ppc: Clean up rtas_flash driver somewhat hostap: proc: Use remove_proc_subtree() drm: proc: Use remove_proc_subtree() drm: proc: Use minor->index to label things, not PDE->name drm: Constify drm_proc_list[] zoran: Don't print proc_dir_entry data in debug reiserfs: Don't access the proc_dir_entry in r_open(), r_start() r_show() proc: Supply an accessor for getting the data from a PDE's parent airo: Use remove_proc_subtree() rtl8192u: Don't need to save device proc dir PDE rtl8187se: Use a dir under /proc/net/r8180/ proc: Add proc_mkdir_data() proc: Move some bits from linux/proc_fs.h to linux/{of.h,signal.h,tty.h} proc: Move PDE_NET() to fs/proc/proc_net.c ...
| * cxt1e1: Don't use create_proc_read_entry()David Howells2013-04-293-302/+175
| | | | | | | | | | | | | | | | | | | | | | Don't use create_proc_read_entry() as that is deprecated, but rather use proc_create_data() and seq_file instead. Signed-off-by: David Howells <dhowells@redhat.com> cc: Bob Beers <bob.beers@gmail.com> cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> cc: devel@driverdev.osuosl.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
* | Drivers: Staging: cxt1e1: strncpy issue, need set zero at the end.Chen Gang2013-04-081-0/+1
|/ | | | | | | | | | | | | | | need set '\0' at the end. or cause issue. it is called by c4_ioctl in drivers/staging/cxt1e1/linux.c all things need be initialized, before provide them to user mode. so we can not use strlcpy instead of strncpy. code style: all contents of the file use 4 spaces instead of '\t', so this patch has to follow, now. Signed-off-by: Chen Gang <gang.chen@asianux.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: buffer overflow in do_del_chan()Dan Carpenter2013-01-251-1/+3
| | | | | | | | If we don't restrict "cp.channum" to 3 digits then the sprintf() will overflow. I've added a check and changed the sprintf() to snprintf(). Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: sbecrc.c: fixes coding style issueJohan Meiring2012-11-211-6/+6
| | | | | | | | This commit sorts out a coding style issue related to spaces between parentheses and function names. Signed-off-by: Johan Meiring <johanmeiring@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: sbecrc.c: fixes brace placementJohan Meiring2012-11-211-10/+5
| | | | | | | This commit sorts out the incorrect placement of braces in the file. Signed-off-by: Johan Meiring <johanmeiring@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: sbecrc.c: fixes indentation issuesJohan Meiring2012-11-211-53/+53
| | | | | | | | This commit converts several instances of space-based indentation to use tabs instead. Signed-off-by: Johan Meiring <johanmeiring@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: musycc.c: fixes placement of parenthesesJohan Meiring2012-11-211-245/+245
| | | | | | | | | This commit fixes several incorrect placements of parantheses, as identified by the checkpatch.pl tool. Signed-off-by: Johan Meiring <johanmeiring@gmail.com> Acked-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: musycc.c: fixes brace placementJohan Meiring2012-11-211-224/+90
| | | | | | | | This commit fixes incorrect brace placement as reported by the checkpatch.pl tool. Signed-off-by: Johan Meiring <johanmeiring@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: musycc.c: uses tabs for indentationJohan Meiring2012-11-211-921/+921
| | | | | | | | This commit converts several instances of space usage for indentation to tabs. Signed-off-by: Johan Meiring <johanmeiring@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: musycc.h: fixes brace placementJohan Meiring2012-11-211-4/+2
| | | | | | | This commit fixes incorrect brace placement. Signed-off-by: Johan Meiring <johanmeiring@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: musycc.h: fixes coding style issuesJohan Meiring2012-11-211-6/+6
| | | | | | | | This commit fixes instances of spaces being used for indentation instead of tabs. Signed-off-by: Johan Meiring <johanmeiring@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: musycc.h: fixes indentation issuesJohan Meiring2012-11-211-113/+113
| | | | | | | | This commit fixes indentation issues that were reported by the checkpatch.pl tool. Signed-off-by: Johan Meiring <johanmeiring@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: solve coding style problemDevendra Naga2012-09-041-5/+5
| | | | | | | | remove the spaces and replace with tabs at the beginning of a line and also remove space between function call and its open paranthesis Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: cxt1e1: use kernel's way of returning error codesDevendra Naga2012-09-042-6/+6
| | | | | | | | The error codes the kernel functions return are -ve numbers, convert this function to follow the other kernel functions Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Merge branch 'staging-next' of ↵Linus Torvalds2012-01-0918-524/+0
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging * 'staging-next' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging: (466 commits) net/hyperv: Add support for jumbo frame up to 64KB net/hyperv: Add NETVSP protocol version negotiation net/hyperv: Remove unnecessary kmap_atomic in netvsc driver staging/rtl8192e: Register against lib80211 staging/rtl8192e: Convert to lib80211_crypt_info staging/rtl8192e: Convert to lib80211_crypt_data and lib80211_crypt_ops staging/rtl8192e: Add lib80211.h to rtllib.h staging/mei: add watchdog device registration wrappers drm/omap: GEM, deal with cache staging: vt6656: int.c, int.h: Change return of function to void staging: usbip: removed unused definitions from header staging: usbip: removed dead code from receive function staging:iio: Drop {mark,unmark}_in_use callbacks staging:iio: Drop buffer mark_param_change callback staging:iio: Drop the unused buffer enable() and is_enabled() callbacks staging:iio: Drop buffer busy flag staging:iio: Make sure a device is only opened once at a time staging:iio: Disallow modifying buffer size when buffer is enabled staging:iio: Disallow changing scan elements in all buffered modes staging:iio: Use iio_buffer_enabled instead of open coding it ... Fix up conflict in drivers/staging/iio/adc/ad799x_core.c (removal of module_init due to using module_i2c_driver() helper, next to removal of MODULE_ALIAS due to using MODULE_DEVICE_TABLE instead).
| * Staging: cxt1e1: remove rcs keywords and logAlexander Beregalov2011-11-2618-416/+0
| | | | | | | | | | Signed-off-by: Alexander Beregalov <a.beregalov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| * Staging: cxt1e1: remove ifdef __cplusplusAlexander Beregalov2011-11-266-58/+0
| | | | | | | | | | Signed-off-by: Alexander Beregalov <a.beregalov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| * Staging: cxt1e1: remove unnecessary includesAlexander Beregalov2011-11-268-50/+0
| | | | | | | | | | | | | | Remove *BSD, SunOS and userspace includes. Signed-off-by: Alexander Beregalov <a.beregalov@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* | treewide: Fix typos in various parts of the kernel, and fix some comments.Justin P. Mattock2011-12-021-1/+1
|/ | | | | | | | | The below patch fixes some typos in various parts of the kernel, as well as fixes some comments. Please let me know if I missed anything, and I will try to get it changed and resent. Signed-off-by: Justin P. Mattock <justinmattock@gmail.com> Acked-by: Randy Dunlap <rdunlap@xenotime.net> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
* staging: Add module.h to more drivers implicitly using it.Paul Gortmaker2011-10-311-0/+1
| | | | | | | | This 2nd batch of implicit module.h users only appeared when we removed the unnecessary module.h from include/linux/miscdevice.h [The 1st batch is already present in Greg's staging tree.] Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
OpenPOWER on IntegriCloud