From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Grundler Subject: Re: [PATCH 25/42] de2104x: convert to net_device_ops Date: Wed, 7 Jan 2009 00:50:29 -0700 Message-ID: <20090107075029.GC23818@colo.lackof.org> References: <20090107003316.784424362@vyatta.com> <20090107003348.195639787@vyatta.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: David Miller , Grant Grundler , Kyle McMartin , netdev@vger.kernel.org To: Stephen Hemminger Return-path: Received: from colo.lackof.org ([198.49.126.79]:60830 "EHLO colo.lackof.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751618AbZAGHuk (ORCPT ); Wed, 7 Jan 2009 02:50:40 -0500 Content-Disposition: inline In-Reply-To: <20090107003348.195639787@vyatta.com> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, Jan 06, 2009 at 04:33:41PM -0800, Stephen Hemminger wrote: > Signed-off-by: Stephen Hemminger Acked-by: Grant Grundler thanks, grant > > --- a/drivers/net/tulip/de2104x.c 2009-01-06 16:08:31.161363983 -0800 > +++ b/drivers/net/tulip/de2104x.c 2009-01-06 16:14:57.956742918 -0800 > @@ -1922,6 +1922,18 @@ bad_srom: > goto fill_defaults; > } > > +static const struct net_device_ops de_netdev_ops = { > + .ndo_open = de_open, > + .ndo_stop = de_close, > + .ndo_set_multicast_list = de_set_rx_mode, > + .ndo_start_xmit = de_start_xmit, > + .ndo_get_stats = de_get_stats, > + .ndo_tx_timeout = de_tx_timeout, > + .ndo_change_mtu = eth_change_mtu, > + .ndo_set_mac_address = eth_mac_addr, > + .ndo_validate_addr = eth_validate_addr, > +}; > + > static int __devinit de_init_one (struct pci_dev *pdev, > const struct pci_device_id *ent) > { > @@ -1944,14 +1956,9 @@ static int __devinit de_init_one (struct > if (!dev) > return -ENOMEM; > > + dev->netdev_ops = &de_netdev_ops; > SET_NETDEV_DEV(dev, &pdev->dev); > - dev->open = de_open; > - dev->stop = de_close; > - dev->set_multicast_list = de_set_rx_mode; > - dev->hard_start_xmit = de_start_xmit; > - dev->get_stats = de_get_stats; > dev->ethtool_ops = &de_ethtool_ops; > - dev->tx_timeout = de_tx_timeout; > dev->watchdog_timeo = TX_TIMEOUT; > > de = netdev_priv(dev); > > --