[dpdk-dev] [PATCH v3 1/2] ethdev: add callback to get register size in bytes
Thomas Monjalon
thomas.monjalon at 6wind.com
Fri Jun 17 12:20:18 CEST 2016
2016-06-13 16:51, Remy Horton:
>
> On 12/06/2016 15:51, Zyta Szpak wrote:
> > I would prefer having only one function rte_eth_dev_get_regs()
> > which returns length and width if data is NULL.
> > The first call is a parameter request before buffer allocation,
> > and the second call fills the buffer.
> >
> > We can deprecate the old API and introduce this new one.
> >
> > Opinions?
> >
> > In my opinion as it is now it works fine. Gathering all parameters in
> > one callback might be a good idea if the maintainer also agrees to that
> > because as I mentioned, it interferes.
>
> From my perspective changing rte_eth_dev_get_regs() isn't a problem, as
> it isn't used directly rather than through rte_ethtool_get_regs()..
Zyta, would you like to make a v4?
More information about the dev
mailing list