[dpdk-dev] [PATCH v2 11/15] net/avp: packet receive functions

Legacy, Allain Allain.Legacy at windriver.com
Wed Mar 1 14:23:12 CET 2017


> -----Original Message-----
> From: Bruce Richardson [mailto:bruce.richardson at intel.com]
> In my experience, checkpatch ignores long lines that are due to error
> messages. Perhaps you need to put the error message on a separate line,
> if other things before the message are of significant size.
I have gone through the entire patchset and reverted my previous changes to reduce line length on any occurrences of debug log strings.   I kept the error message on the first line and all input variables on subsequent lines (and no longer than 80).   checkpatches.sh flags most of them as warnings, but since unbroken strings seems more important I will submit my next patchset version (v3) like this. 

WARNING:LONG_LINE_STRING: line over 80 characters
#120: FILE: drivers/net/avp/avp_ethdev.c:236:
+			PMD_DRV_LOG(ERR, "Timeout while waiting for a response for %u\n",



More information about the dev mailing list