Macro includes commented out code. Removing dead code line enables braces to be removed. Macro is easier to read if the code is clean. Clean up macro ps_confirm_wait_inc. Signed-off-by: Tobin C. Harding <me@xxxxxxxx> --- drivers/staging/ks7010/ks_hostif.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index 4d6bc4f..4561687 100644 --- a/drivers/staging/ks7010/ks_hostif.c +++ b/drivers/staging/ks7010/ks_hostif.c @@ -1283,11 +1283,11 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *skb) return ret; } -#define ps_confirm_wait_inc(priv) do { \ - if (atomic_read(&priv->psstatus.status) > PS_ACTIVE_SET) { \ - atomic_inc(&priv->psstatus.confirm_wait); \ - /* atomic_set(&priv->psstatus.status, PS_CONF_WAIT);*/ \ - } } while (0) +#define ps_confirm_wait_inc(priv) \ + do { \ + if (atomic_read(&priv->psstatus.status) > PS_ACTIVE_SET) \ + atomic_inc(&priv->psstatus.confirm_wait); \ + } while (0) static void hostif_mib_get_request(struct ks_wlan_private *priv, -- 2.7.4 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel