summaryrefslogtreecommitdiffstats
path: root/drivers/staging/i4l
Commit message (Collapse)AuthorAgeFilesLines
* staging: i4l: delete the whole thingGreg Kroah-Hartman2017-01-0532-9626/+0
| | | | | | | | | | | It's now 2017, and a new LTS kernel has been chosen, so let's do what we said we would do in the TODO file and delete this code. If it's still needed, and a maintainer steps up to take it over, we will easily revert it. Cc: Arnd Bergmann <arnd@arndb.de> Cc: Karsten Keil <isdn@linux-pingi.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: pcbit: remove duplicated include from capi.cWei Yongjun2016-10-251-1/+0
| | | | | | | Remove duplicated include. Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: pcbit: drv: remove duplicated include from drv.cWei Yongjun2016-10-251-1/+0
| | | | | | | Remove duplicated include. Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: pcbit: replace some include asm/*.h with linux/*.hAlexander Alemayhu2016-10-165-7/+7
| | | | | | | | | | | | | | | Fixes the following checkpatch output: ./pcbit/capi.c:39: WARNING: Use #include <linux/io.h> instead of <asm/io.h> ./pcbit/capi.c:40: WARNING: Use #include <linux/string.h> instead of <asm/string.h> ./pcbit/callbacks.c:25: WARNING: Use #include <linux/io.h> instead of <asm/io.h> ./pcbit/layer2.c:39: WARNING: Use #include <linux/io.h> instead of <asm/io.h> ./pcbit/drv.c:34: WARNING: Use #include <linux/string.h> instead of <asm/string.h> ./pcbit/drv.c:35: WARNING: Use #include <linux/io.h> instead of <asm/io.h> ./pcbit/edss1.c:26: WARNING: Use #include <linux/io.h> instead of <asm/io.h> Signed-off-by: Alexander Alemayhu <alexander@alemayhu.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: remove parens around return statementElizabeth Ferdman2016-10-162-10/+10
| | | | | | | | | | | | | | | | | | | | | | | Remove parentheses surrounding return statements. Error found by checkpatch. changes made using coccinelle script: @@ expression e,e1; @@ ( return (e / e1); | return -( e -) ; ) Signed-off-by: Elizabeth Ferdman <gnudevliz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: icn: Fixed open brace should be on previous line errorHarman Kalra2016-10-161-2/+1
| | | | | | | | This patch resolves the following error caught by checkpatch.pl: ERROR: that open brace { should be on the previous line Signed-off-by: Harman Kalra <harman4linux@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: icn: prefer pr_* instead of printkHarman Kalra2016-10-161-1/+1
| | | | | | | | | | | This patch replaces call to printk with appropriate pr_* function thus addressing the following warning generated by the checkpatch script: Prefer [subsystem eg: netdev]_dbg([subsystem]dev, ... then dev_dbg(dev, ... then pr_debug(... to printk(KERN_DEBUG ... Signed-off-by: Harman Kalra <harman4linux@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: icn: Fixed open brace should be on previous line errorHarman Kalra2016-10-161-2/+1
| | | | | | | | This patch resolves the following error caught by checkpatch.pl: ERROR: that open brace { should be on the previous line Signed-off-by: Harman Kalra <harman4linux@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: add blank line after declarationsElizabeth Ferdman2016-10-163-0/+9
| | | | | | | | Fix checkpatch error "missing blank line after declarations" to conform to kernel coding style. Signed-off-by: Elizabeth Ferdman <gnudevliz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: move open brace to previous lineElizabeth Ferdman2016-10-161-2/+1
| | | | | | | | Fix checkpatch error "open brace should be on previous line" since only functions should have their opening brace on a new line. Signed-off-by: Elizabeth Ferdman <gnudevliz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: act2000: Move assignment out of if statementsayli karnik2016-09-201-1/+2
| | | | | | | | | | | | | | | | | | | This patch places assignments which are inside the if condition, above it. Done using coccinelle: @@ statement s; expression e; identifier id; @@ - if ((id = e)) + id = e; + if (id) s Signed-off-by: sayli karnik <karniksayli1995@gmail.com> Acked-by: Julia Lawall <julia.lawall@lip6.fr> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: Remove unused variableSandhya Bankar2016-09-201-2/+0
| | | | | | | | | | | | | | | | | | | | | | | Remove unused variable. This change is made by below coccinelle script: @@ type T; identifier i; constant C; @@ ( extern T i; | - T i; <+... when != i - i = C; ...+> ) Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Acked-by: Julia Lawall <julia.lawall@lip6.fr> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: Use !x instead of x == NULL.Sandhya Bankar2016-09-201-1/+1
| | | | | | | Use !x instead of x == NULL. This issue was found by checkpatch. Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: Do not initialise statics to 0.Sandhya Bankar2016-09-201-1/+1
| | | | | | | | Do not initialise statics to 0. Static variable by default initialise to 0, so no need to explicit initialisation. This issue was found by checkpatch. Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: Remove NULL comparison.Sandhya Bankar2016-09-201-2/+2
| | | | | | | | This patch removes the explicit NULL comparison. This issue was found by checkpatch. Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: correct misspelled "ressources" to "resources"Gargi Sharma2016-09-161-1/+1
| | | | | | | | change ressources to resources to fix the checkpatch issue 'ressources' may be misspelled. Signed-off-by: Gargi Sharma <gs051095@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: act2000: Remove braces for single statementAnson Jacob2016-09-021-2/+2
| | | | | | | | Fix checkpatch.pl warning: braces {} are not necessary for single statement blocks Signed-off-by: Anson Jacob <ansonjacob.aj@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: act2000: capi: Fix checkpatch warningAnson Jacob2016-08-311-2/+5
| | | | | | | | | Fix checkpath.pl warning: trailing statements should be on next line open brace '{' following function declarations go on the next line Signed-off-by: Anson Jacob <ansonjacob.aj@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: icn: fix incorrect type of argumentsSudip Mukherjee2016-08-311-14/+21
| | | | | | | | | | | | | | | | | | | | sparse was warning about incorrect type of argument: drivers/staging/i4l/icn/icn.c:1048:49: warning: incorrect type in argument 2 (different address spaces) drivers/staging/i4l/icn/icn.c:1048:49: expected void const [noderef] <asn:1>*from drivers/staging/i4l/icn/icn.c:1048:49: got unsigned char const [usertype] *buf drivers/staging/i4l/icn/icn.c:1476:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/i4l/icn/icn.c:1476:38: expected unsigned char const [usertype] *buf drivers/staging/i4l/icn/icn.c:1476:38: got unsigned char const [noderef] [usertype] <asn:1>*buf The function icn_writecmd() was used to copy from userspace and also from the kernelspace. Add another argument to the function to have two separate pointers, one for the userspace and one for the kernelspace. Based on the value of user as passed from the caller we use one of the two pointers. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: icn: remove blank linesSudip Mukherjee2016-08-311-3/+0
| | | | | | | | Blank lines are not needed after starting brace or before a closing brace. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: icn: space not needed after castSudip Mukherjee2016-08-311-33/+33
| | | | | | | No need provide a space after a typecast. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: act2000: remove unused macroSudip Mukherjee2016-08-311-7/+0
| | | | | | | | | The macro EVAL_PLCI and MAKE_PLCI are not being used. Remove them. But keep the comment preceding them as it contains information regarding message format. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: act2000: remove unused argumentSudip Mukherjee2016-08-312-4/+2
| | | | | | | | | The macro EVAL_NCCI was only being used in capi.c and the argument controller was not used. Remove the argument and at the same time remove the variable which now becomes unused. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: icn: use memdup_userSudip Mukherjee2016-08-311-11/+4
| | | | | | | | | | Its better to use memdup_user which does the same thing which this code has implemented. Also removed a related warning as we will be warned if allocation fails. Suggested-by: Fengguang Wu <fengguang.wu@intel.com> Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: Reformat braces around elseAnthony Sheldon2016-08-211-3/+1
| | | | | | | As suggested by Documentation/CodingStyle move braces onto same line as else. Signed-off-by: Anthony Sheldon <sheldon.anthony.sheldon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: icn: remove bracesSudip Mukherjee2016-08-211-6/+3
| | | | | | | Braces are not required in a single statement block. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: icn: donot assign in if statementSudip Mukherjee2016-08-211-11/+22
| | | | | | | | It is not the kernel coding style to make assignments in the if statement and checkpatch was warning us about it. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: icn: do not use return as a functionSudip Mukherjee2016-08-211-8/+8
| | | | | | | return is not a function so no need to use the parenthesis. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: icn: use setup_timer() and mod_timer().Muhammad Falak R Wani2016-08-211-10/+9
| | | | | | | | | | | | | | | Use setup_timer() instead of init_timer(), being the preferred/standard way to set a timer up. Also, quoting the mod_timer() function comment: -> mod_timer() is a more efficient way to update the expire field of an active timer (if the timer is inactive it will be activated). Use setup_timer and mod_timer to setup and arm a timer, to make the code cleaner and easier to read. Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: pcbit: layer2: use setup_timer() and mod_timer().Muhammad Falak R Wani2016-08-211-5/+3
| | | | | | | | | | | | | | | Use setup_timer() instead of init_timer(), being the preferred/standard way to set a timer up. Also, quoting the mod_timer() function comment: -> mod_timer() is a more efficient way to update the expire field of an active timer (if the timer is inactive it will be activated). Use setup_timer and mod_timer to setup and arm a timer, to make the code cleaner and easier to read. Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: pcbit: edss1: use setup_timer() and mod_timer().Muhammad Falak R Wani2016-08-211-5/+2
| | | | | | | | | | | | | | | Use setup_timer() instead of init_timer(), being the preferred/standard way to set a timer up. Also, quoting the mod_timer() function comment: -> mod_timer() is a more efficient way to update the expire field of an active timer (if the timer is inactive it will be activated). Use setup_timer and mod_timer to setup and arm a timer, to make the code cleaner and easier to read. Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: pcbit: drv: use setup_timer() and mod_timer().Muhammad Falak R Wani2016-08-211-11/+5
| | | | | | | | | | | | | | | Use setup_timer() instead of init_timer(), being the preferred/standard way to set a timer up. Also, quoting the mod_timer() function comment: -> mod_timer() is a more efficient way to update the expire field of an active timer (if the timer is inactive it will be activated). Use setup_timer and mod_timer to setup and arm a timer, to make the code cleaner and easier to read. Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: act2000: remove extra spaceSudip Mukherjee2016-05-011-6/+6
| | | | | | | | It is not kernel coding style to give an extra space after a cast. We get warned about it by checkpatch. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: act2000: remove blank line after braceSudip Mukherjee2016-05-011-1/+0
| | | | | | | checkpatch complains about an extra blank line after an opening brace. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: act2000: do not assign in ifSudip Mukherjee2016-05-011-3/+6
| | | | | | | | It is not the kernel coding style to assign values to some variable in if statement. Split them up into different statements. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: i4l: act2000: fix use of returnSudip Mukherjee2016-05-011-1/+1
| | | | | | | | checkpatch warns that return is not a function and as such the brace after it is not required. Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: pcbit: drv: Remove unnecessary semicolon.Sandhya Bankar2016-03-281-2/+2
| | | | | | | Remove unnecessary semicolon.This issue is detected by coccinelle script. Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: pcbit: capi: Add parentheses to complex macro.Sandhya Bankar2016-03-281-1/+1
| | | | | | | Add parentheses to complex macro. Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: pcbit: drv: Do not initialise statics to 0.Sandhya Bankar2016-03-281-2/+2
| | | | | | | Do not initialise statics to 0. Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: pcbit: edss1: Use !x instead of x == NULL.Sandhya Bankar2016-03-281-1/+1
| | | | | | | Use !x instead of x == NULL. Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: i4l: pcbit: layer2: Add parentheses to complex macro.Sandhya Bankar2016-03-281-1/+1
| | | | | | | Add parentheses to complex macro. Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* isdn: i4l: move active-isdn drivers to stagingArnd Bergmann2016-03-0532-0/+9636
The icn, act2000 and pcbit drivers are all for very old hardware, and it is highly unlikely that anyone is actually still using them on modern kernels, if at all. All three drivers apparently are for hardware that predates PCI being the common connector, as they are ISA-only and active PCI ISDN cards were widely available in the 1990s. Looking through the git logs, it I cannot find any indication of a patch to any of these drivers that has been tested on real hardware, only cleanups or global API changes. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Acked-by: Karsten Keil <isdn@linux-pingi.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
OpenPOWER on IntegriCloud