linux-stable.git/drivers/staging/pi433, branch linux-5.0.y

staging: pi433: add missing call to cdev_del()

2018-12-05T08:39:45+00:00

If cdev_add() fails, cdev_del() should be called.
Add the missing cdev_del() call as pointed out by
Dan Carpenter.

Signed-off-by: Michael Straube 
Signed-off-by: Greg Kroah-Hartman

staging: pi433: fix potential null dereference

2018-11-26T19:59:05+00:00

Add a test for successful call to cdev_alloc() to avoid
potential null dereference. Issue reported by smatch.

Signed-off-by: Michael Straube 
Fixes: 874bcba65f9a ("staging: pi433: New driver")
Signed-off-by: Greg Kroah-Hartman

staging: pi433: remove GPL boiler plate text

2018-11-20T09:53:54+00:00

The SPDX identifier is a legally binding shorthand,
which can be used instead of the full boiler plate text.

All source and header files have SPDX identifiers.
Remove the GPL boiler plate text to reduce file size.

Signed-off-by: Michael Straube 
Signed-off-by: Greg Kroah-Hartman

staging: pi433: eliminate a temporary variable

2018-08-27T10:07:56+00:00

We can just get rid of the "spi_address".  It's not clear what the
"address" part of the name means, and the type should have been a u8.

Signed-off-by: Dan Carpenter 
Signed-off-by: Greg Kroah-Hartman

staging: pi433: fix error return code in pi433_probe()

2018-07-12T19:49:24+00:00

Fix to return a negative error code from the kthread_run() error
handling case instead of 0, as done elsewhere in this function.

Signed-off-by: Wei Yongjun 
Signed-off-by: Greg Kroah-Hartman

staging: pi433: Make only one statement per line

2018-07-11T11:12:34+00:00

Lines containing multiple statements were broken into multiple
lines, increasing readability and complying with the coding standard.
This also fixes several checkpatch.pl errors complaining about
the lines being too long.

Signed-off-by: Sophie Matter 
Signed-off-by: Rico Schrage 
Signed-off-by: Greg Kroah-Hartman

staging: pi433: Use preferred commenting style

2018-07-11T11:12:34+00:00

For multi-line comments, the preferred commenting style from the
coding style Documentation was applied to the comments, meaning
almost blank lines at the beginning and end of the comment.
One changed comment includes a line over 80 characters, causing
checkpatch.pl to complain, however breaking this line would not
make much sense, so it is kept like it is.

Signed-off-by: Sophie Matter 
Signed-off-by: Rico Schrage 
Signed-off-by: Greg Kroah-Hartman

staging: pi433: Comply with 80 character column limit

2018-07-11T11:12:33+00:00

Lines have been split where it makes sense to shorten them in order
to comply with the coding standards and fix checkpatch.pl warnings.
There are still lines left that are too long, however breaking
those would impair readability.

Changes in v2:
 - now working on staging-next branch of the staging tree
 - the changes to the defines previously made are deleted
   due to readability

Signed-off-by: Sophie Matter 
Signed-off-by: Rico Schrage 
Signed-off-by: Greg Kroah-Hartman

staging: pi433: cleanup comments in rf69.h

2018-06-28T13:00:13+00:00

Fixes checkpatch warning:

  WARNING: line over 80 characters

Signed-off-by: Valentin Vidic 
Signed-off-by: Greg Kroah-Hartman

staging: pi433: add SPDX-License-Identifier tag

2018-06-28T13:00:13+00:00

Use GPL-2.0+ based on the license text in each of the files.

Signed-off-by: Valentin Vidic 
Signed-off-by: Greg Kroah-Hartman