Subject: Re: [PATCH 1/1 V4] net: Add Aeroflex Gaisler 10/100/1G Ethernet MAC driver

From: Stephen Hemminger <[email-masked]>
Date: Mon, 25 Jan 2010 08:36:10 -0800

On Mon, 25 Jan 2010 12:01:12 +0100
Kristoffer Glembo <[email-masked]> wrote:

> +
> +static struct net_device_stats *greth_get_stats(struct net_device *dev)
> +{
> +	struct greth_private *greth = netdev_priv(dev);
> +	return &greth->stats;
> +}

If you used dev->stats instead of own private structure, then you
could also avoid having this handle.  The default function for
.ndo_get_stats is to return dev->stats.

--