linux/drivers/staging/dgap, branch v4.5

staging: dgap: add spaces around binary operator '|'

2015-10-25T02:33:42Z

This patch adds spaces around binary operator '|'. Done with coccinelle semantic patch: @@ identifier x, y, z; @@ ( - x|y|z + x | y | z | - x|y + x | y ) Signed-off-by: Ioana Ciornei Signed-off-by: Greg Kroah-Hartman

staging: dgap: remove unnecessary brackets

2015-10-25T02:33:42Z

This patch removes unnecessary brackets when dealing with unary operators like '&'. Done with coccinelle semantic patch: @@ expression e; @@ ( - &(e) + &e ) Signed-off-by: Ioana Ciornei Signed-off-by: Greg Kroah-Hartman

staging: dgap: remove unnecessary space after cast

2015-10-25T02:33:42Z

This patch removes unnecessary spaces after the cast. Patch done with coccinelle semantic patch: @rule0@ type t; identifier e; constant c; expression expr; @@ ( - (t) e + (t)e | - (t) c + (t)c | - (t) expr + (t)expr ) Signed-off-by: Ioana Ciornei Signed-off-by: Greg Kroah-Hartman

staging: dgap: properly indent to match open paranthesis

2015-10-25T02:33:42Z

Indent parameters and arguments passed to function calls to match open paranthesis Signed-off-by: Ioana Ciornei Signed-off-by: Greg Kroah-Hartman

staging: dgap: remove unnecessary blank lines

2015-10-25T02:33:42Z

This patch removes the unnecessary blank lines before a closed bracket and after an open bracket. Signed-off-by: Ioana Ciornei Signed-off-by: Greg Kroah-Hartman

Staging: dgap: Remove unnecessary test expression

2015-10-17T06:06:38Z

struct bd is already NULL tested so, any further NULL checks should be removed. Signed-off-by: Shivani Bhardwaj Signed-off-by: Greg Kroah-Hartman

staging: dgap: remove unused configuration

2015-10-13T04:09:47Z

The IO configuration was not used to configure the board. It was only read from the configuration file. Stop reading it and also remove the other related variables defined for it. Signed-off-by: Sudip Mukherjee Signed-off-by: Greg Kroah-Hartman

staging: dgap: fix memory leak in dgap_parsefile()

2015-09-30T10:26:20Z

In dgap_parsefile() char pointers are set with kstrdup() without checking that some string is allocated to that char pointer before. This patch frees the memory if already allocated and then set the poniter with kstrdup(). Signed-off-by: Ronit halder Signed-off-by: Greg Kroah-Hartman

staging: dgap: fix returned errno code in dgap_parsefile()

2015-09-29T02:16:29Z

The driver is using -1 instead of the -ENOMEM defined macro to specify that a buffer allocation failed. Since the error number is propagated, the caller will get a -EPERM which is the wrong error condition. Also, the smatch tool complains with the following warning: dgap_parsefile() warn: returning -1 instead of -ENOMEM is sloppy Signed-off-by: Javier Martinez Canillas Signed-off-by: Greg Kroah-Hartman

staging/dgap: Use strpbrk() instead of dgap_sindex()

2015-09-13T01:24:57Z

The provides strpbrk() function that does the same that the dgap_sindex(). Let's use already defined function instead of writing custom. Signed-off-by: Alexander Kuleshov Signed-off-by: Greg Kroah-Hartman