On Tue, May 03, 2016 at 06:01:56AM +0300, Nikita Eshkeev wrote:
> This patch fixes the checkpatch.pl warning:
> 
> WARNING: Macros with flow control statements should be avoided
> +#define board_staging(str, fn)                       \
> +static int __init runtime_board_check(void)  \
> +{                                            \
> +     if (of_machine_is_compatible(str))      \
> +             fn();                           \
> +                                             \
> +     return 0;                               \
> +}                                            \
> +                                             \
> +device_initcall(runtime_board_check)
> +

That's a dumb warning, the code is fine, as you can see.

checkpatch isn't always correct.

sorry,

greg k-h
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to