linux.git/drivers/staging/bcm/Misc.c, branch v3.2

Staging: bcm: Fix assignment issue in if statement reported by checkpatch.pl.

2011-10-11T16:18:21+00:00

This patch fixes an error where an assignment "="
was being used in an if statement to determine if
Firmware was downloaded. This patch removes that
assignment, and places it above the if statement.
The if statement then evaluates the status to
verify if "0" successful, or != 0 failure.

Signed-off-by: Kevin McKinney 
Signed-off-by: Greg Kroah-Hartman

Staging: bcm: Remove assignment in if condition reported by checkpatch.pl

2011-10-11T16:18:21+00:00

There is a case where an assignment was being done
in an if condition. This patch removes the assignment
from the if condition and places the assignment above
the if statement; thereby improving the readability of
the code.

Signed-off-by: Kevin McKinney 
Signed-off-by: Greg Kroah-Hartman

Staging: bcm: Replace dated variable FUNCTION reported by checkpatch.pl

2011-10-11T16:18:21+00:00

This patch replaces the obsolete variable, __FUNCTION__, that
holds the name of the current function with variable, __func__.

Signed-off-by: Kevin McKinney 
Signed-off-by: Greg Kroah-Hartman

Staging: bcm: Fix coding style issues reported by checkpatch.pl

2011-10-11T16:18:20+00:00

This patch cleans up several hundred code style issues found
in Misc.c reported by checkpatch.pl.

Signed-off-by: Kevin McKinney 
Signed-off-by: Greg Kroah-Hartman

Staging: bcm: Add size maximum size restrictions for IOCTL_IDLE_REQ

2011-09-19T17:46:17+00:00

In the first alteration, the MAX_CNTL_PKT_SIZE is the
maximum size of the control packet in ->Adapter->txctlpacket[]
which is defined in InitAdapter(). This caps the size of
kmalloc memory allocation. In the second change, this max
cap fixes a potential memory corruption bug when subsequent
memset and memcpy calls are invoked.

Signed-off-by: Kevin McKinney 
Signed-off-by: Greg Kroah-Hartman

staging: remove unnecessary code

2011-06-28T20:13:42+00:00

Compile tested.
remove unnecessary code that matches this coccinelle pattern
	if (...)
		return ret;
	return ret;

Signed-off-by: Greg Dietsche 
Acked-by: Franky Lin 
Signed-off-by: Greg Kroah-Hartman

Fix common misspellings

2011-03-31T14:26:23+00:00

Fixes generated by 'codespell' and manually reviewed.

Signed-off-by: Lucas De Marchi

staging: bcm: optimize kmalloc to kzalloc

2011-03-09T23:54:31+00:00

Use kzalloc rather than kmalloc followed by memset with 0.
Found by coccinelle.

Signed-off-by: Alexander Beregalov 
Signed-off-by: Greg Kroah-Hartman

Staging: Beceem: remove old kernel compatibility code

2010-12-07T00:09:45+00:00

This drops compatability for everything from 2.4 to 2.6.35.  Now it only
works on the latest kernel.

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

Staging: bcm: signedness bug in InitCardAndDownloadFirmware()

2010-11-16T20:31:50+00:00

status is used to store negative error codes throughout.  The only place
where this is a runtime bug is if create_worker_threads() fails.

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

linux.git/drivers/staging/bcm/Misc.c, branch v3.2

Staging: bcm: Fix assignment issue in if statement reported by checkpatch.pl.

Staging: bcm: Remove assignment in if condition reported by checkpatch.pl

Staging: bcm: Replace dated variable __FUNCTION__ reported by checkpatch.pl

Staging: bcm: Fix coding style issues reported by checkpatch.pl

Staging: bcm: Add size maximum size restrictions for IOCTL_IDLE_REQ

staging: remove unnecessary code

Fix common misspellings

staging: bcm: optimize kmalloc to kzalloc

Staging: Beceem: remove old kernel compatibility code

Staging: bcm: signedness bug in InitCardAndDownloadFirmware()

Staging: bcm: Replace dated variable FUNCTION reported by checkpatch.pl