From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH 1/1 V4] net: Add Aeroflex Gaisler 10/100/1G Ethernet MAC driver Date: Mon, 25 Jan 2010 08:36:10 -0800 Message-ID: <20100125083610.387ba71b@nehalam> References: <1264417272-3800-1-git-send-email-kristoffer@gaisler.com> <1264417272-3800-2-git-send-email-kristoffer@gaisler.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, davem@davemloft.net, Kristoffer Glembo To: Kristoffer Glembo Return-path: Received: from mail.vyatta.com ([76.74.103.46]:34149 "EHLO mail.vyatta.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753499Ab0AYQgg (ORCPT ); Mon, 25 Jan 2010 11:36:36 -0500 In-Reply-To: <1264417272-3800-2-git-send-email-kristoffer@gaisler.com> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, 25 Jan 2010 12:01:12 +0100 Kristoffer Glembo 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. --