Hi Jerin, <snip>
> > > Subject: [dpdk-dev] [PATCH v3 4/4] app/testpmd: collect bad outer L4 > > > checksum for csum engine > > > > > > Collect and prints the statistics for PKT_RX_EL4_CKSUM_BAD errors. > > > > > > Signed-off-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> > > > Reviewed-by: Andrew Rybchenko <arybche...@solarflare.com> > > > --- > > > > <snip> > > > > The following checkpatch warnings should probably be fixed > > > > WARNING:LONG_LINE: line over 80 characters > > #86: FILE: app/test-pmd/testpmd.c:1043: > > + printf(" Bad-ipcsum: %-14"PRIu64" Bad-l4csum: > > + %-14"PRIu64"Bad-outer-l4csum: %-14"PRIu64"\n", > > > > WARNING:LONG_LINE: line over 80 characters > > #98: FILE: app/test-pmd/testpmd.c:1063: > > + printf(" Bad-ipcsum:%14"PRIu64" > > Bad-l4csum:%14"PRIu64" > Bad-outer-l4csum: %-14"PRIu64"\n", > > > > AFAIK, printf's string can be >80 to allow string strip Yes, there is flexibility, not sure how much, the above lines are > 100 chars. Otherwise Acked-by: Bernard Iremonger <bernard.iremon...@intel.com>