- Feb 17, 2017
-
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Derek Robson authored
Fixed style of block comments Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
simran singhal authored
Fix checkpatch issues: "CHECK: Blank lines aren't necessary before a close brace '}'". Signed-off-by:
simran singhal <singhalsimran0@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
simran singhal authored
Fixes checkpatch.pl warning: WARNING: missing space after struct definition WARNING: missing space after enum definition Signed-off-by:
simran singhal <singhalsimran0@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
simran singhal authored
Fixes checkpatch.pl warning: WARNING: missing space after struct definition Signed-off-by:
simran singhal <singhalsimran0@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Shiva Kerdel authored
Spaces should be added around operators to improve readability and are required in some cases. Signed-off-by:
Shiva Kerdel <shiva@exdev.nl> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Shiva Kerdel authored
Multiple blank lines shouldn't be used. Signed-off-by:
Shiva Kerdel <shiva@exdev.nl> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Shiva Kerdel authored
A blank line should be added after function/struct/union/enum declarations. Signed-off-by:
Shiva Kerdel <shiva@exdev.nl> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Stefan Svinciak authored
Newer version is more readable and needs less changes if/when timer_struct is to be changed. Signed-off-by:
Stefan Svinciak <xsvinc1@fi.muni.cz> Signed-off-by:
Jiri Slaby <jslaby@suse.cz> Cc: David Kershner <david.kershner@unisys.com> Cc: <sparmaintainer@unisys.com> Acked-by:
David Kershner <david.kershner@unisys.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael S. Hansen authored
Fixes checkpatch warning: Avoid multiple line dereference Signed-off-by:
Michael S. Hansen <michael.schacht.hansen@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Arushi Singhal authored
Fix checkpatch issues: "CHECK: Alignment should match open parenthesis" Signed-off-by:
Arushi Singhal <arushisinghal19971997@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Arushi Singhal authored
Fix checkpatch issues: "CHECK: Alignment should match open parenthesis" Signed-off-by:
Arushi Singhal <arushisinghal19971997@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Bo YU authored
Missing a blank line after declarations Signed-off-by:
Bo YU <tsu.yubo@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dan Carpenter authored
We accidentally allocate sizeof(void *) bytes instead of 112 bytes. It results in memory corruption. Fixes: 23b028c8 ("staging: bcm2835-audio: initial staging submission") Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dan Carpenter authored
We intended to mask away the upper bits but there is a "&&" vs "&" typo so it's broken. Fixes: 23b028c8 ("staging: bcm2835-audio: initial staging submission") Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Gargi Sharma authored
remove the space before the tab to fix the the checkpatch issue, no spaces before tabs. Signed-off-by:
Gargi Sharma <gs051095@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Maxime Rossi Bellom authored
sparse warn about _u32 used for two arguments in function osc_match_base instead of enum ldlm_type and enum ldlm_mode as used in the prototype. Signed-off-by:
Maxime Rossi Bellom <mrossibellom@gmail.com> Reviewed-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Cheah Kok Cheong authored
Currently this module needs to be manually configured by COMEDI userspace tool before the test waveform can be read by a COMEDI compatible application. This patch adds auto-configuration capability and makes it the default loading option. This is achieved by creating a device during init to stand in for a real hardware device. This allows comedi_auto_config() to perform auto-configuration. With this patch, the test waveform can be read by a COMEDI compatible application without needing manual configuration. Previous behaviour is still selectable via module loading parameter. Module loading without passing any parameter will default to auto-configuration with the same default waveform amplitude and period values. For auto-configuration, different amplitude and period values can be set via module loading parameters. Tested on Xubuntu 16.04 using Xoscope ver: 2.0 which is available in the Ubuntu repository. Xoscope is a COMEDI compatible digital oscilloscope application. For manual configuration, only module loading/unloading is tested. Here are the truncated dmesg output. [sudo modprobe comedi_test] comedi_test: 1000000 microvolt, 100000 microsecond waveform attached driver 'comedi_test' has successfully auto-configured 'comedi_test'. [sudo modprobe comedi_test amplitude=2500000 period=150000] comedi_test: 2500000 microvolt, 150000 microsecond waveform attached driver 'comedi_test' has successfully auto-configured 'comedi_test'. [sudo modprobe comedi_test noauto=1] comedi_test: module is from the staging directory, the quality is unknown, you have been warned. For those without an actual hardware, the comedi_test module is as close as one can get to test the COMEDI system. Having both auto and manual configuration capability will broaden the test function of this module. Hopefully this will make it easier for people to check out the COMEDI system and contribute to its development. Signed-off-by:
Cheah Kok Cheong <thrust73@gmail.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
- Feb 15, 2017
-
-
Artur Lorincz authored
Fixed multiple line dereference for &cmd->scan_begin_arg. Signed-off-by:
Artur Lorincz <larturus@yahoo.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Artur Lorincz authored
Added the const type qualifier to the comedi_lrange structure. Signed-off-by:
Artur Lorincz <larturus@yahoo.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Arushi Singhal authored
staging:nvec:nvec_ps2.c: Prefering kzalloc(sizeof(*ser_dev)...) over kzalloc(sizeof(struct serio)...) Prefer kzalloc(sizeof(*ser_dev)...) over kzalloc(sizeof(struct serio)...) as reported by checkpatch.pl. Signed-off-by:
Arushi Singhal <arushisinghal19971997@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Shishir Kulkarni authored
Fixes checkpatch.pl warning: "WARNING: line over 80 characters" Signed-off-by:
Shishir Kulkarni <shishir.kulkarni.2307@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Hendrik v. Raven authored
The VC_AUDIO_MAX_MSG_LEN macro is not used by anything in the bcm2835-audio driver. Signed-off-by:
Hendrik v. Raven <hendrik@consetetur.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Abhijit Naik authored
bcm2835-vchiq.c: fixing ERROR: Checkpatch warnings due to extra blank lines or absence of blank lines where needed. Signed-off-by:
Abhijit Naik <abhijitnaik27@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Franck Demathieu authored
The i2c address is unsigned according to the dt-bindings. Fix sparse issue (-Wtypesign): drivers/staging/nvec/nvec.c:781:35: warning: incorrect type in argument 3 (different signedness) drivers/staging/nvec/nvec.c:781:35: expected unsigned int [usertype] *out_value drivers/staging/nvec/nvec.c:781:35: got int *<noident> Signed-off-by:
Franck Demathieu <fdemathieu@gmail.com> Acked-by:
Marc Dietrich <marvin24@gmx.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Bogdan Purcareata authored
Compiling the fsl-mc bus driver will yield a couple of static analysis errors: warning: symbol 'fsl_mc_msi_domain_alloc_irqs' was not declared warning: symbol 'fsl_mc_msi_domain_free_irqs' was not declared. warning: symbol 'its_fsl_mc_msi_init' was not declared. warning: symbol 'its_fsl_mc_msi_cleanup' was not declared. Since these are properly declared, but the header is not included, add it in the source files. This way the symbol is properly exported. Signed-off-by:
Bogdan Purcareata <bogdan.purcareata@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Karthik Nayak authored
The struct 'ni_private' holds the variable 'serial_number' which post assignment is never used. Remove the variable and code pertaining to obtaining its value. As a side note, this also fixes the following sparse error: drivers/staging/comedi/drivers//ni_pcimio.c:1229:32: warning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers//ni_pcimio.c:1229:32: expected restricted __be32 [usertype] serial_number drivers/staging/comedi/drivers//ni_pcimio.c:1229:32: got unsigned int Signed-off-by:
Karthik Nayak <Karthik.188@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: usleep_range is preferred over udelay Signed-off-by:
Saber Rezvani <irsaber@gmail.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: usleep_range is preferred over udelay Signed-off-by:
Saber Rezvani <irsaber@gmail.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: spaces preferred around that '|' (ctx:VxV) Signed-off-by:
Saber Rezvani <irsaber@gmail.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: Please use a blank line after function/struct/union/enum declarations Signed-off-by:
Saber Rezvani <irsaber@gmail.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
- Feb 13, 2017
-
-
Jacob Zachariah authored
Fix multiple checkpatch.pl warnings WARNING: function definition argument '..' should also have an identifier name Signed-off-by:
Jacob Zachariah <jacob.zac316@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Ran Algawi authored
Fixed a coding style issue, where an octal value was needed insted of decimal. Signed-off-by:
Ran Algawi <ran.algawi@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
- Feb 12, 2017
-
-
simran singhal authored
This patch fixes following checkpatch.pl warnings: WARNING:do not add new typedefs. All the related files have been modified. Signed-off-by:
simran singhal <singhalsimran0@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-