[dpdk-dev] [PATCH v2 03/16] fm10k: Add a new func to initialize all parameters
Stephen Hemminger
stephen at networkplumber.org
Thu Oct 22 17:57:54 CEST 2015
On Thu, 22 Oct 2015 17:44:51 +0800
"Chen Jing D(Mark)" <jing.d.chen at intel.com> wrote:
> +static void
> +fm10k_params_init(struct rte_eth_dev *dev)
> +{
> + struct fm10k_hw *hw = FM10K_DEV_PRIVATE_TO_HW(dev->data->dev_private);
> + struct fm10k_dev_info *info = FM10K_DEV_PRIVATE_TO_INFO(dev);
> + /* Inialize bus info. Normally we would call fm10k_get_bus_info(), but
> + * there is no way to get link status without reading BAR4. Until this
> + * works, assume we have maximum bandwidth.
> + * @todo - fix bus info
Minor nit. I would prefer that DPDK follow current Linux kernel
style which is to always have a blank line after declarations.
This improves readability.
I.e:
static void
fm10k_params_init(struct rte_eth_dev *dev)
{
struct fm10k_hw *hw = FM10K_DEV_PRIVATE_TO_HW(dev->data->dev_private);
struct fm10k_dev_info *info = FM10K_DEV_PRIVATE_TO_INFO(dev);
/* Inialize bus info. Normally we would call fm10k_get_bus_info(), but
* there is no way to get link status without reading BAR4. Until this
* works, assume we have maximum bandwidth.
* @todo - fix bus info
More information about the dev
mailing list