[PATCH 14/14 V2] rtlwifi: Remove some redundant code

Kalle Valo kvalo at codeaurora.org
Fri Dec 30 13:53:03 UTC 2016


Larry Finger <Larry.Finger at lwfinger.net> writes:

> The symbol DBG_EMERG is no longer used and is removed.
>
> In a number of places, the code has redundant messages. For example, if
> the failure for the firmware to run is logged, it is not necessary to
> log that the firmware has been started. In addition, extraneous braces are
> removed.
>
> Signed-off-by: Larry Finger <Larry.Finger at lwfinger.net>
> Cc: Ping-Ke Shih <pkshih at realtek.com>

[...]

> --- wireless-drivers-next.orig/drivers/net/wireless/realtek/rtlwifi/debug.h
> +++ wireless-drivers-next/drivers/net/wireless/realtek/rtlwifi/debug.h
> @@ -36,7 +36,7 @@
>   *unexpected HW behavior, HW BUG
>   *and so on.
>   */
> -#define DBG_EMERG			0
> +/*#define DBG_EMERG			0 */

You are not really removing, just commenting it out :) But this is good
enough for me, if you really want to remove it submit a followup patch.

-- 
Kalle Valo


More information about the devel mailing list