linux-dev/drivers/staging/rtl8723au/hal, branch master

staging: Remove rtl8723au driver

2016-09-10T13:21:43Z

This driver is superseded by rtl8xxxu and has been marked as scheduled for deletion since 4.6 Signed-off-by: Jes Sorensen Signed-off-by: Greg Kroah-Hartman

drivers: staging: rtl8723au: hal: Remove pointless test

2016-09-01T15:44:01Z

This patch removes the pointless `else if` test. Signed-off-by: Matthias Beyer Reported-by: David Binderman Signed-off-by: Greg Kroah-Hartman

staging: rtl8723au: hal: check BT_Active and BT_State with correct bit pattern

2016-08-21T16:35:46Z

BT_Active and BT_State are being masked with 0x00ffffff so it the subsequent comparisons with 0xffffffff are therefore a buggy check. Instead, check them against 0x00ffffff. Unfortunately I couldn't find a datasheet or hardware to see if 0xffffffff is an expected invalid bit pattern that should be checked before BT_Active and BT_State are masked with 0x00ffffff, so for now, this fix seems like the least risky approach. Signed-off-by: Colin Ian King Signed-off-by: Greg Kroah-Hartman

Staging: rtl8723au: Remove unnecessary return statement.

2016-03-28T14:30:36Z

Remove unnecessary return statement. Signed-off-by: Sandhya Bankar Signed-off-by: Greg Kroah-Hartman

Staging: rtl8723au: rtl8723a_rf6052: Remove unnecessary semicolon.

2016-03-28T14:30:36Z

Remove unnecessary semicolon. Coccinelle sementic patch as follows: @r_case@ position p; @@ switch (...) { case ...: ...;@p } @r_default@ position p; @@ switch (...) { default: ...;@p } @r1@ statement S; position p1; position p != {r_case.p, r_default.p}; identifier label; @@ ( label:; | S@p1;@p ) Signed-off-by: Sandhya Bankar Signed-off-by: Greg Kroah-Hartman

taging: rtl8723au: Remove empty function odm_Init_RSSIForDM23a

2016-03-12T06:09:09Z

The function odm_Init_RSSIForDM23a is empty and therefore there is no use of keeping it. Also, remove its prototype. Signed-off-by: Bhumika Goyal Signed-off-by: Greg Kroah-Hartman

staging: rtl8723au: hal: Remove unnecessary function and its call

2016-03-12T06:09:09Z

The function odm_DynamicBBPowerSaving23a on being called, simply returns back. The function hasn't been mentioned in the TODO and doesn't have FIXME code around. Hence, odm_DynamicBBPowerSaving23a and its calls have been removed. This was done using Coccinelle. @@ identifier f; @@ void f(...) { -return; } Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman

staging: rtl8723au: hal: Use macro DIV_ROUND_UP

2016-03-12T06:09:09Z

The macro DIV_ROUND_UP performs the computation (((n) + (d) - 1) /(d)). It clarifies the divisor calculations. This was done using the coccinelle script: @@ expression e1; expression e2; @@ ( - ((e1) + e2 - 1) / (e2) + DIV_ROUND_UP(e1,e2) | - ((e1) + (e2 - 1)) / (e2) + DIV_ROUND_UP(e1,e2) ) Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman

staging: rtl8723au: hal: Remove Unused macro

2016-03-12T06:09:09Z

PlatformIndicateBTACLData is an unused macro. Hence, removed. Signed-off-by: Bhaktipriya Shridhar Signed-off-by: Greg Kroah-Hartman

Staging: rtl8723au: Remove print statements and debug messages

2016-03-12T06:09:09Z

The memory allocation functions generates a call stack containing all the context information on failure, so print statements and debug messages can be removed on failure of these functions. Also remove unwanted {} around if block after removal of these messages. Done using coccinelle: @@ expression e; @@ e=\(kmalloc\|kmalloc\|kmalloc_array\|alloc_netdev\|kzalloc\| devm_kzalloc\|devm_ioremap\)(...); ... if(!e){ - \(DBG_8723A\|printk\|pr_err\|CERROR\|DBG_88E\)(...); ...when any } Signed-off-by: Bhumika Goyal Signed-off-by: Greg Kroah-Hartman