linux.git/drivers/staging/crystalhd, branch v3.14

staging: delete non-required instances of include

2013-12-17T18:08:14+00:00

None of these files are actually using any __init type directives
and hence don't need to include .  Most are just a
left over from __devinit and __cpuinit removal, or simply due to
code getting copied from one driver to the next.

Signed-off-by: Paul Gortmaker 
Signed-off-by: Greg Kroah-Hartman

staging: crystalhd: Fix typo in crystalhd

2013-12-06T21:04:56+00:00

Correct spelling typo.

Signed-off-by: Masanari Iida 
Acked-by: Randy Dunlap 
Signed-off-by: Greg Kroah-Hartman

staging: remove DEFINE_PCI_DEVICE_TABLE macro

2013-12-03T06:32:35+00:00

Don't use DEFINE_PCI_DEVICE_TABLE macro, because this macro
is not preferred.

Signed-off-by: Jingoo Han 
Signed-off-by: Greg Kroah-Hartman

staging: crystalhd: return true and false instead of 1 and 0

2013-11-10T19:19:58+00:00

This fixes coccinelle error regarding functions that return
bool and return 1 and 0 instead of true and false.

Signed-off-by: Valentina Manea 
Reviewed-by: Lisa Nguyen 
Reviewed-by: Josh Triplett 
Signed-off-by: Greg Kroah-Hartman

Staging: crystalhd: use vfree() instead of kfree()

2013-10-11T22:47:06+00:00

Use vfree() instead of kfree() to free vmalloc()
allocated data.

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

Staging: crystalhd: Fix sparse warnings regarding static functions

2013-10-08T01:39:32+00:00

This fixes the following sparse warnings:
* drivers/staging/crystalhd/crystalhd_lnx.c:78:29: warning:
symbol 'chd_dec_alloc_iodata' was not declared. Should it be static?
* drivers/staging/crystalhd/crystalhd_lnx.c:99:6: warning:
symbol 'chd_dec_free_iodata' was not declared. Should it be static?
* drivers/staging/crystalhd/crystalhd_lnx.c:630:5: warning:
symbol 'chd_dec_pci_suspend' was not declared. Should it be static?
* drivers/staging/crystalhd/crystalhd_lnx.c:664:5: warning:
symbol 'chd_dec_pci_resume' was not declared. Should it be static?

Signed-off-by: Valentina Manea 
Reviewed-by: Josh Triplett 
Signed-off-by: Greg Kroah-Hartman

Staging: crystalhd: Fix assignment of 0/1 to bool variables

2013-09-25T23:48:45+00:00

Convert 0 to false and 1 to true when assigning values to bool
variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.

The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):

@@
bool b;
@@
(
-b = 0
+b = false
|
-b = 1
+b = true
)

Signed-off-by: Peter Senna Tschudin 
Signed-off-by: Greg Kroah-Hartman

staging: crystalhd: Resolve sparse 'different base types' warnings.

2013-08-30T18:46:46+00:00

The result from crystalhd_get_sgle_paddr and crystalhd_get_sgle_len are later
used in calculations, so the result should be in CPU byte ordering.

Signed-off-by: Shaun Laing 
Signed-off-by: Greg Kroah-Hartman

Staging: crystalhd: fixed whitespace and string style issues

2013-08-28T22:28:43+00:00

Fixed 3 instances of user-visible string being broken into two string.
Fixed 2 instances of illegal whitespace.

Signed-off-by: Greg Kroah-Hartman

Staging: crystalhd: remove an unneeded NULL check

2013-06-03T19:26:18+00:00

We already established earlier in the function that "temp" is
non-NULL.  We also don't need to set to NULL because it's a stack
variable an we return immediately.

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