On 6/7/2016 5:40 PM, Jerin Jacob wrote: > Signed-off-by: Jerin Jacob <jerin.jacob at caviumnetworks.com> > Signed-off-by: Maciej Czekaj <maciej.czekaj at caviumnetworks.com> > Signed-off-by: Kamil Rytarowski <Kamil.Rytarowski at caviumnetworks.com> > Signed-off-by: Zyta Szpak <zyta.szpak at semihalf.com> > Signed-off-by: Slawomir Rosek <slawomir.rosek at semihalf.com> > Signed-off-by: Radoslaw Biernacki <rad at semihalf.com> > --- > drivers/net/thunderx/Makefile | 2 + > drivers/net/thunderx/nicvf_ethdev.c | 5 +- > drivers/net/thunderx/nicvf_rxtx.c | 256 > ++++++++++++++++++++++++++++++++++++ > drivers/net/thunderx/nicvf_rxtx.h | 93 +++++++++++++ > 4 files changed, 355 insertions(+), 1 deletion(-) > create mode 100644 drivers/net/thunderx/nicvf_rxtx.c > create mode 100644 drivers/net/thunderx/nicvf_rxtx.h
Patch is generating following checkpatch warnings: CHECK:BRACES: Blank lines aren't necessary after an open brace '{' #234: FILE: drivers/net/thunderx/nicvf_rxtx.c:154: + sq->xmit_bufs > sq->tx_free_thresh) { + ERROR:CODE_INDENT: code indent should use tabs where possible #421: FILE: drivers/net/thunderx/nicvf_rxtx.h:79: + entry->buff[0] = (uint64_t)SQ_DESC_TYPE_GATHER << 60 |$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #421: FILE: drivers/net/thunderx/nicvf_rxtx.h:79: + entry->buff[0] = (uint64_t)SQ_DESC_TYPE_GATHER << 60 |$ ERROR:CODE_INDENT: code indent should use tabs where possible #424: FILE: drivers/net/thunderx/nicvf_rxtx.h:82: + entry->buff[1] = rte_mbuf_data_dma_addr(pkt);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #424: FILE: drivers/net/thunderx/nicvf_rxtx.h:82: + entry->buff[1] = rte_mbuf_data_dma_addr(pkt);$