Re: [PATCH 2/7] staging: rtl8723bs: Wrap multi-line macros in do-while loop

[Larry Finger]

On 04/27/2017 07:09 PM, Justin Vreeland wrote:
> Signed-off-by: Justin Vreeland <justin@xxxxxxxxxxxxx>
> ---

The patch is OK, but most maintainers require a non-blank commit message.

Larry

>  drivers/staging/rtl8723bs/hal/odm_debug.h | 80 +++++++++++++++++--------------
>  1 file changed, 44 insertions(+), 36 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/hal/odm_debug.h b/drivers/staging/rtl8723bs/hal/odm_debug.h
> index a89690ea6ba9..f720eafe46fe 100644
> --- a/drivers/staging/rtl8723bs/hal/odm_debug.h
> +++ b/drivers/staging/rtl8723bs/hal/odm_debug.h
> @@ -105,51 +105,59 @@
>
>  #if DBG
>  #define ODM_RT_TRACE(pDM_Odm, comp, level, fmt)\
> -	if (\
> -		(comp & pDM_Odm->DebugComponents) &&\
> -		(level <= pDM_Odm->DebugLevel || level == ODM_DBG_SERIOUS)\
> -	) {\
> -		RT_PRINTK fmt;\
> -	}
> +	do {\
> +		if (\
> +			(comp & pDM_Odm->DebugComponents) &&\
> +			(level <= pDM_Odm->DebugLevel || level == ODM_DBG_SERIOUS)\
> +		) {\
> +			RT_PRINTK fmt;\
> +		} \
> +	} while (0)
>
>  #define ODM_RT_TRACE_F(pDM_Odm, comp, level, fmt)\
> -	if (\
> -		(comp & pDM_Odm->DebugComponents) &&\
> -		(level <= pDM_Odm->DebugLevel)\
> -	) {\
> -		RT_PRINTK fmt;\
> -	}
> +	do {\
> +		if (\
> +			(comp & pDM_Odm->DebugComponents) &&\
> +			(level <= pDM_Odm->DebugLevel)\
> +		) {\
> +			RT_PRINTK fmt;\
> +		} \
> +	} while (0)
>
>  #define ODM_RT_ASSERT(pDM_Odm, expr, fmt)\
> -	if (!expr) {\
> -		DbgPrint("Assertion failed! %s at ......\n", #expr);\
> -		DbgPrint(\
> -			"      ......%s,%s, line =%d\n",\
> -			__FILE__,\
> -			__func__,\
> -			__LINE__\
> -		);\
> -		RT_PRINTK fmt;\
> -		ASSERT(false);\
> -	}
> +	do {\
> +		if (!expr) {\
> +			DbgPrint("Assertion failed! %s at ......\n", #expr);\
> +			DbgPrint(\
> +				"      ......%s,%s, line =%d\n",\
> +				__FILE__,\
> +				__func__,\
> +				__LINE__\
> +			);\
> +			RT_PRINTK fmt;\
> +			ASSERT(false);\
> +		} \
> +	} while (0)
>  #define ODM_dbg_enter() { DbgPrint("==> %s\n", __func__); }
>  #define ODM_dbg_exit() { DbgPrint("<== %s\n", __func__); }
>  #define ODM_dbg_trace(str) { DbgPrint("%s:%s\n", __func__, str); }
>
>  #define ODM_PRINT_ADDR(pDM_Odm, comp, level, title_str, ptr)\
> -	if (\
> -		(comp & pDM_Odm->DebugComponents) &&\
> -		(level <= pDM_Odm->DebugLevel)\
> -	) {\
> -		int __i;\
> -		u8 *__ptr = (u8 *)ptr;\
> -		DbgPrint("[ODM] ");\
> -		DbgPrint(title_str);\
> -		DbgPrint(" ");\
> -		for (__i = 0; __i < 6; __i++)\
> -			DbgPrint("%02X%s", __ptr[__i], (__i == 5) ? "" : "-");\
> -		DbgPrint("\n");\
> -	}
> +	do {\
> +		if (\
> +			(comp & pDM_Odm->DebugComponents) &&\
> +			(level <= pDM_Odm->DebugLevel)\
> +		) {\
> +			int __i;\
> +			u8 *__ptr = (u8 *)ptr;\
> +			DbgPrint("[ODM] ");\
> +			DbgPrint(title_str);\
> +			DbgPrint(" ");\
> +			for (__i = 0; __i < 6; __i++)\
> +				DbgPrint("%02X%s", __ptr[__i], (__i == 5) ? "" : "-");\
> +			DbgPrint("\n");\
> +		} \
> +	} while (0)
>  #else
>  #define ODM_RT_TRACE(pDM_Odm, comp, level, fmt)
>  #define ODM_RT_TRACE_F(pDM_Odm, comp, level, fmt)