linux/drivers/staging/dgap, branch v3.13

staging: dgap: dgap_tty: Do not use 0 for pointers

2013-10-11T22:36:30Z

0 should not be used instead of NULL for pointers. Signed-off-by: Sachin Kamat Cc: Lidza Louina Signed-off-by: Greg Kroah-Hartman

staging: dgap: dgap_fep5: Do not use 0 for NULL pointer

2013-10-11T22:36:30Z

Do not compare NULL pointer with 0. Signed-off-by: Sachin Kamat Cc: Lidza Louina Signed-off-by: Greg Kroah-Hartman

staging: dgap: dgap_fep5: Remove braces around return

2013-10-11T22:36:30Z

Braces are not needed around return values. Signed-off-by: Sachin Kamat Cc: Lidza Louina Signed-off-by: Greg Kroah-Hartman

staging: dgap: dgap_fep5: Remove braces around single line statements

2013-10-11T22:36:30Z

Single line statements do not require braces. Signed-off-by: Sachin Kamat Cc: Lidza Louina Signed-off-by: Greg Kroah-Hartman

staging: dgap: tty.c: removes smatch warnings "redundant null check"

2013-09-25T23:18:10Z

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 Signed-off-by: Greg Kroah-Hartman

staging: dgap: tty.c: removes smatch warning "ignoring unreachable code"

2013-09-25T23:18:10Z

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 Signed-off-by: Greg Kroah-Hartman

staging: dgap: driver.c: removes smatch warning "redundant null check"

2013-09-25T23:09:24Z

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 Signed-off-by: Greg Kroah-Hartman

drivers: staging: dgap: move DG_NAME and DG_PART from "Makefile" to "dgap_driver.h"

2013-09-25T23:09:24Z

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 Signed-off-by: Greg Kroah-Hartman

Merge 3.12-rc2 into staging-next.

2013-09-25T15:59:04Z

This resolves the merge problem with two iio drivers that Stephen Rothwell pointed out. Reported-by: Stephen Rothwell Signed-off-by: Greg Kroah-Hartman

staging: dgap: Fix typo in staging/dgap

2013-09-17T14:47:46Z

Correct spelling typo in comments. Signed-off-by: Masanari Iida Signed-off-by: Greg Kroah-Hartman