2017-03-01 13:23, Legacy, Allain: > > -----Original Message----- > > From: Bruce Richardson [mailto:bruce.richard...@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", >
There is maybe something to fix in the checkpatches.sh options. Please could you look at SPLIT_STRING and LONG_LINE_STRING?
