Hi All, I am really sorry I made a mistake by submitting patches from a wrong branch and tried to rescue quickly and made another mistake(submitted one more patch not mine).
Please skip these wrong patches(I already superseded them) and help review the new 3 patches for rte ring. Really sorry again for the confusion and inconveniences! Best Regards, Gavin > -----Original Message----- > From: Gavin Hu <[email protected]> > Sent: Monday, September 17, 2018 3:48 PM > To: [email protected] > Cc: Gavin Hu (Arm Technology China) <[email protected]>; Honnappa > Nagarahalli <[email protected]>; Steve Capper > <[email protected]>; Ola Liljedahl <[email protected]>; > [email protected]; nd <[email protected]>; Ferruh Yigit > <[email protected]> > Subject: [PATCH v3 2/3] net/i40e: remove invalid comment > > From: Ferruh Yigit <[email protected]> > > Comments says "no csum error report support" but there is no check related > csum offloads. Removing the comment. > > Signed-off-by: Ferruh Yigit <[email protected]> > Acked-by: Qi Zhang <[email protected]> > --- > drivers/net/i40e/i40e_rxtx_vec_common.h | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/net/i40e/i40e_rxtx_vec_common.h > b/drivers/net/i40e/i40e_rxtx_vec_common.h > index 63cb17742..f00f6d648 100644 > --- a/drivers/net/i40e/i40e_rxtx_vec_common.h > +++ b/drivers/net/i40e/i40e_rxtx_vec_common.h > @@ -199,9 +199,7 @@ > i40e_rx_vec_dev_conf_condition_check_default(struct rte_eth_dev *dev) > if (fconf->mode != RTE_FDIR_MODE_NONE) > return -1; > > - /* - no csum error report support > - * - no header split support > - */ > + /* no header split support */ > if (rxmode->offloads & DEV_RX_OFFLOAD_HEADER_SPLIT) > return -1; > > -- > 2.11.0
