[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