[dpdk-dev] [PATCH 03/18] i40e: adjustment of register definitions and relevant

Zhang, Helin helin.zhang at intel.com
Tue Apr 28 04:29:16 CEST 2015


Hi Thomas

New version is needed, as previous version has some code piece which shouldn't be present there.

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Tuesday, April 28, 2015 1:17 AM
> To: Zhang, Helin
> Cc: dev at dpdk.org; Kenguva, Monica; Murray, Steven J; Shih, Chiu-Pi
> Subject: Re: [dpdk-dev] [PATCH 03/18] i40e: adjustment of register definitions
> and relevant
> 
> 2015-04-20 16:22, Helin Zhang:
> > More register definitions and their relevant masks are added
> > publically. Also few useless macros are deleted.
> >
> > Signed-off-by: Helin Zhang <helin.zhang at intel.com>
> > ---
> >  lib/librte_pmd_i40e/i40e/i40e_register.h | 1981
> > +++++++++++++++++++++++++++++-
> 
> All these values are not used in the driver, right?
Driver read/write some of registers defined in this header file. All registers and their
relevant can be used by drivers, as these registers are treated as public registers.
Of cause there are registers which is for internal use only.

> 
> > --- a/lib/librte_pmd_i40e/i40e_ethdev.c
> > +++ b/lib/librte_pmd_i40e/i40e_ethdev.c
> > @@ -1269,9 +1269,6 @@ i40e_dev_stats_get(struct rte_eth_dev *dev,
> struct rte_eth_stats *stats)
> >  			    I40E_GLPRT_BPTCL(hw->port),
> >  			    pf->offset_loaded, &os->eth.tx_broadcast,
> >  			    &ns->eth.tx_broadcast);
> > -	i40e_stat_update_32(hw, I40E_GLPRT_TDPC(hw->port),
> > -			    pf->offset_loaded, &os->eth.tx_discards,
> > -			    &ns->eth.tx_discards);
> 
> The relation between this deletion and the rest of the patch is weird.
All the registers defined in i40e_registers.h are public, though may not already been
used in driver. It is not so weird that one of the deleted registers was being used in driver.

Regards,
Helin


More information about the dev mailing list