On Fri, 20 Apr 2018 16:15:40 +0530
Sunil Kumar Kori <sunil.k...@nxp.com> wrote:

>       dpaa_eth_dev_info(dev, &dev_info);
>       if (((~(dev_info.rx_offload_capa) & rx_offloads) != 0)) {
> -             DPAA_PMD_ERR("Some Rx offloads are not supported "
> +             DPAA_PMD_WARN("Some Rx offloads are not supported "
>                       "requested 0x%" PRIx64 " supported 0x%" PRIx64,

The if statement has more parens than necessary.

Please don't break error messages across lines, it is better to have a long 
source
line since it allows users to use tools like grep to search for error messages.

Reply via email to