summaryrefslogtreecommitdiffstats
path: root/drivers/staging/dgap
Commit message (Collapse)AuthorAgeFilesLines
* staging: dgap: dgap_tty: Do not use 0 for pointersSachin Kamat2013-10-111-3/+4
| | | | | | | | 0 should not be used instead of NULL for pointers. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Cc: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: dgap_fep5: Do not use 0 for NULL pointerSachin Kamat2013-10-111-1/+1
| | | | | | | | Do not compare NULL pointer with 0. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Cc: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: dgap_fep5: Remove braces around returnSachin Kamat2013-10-111-19/+19
| | | | | | | | Braces are not needed around return values. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Cc: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: dgap_fep5: Remove braces around single line statementsSachin Kamat2013-10-111-10/+5
| | | | | | | | Single line statements do not require braces. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Cc: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: tty.c: removes smatch warnings "redundant null check"Lidza Louina2013-09-251-12/+6
| | | | | | | | | | | | | This patch removes these smatch warnings: redundant null check on dgap_TmpWriteBuf calling kfree() redundant null check on brd->SerialDriver->ttys calling kfree() redundant null check on brd->PrintDriver->ttys calling kfree() The code checked to see if these variables are null before freeing. This check isn't needed. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: tty.c: removes smatch warning "ignoring unreachable code"Lidza Louina2013-09-251-12/+0
| | | | | | | | | | | | | This patch removes this smatch warning: info: ignoring unreachable code. There were instances where there was extra code after the default action in switch statements. These default actions ended with a break so the code wasn't being run at anytime. This patch removes that extra code. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: driver.c: removes smatch warning "redundant null check"Lidza Louina2013-09-251-6/+3
| | | | | | | | | | | This patch removes these smatch warnings from dgap_driver.c: redundant null check on dgap_config_buf calling kfree() redundant null check on brd->flipbuf calling kfree() redundant null check on brd->flipflagbuf calling kfree() Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* drivers: staging: dgap: move DG_NAME and DG_PART from "Makefile" to ↵Chen Gang2013-09-252-2/+3
| | | | | | | | | | | | | | | | | | | "dgap_driver.h" Normally, the macros from command line are system specific macros (e.g __linux, __KERNEL__ ...), and module own macros are usually defined in their header files. DG_NAME and DG_PART are driver 'dgap' owned macros which are used by multiple files within driver, and need be defined in the driver main header file. So move DG_NAME and DG_PART to "dgap_driver.h", it not only can make code clearer, but also can avoid compiling failure when EXTRA_CFLAGS appended to make command line (e.g. "EXTRA_CFLAGS=-W"). Signed-off-by: Chen Gang <gang.chen@asianux.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Merge 3.12-rc2 into staging-next.Greg Kroah-Hartman2013-09-251-7/+10
|\ | | | | | | | | | | | | | | This resolves the merge problem with two iio drivers that Stephen Rothwell pointed out. Reported-by: Stephen Rothwell <sfr@canb.auug.org.au> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: dgap: fix overflows and format stringsKees Cook2013-09-171-7/+10
| | | | | | | | | | | | | | | | | | The boot message buffer could potentially overflow the stack and the heap. Additionally make sure format strings could not leak into printk() calls. Signed-off-by: Kees Cook <keescook@chromium.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: dgap: Fix typo in staging/dgapMasanari Iida2013-09-177-11/+11
| | | | | | | | | | | | | | Correct spelling typo in comments. Signed-off-by: Masanari Iida <standby24x7@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: dgap: Remove casting the return value which is a void pointerJingoo Han2013-09-172-13/+15
| | | | | | | | | | | | | | | | | | Casting the return value which is a void pointer is redundant. The conversion from void pointer to any other pointer type is guaranteed by the C programming language. Signed-off-by: Jingoo Han <jg1.han@samsung.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: dgap: Remove version check in dgap_kcompat.hSachin Kamat2013-09-171-29/+0
| | | | | | | | | | | | | | | | Code should be for the kernel version it is merged in. Version check is not necessary. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: dgap: Remove version check in dgap_driver.cSachin Kamat2013-09-171-4/+0
|/ | | | | | | | Code should be for the kernel version it is merged in. Version check is not necessary. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: Add missing #include <linux/slab.h>Geert Uytterhoeven2013-08-301-0/+1
| | | | | | | | | | | drivers/staging/dgap/dgap_driver.c: In function ‘dgap_cleanup_module’: drivers/staging/dgap/dgap_driver.c:423: error: implicit declaration of function ‘kfree’ drivers/staging/dgap/dgap_driver.c: In function ‘dgap_driver_kzmalloc’: drivers/staging/dgap/dgap_driver.c:940: error: implicit declaration of function ‘kmalloc’ drivers/staging/dgap/dgap_driver.c:940: warning: initialization makes pointer from integer without a cast Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: Remove version.h header inclusion in dgap_trace.cSachin Kamat2013-08-281-1/+0
| | | | | | | | | version.h header inclusion is not necessary as detected by versioncheck. Cc: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: Remove version.h header inclusion in dgap_sysfs.cSachin Kamat2013-08-281-1/+0
| | | | | | | | | version.h header inclusion is not necessary as detected by versioncheck. Cc: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: Remove version.h header inclusion in dgap_parse.cSachin Kamat2013-08-281-1/+0
| | | | | | | | | version.h header inclusion is not necessary as detected by versioncheck. Cc: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: removes proc.hLidza Louina2013-08-234-154/+0
| | | | | | | | This patch removes the proc.h file and removes '#include dgap_proc.h' from headers in the driver. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: removes proc.cLidza Louina2013-08-231-1923/+0
| | | | | | | | | This patch removes proc.c. This code isn't needed anymore because a previous patch removed references to it and because the kernel uses sysfs instead of proc. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: removes references to proc codeLidza Louina2013-08-232-10/+0
| | | | | | | | | This patch removes references to proc functions and structs in the code. The kernel uses sysfs instead of proc now. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: removes mgmt.hLidza Louina2013-08-235-37/+0
| | | | | | | | | This patch removes mgmt.h and the headers that include it of files in dgap. This file isn't needed for the driver to work with the kernel. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: removes mgmt.cLidza Louina2013-08-232-762/+1
| | | | | | | | This patch removes mgmt.c. This file isn't needed for the driver to work with the kernel. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: removes references to mgmt codeLidza Louina2013-08-231-3/+0
| | | | | | | | | This patch removes references to mgmt functions from the code. The files mgmt.c and mgmt.h will be removed in the patches that follow. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: ifdef HAVE_UNLOCKED_IOCTL conditionalsLidza Louina2013-08-223-15/+0
| | | | | | | | | This patch removes the HAVE_UNLOCKED_IOCTL conditional statements from driver.c, mgmt.c and mgmt.h. This was used to support older kernels. It isn't needed now. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: mgmt.c: removes unused variableLidza Louina2013-08-221-1/+0
| | | | | | | | This patch removes the unused variable page in mgmt.c. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: tty.c: fixes incompatible type errorLidza Louina2013-08-221-1/+1
| | | | | | | | | This patch fixes the error: incompatible types when assigning to type ‘struct ktermios *’ from type ‘struct ktermios’ Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: tty.c: fixes ioctl param listLidza Louina2013-08-221-37/+2
| | | | | | | | | | | | The declaration for the ioctl function has changed. The previous version of this declaration took struct file *file as a parameter and the new one does not. This patch removes that parameter. It also removes cases for the commands TIOCGETP TCGETS and TCGETA. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: tty.c: removes read_cnt, real_raw and rawreadokLidza Louina2013-08-221-33/+14
| | | | | | | | | This patch removes the use of read_cnt, real_raw and rawreadok. These variables don't exist in the new API. Reading the data raw is no longer supported by the tty layer. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: tty.c: fixes termios errorLidza Louina2013-08-221-14/+14
| | | | | | | | This patch fixes this error: invalid type argument of ‘->’ (have ‘struct ktermios’). Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: tty.c: fixes errors with tty function callsLidza Louina2013-08-221-5/+5
| | | | | | | | | | | | This patch fixes errors with the tty function calls tty_buffer_request_room, tty_insert_flip_string_flags, tty_insert_flip_string and tty_flip_buffer_push. They now take struct tty_port as a parameter instead of tty_struct. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: fep5.c: fixes errors with tty function callsLidza Louina2013-08-221-3/+3
| | | | | | | | | | This patch fixes errors with the tty function calls tty_buffer_request_room, tty_insert_flip_char and tty_flip_buffer_push. They now take struct tty_port as a parameter instead of tty_struct. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: removes references to proc codeLidza Louina2013-08-224-10/+3
| | | | | | | | | | This patch removes references to proc code in this driver. It still has proc.c, proc.h and a board state called NEEDS_PROC_CREATION. All three of these will be removed in another patch. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: fixes incompatible type errorLidza Louina2013-08-221-1/+1
| | | | | | | | | This patch fixes the error: incompatible types when assigning to type ‘struct ktermios *’ from type ‘struct ktermios’ Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: adds dgap driver to stagingScott_Kilau@digi.com2013-08-2225-0/+14712
This patch adds the dgap driver to staging. This is a TTY serial port driver for the EPCA PCI based product line by Digi International <www.digi.com>. Signed-off-by: Lidza Louina <lidza.louina@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
OpenPOWER on IntegriCloud