Re: [PATCH net-next] drivers/net: enic: Move ethtool code to a separate file

[Joe Perches <joe@perches.com>]

On Fri, 2013-07-19 at 13:28 -0700, Neel Patel wrote:
> From: Neel Patel <neepatel@cisco.com>

Hi Neel.

> This patch moves all enic ethtool hooks from enic_main.c to a new file
> enic_ethtool.c

These comments aren't for new issues and are trivial.

> diff --git a/drivers/net/ethernet/cisco/enic/enic_ethtool.c b/drivers/net/ethernet/cisco/enic/enic_ethtool.c
[]
> +struct enic_stat {
> +	char name[ETH_GSTRING_LEN];
> +	unsigned int offset;
> +};

Offset is really an index, so use index

> +#define ENIC_TX_STAT(stat)	\
> +	{ .name = #stat, .offset = offsetof(struct vnic_tx_stats, stat) / 8 }
> +#define ENIC_RX_STAT(stat)	\
> +	{ .name = #stat, .offset = offsetof(struct vnic_rx_stats, stat) / 8 }

index = offsetof(struct, stat) / sizeof(struct.stat)
to avoid use of 8?

> +static void enic_get_ethtool_stats(struct net_device *netdev,
> +	struct ethtool_stats *stats, u64 *data)
> +{
[]
> +	for (i = 0; i < enic_n_tx_stats; i++)
> +		*(data++) = ((u64 *)&vstats->tx)[enic_tx_stats[i].offset];
> +	for (i = 0; i < enic_n_rx_stats; i++)
> +		*(data++) = ((u64 *)&vstats->rx)[enic_rx_stats[i].offset];

s/offset/index/ ?