> From: Avadhut Naik <[email protected]>
> [...]
> diff --git a/include/trace/stages/stage3_trace_output.h
> b/include/trace/stages/stage3_trace_output.h
> index c1fb1355d309..8e3215093e1f 100644
> --- a/include/trace/stages/stage3_trace_output.h
> +++ b/include/trace/stages/stage3_trace_output.h
> @@ -119,6 +119,14 @@
>               trace_print_array_seq(p, array, count, el_size);        \
>       })
> 
> +#undef __print_dynamic_array
> +#define __print_dynamic_array(array, el_size)                                
> \
> +     ({                                                              \
> +             __print_array(__get_dynamic_array(array),               \
> +                           __get_dynamic_array_len(array) / el_size, \
> +                           el_size);                                 \
> +     })
> +

When running the check below: 

    ${LINUX}/scripts/checkpatch.pl --strict  <this patch> 

it complains:

    CHECK: Macro argument 'el_size' may be better as '(el_size)' to avoid 
precedence issues
    #36: FILE: include/trace/stages/stage3_trace_output.h:123:
    +#define __print_dynamic_array(array, el_size)                          \
    +       ({                                                              \
    +               __print_array(__get_dynamic_array(array),               \
    +                             __get_dynamic_array_len(array) / el_size, \
    +                             el_size);                                 \
    +       })

-Qiuxu
> [...]

Reply via email to