linux.git/drivers/staging/rtl8712, branch v4.10

Staging:rtl8712: Fixed coding style issue

2016-12-06T09:18:07+00:00

Fixes following checkpatch.pl warning
	Block comments should align the * on each line

Signed-off-by: Harsh Jain 
Signed-off-by: Greg Kroah-Hartman

Staging: rtl8712: rtl871x_*: Removed unnecessary else statements after a break or return

2016-11-21T10:25:41+00:00

The indent code blocks of the else statements were unnecessary
and are better written without them.

Signed-off-by: Shiva Kerdel 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8712: Fix coding style warnings on Block comments

2016-11-21T10:15:18+00:00

Fixed checkpatch.pl warnings related to Block comments in
staging/rtl8712/*.c files.

Signed-off-by: Vijai Kumar K 
Signed-off-by: Greg Kroah-Hartman

Staging: rtl8712: Fixed 'tabstop' coding style warnings.

2016-11-17T15:32:33+00:00

Fixed checkpatch warning "Statements should start on a tabstop" in
rtl8712 module.

Signed-off-by: Yamanappagouda Patil 
Signed-off-by: Greg Kroah-Hartman

staging : rtl8712: Free memory when kmalloc fails

2016-11-07T10:24:52+00:00

There are few functions where we need to free previously allocated
memory when kmalloc fails. Else it may lead to memory leakage. In
_init_cmd_priv() and _r8712_init_xmit_priv(), in few places we are
not freeing previously allocated memory when kmalloc fails.

Signed-off-by: Souptick joarder 
Signed-off-by: Greg Kroah-Hartman

Staging: rtl8712: remove unused functions

2016-10-16T08:24:51+00:00

Remove the functions sleep_schedulable, down_scanned_network,
up_scanned_network and get_free_xmit_queue as they are not used anywhere
in the kernel. Functions detected using coccinelle but changes done by
hand.
Script:

@initialize:python@
@@
def display(name,p):
        print(name,p[0].file)

@r1@
identifier func;
type T;
position p;
@@
static T func@p(...)
{
...
}

@r@
identifier r1.func;
@@
func

@script:python depends on !r@
func << r1.func;
p << r1.p;
@@
display(func,p)

Signed-off-by: Bhumika Goyal 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8712: Change _LED_STATE enum in rtl871x driver to avoid conflicts with LED namespace

2016-09-28T09:36:45+00:00

Adding led support for phy causes namespace conflicts for some
phy drivers.

The rtl871 driver declared an enum for representing LED states. The enum
contains constant LED_OFF which conflicted with declaration found in
linux/leds.h. LED_OFF changed to LED_STATE_OFF
In order to avoid a possible future collision LED_ON was changed to
LED_STATE_ON as well.

Signed-off-by: Zach Brown 
Signed-off-by: Greg Kroah-Hartman

Fixes: 3d44a78f0d8b ("staging: rtl8712: Remove unnecessary 'else'")

2016-09-28T06:10:36+00:00

An "unnecessary" 'else' was removed due to complains from checkpatch.pl
as it is preceded by a 'return', however the 'else' branch is necessary
as an earlier branch of the 'if' falls through. By removing the 'else',
that route now hits the 'break' and the 'while' loop exits prematurely.

This commit reverts that change and puts the original 'else' back in
place.

Signed-off-by: Matthew Kilgore 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8712: Remove unnecessary 'else'

2016-09-27T11:08:05+00:00

Remove unnecessary 'else' after 'return' to resolve checkpatch issue.

Signed-off-by: Namrata A Shettar 
Signed-off-by: Greg Kroah-Hartman

staging: rtl8172: usb_intf.c: Fixed block comment coding style

2016-09-26T20:22:13+00:00

Fixed a block comment indentation in the rtl8712 usb_intf.c file.

Signed-off-by: Gonçalo Salazar 
Signed-off-by: Greg Kroah-Hartman