[dpdk-dev] [PATCH v2 03/16] fm10k: Add a new func to initialize all parameters

[Stephen Hemminger]

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