From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sunset.davemloft.net (unknown [74.93.104.97]) by ozlabs.org (Postfix) with ESMTP id DD65DDDEF0 for ; Wed, 25 Mar 2009 10:29:26 +1100 (EST) Date: Tue, 24 Mar 2009 16:29:13 -0700 (PDT) Message-Id: <20090324.162913.226453070.davem@davemloft.net> To: Joakim.Tjernlund@transmode.se Subject: Re: [PATCH] ucc_geth: Convert to net_device_ops From: David Miller In-Reply-To: References: <20090324.154902.206324978.davem@davemloft.net> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Cc: linuxppc-dev@ozlabs.org, leoli@freescale.com, netdev@vger.kernel.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Joakim Tjernlund Date: Wed, 25 Mar 2009 00:08:53 +0100 > David Miller wrote on 24/03/2009 23:49:02: > > > > From: Joakim Tjernlund > > Date: Tue, 24 Mar 2009 23:45:13 +0100 > > > > > I don't follow. Are these mandatory now? They were not in the old > > > impl. either. > > > > See ether_setup() which is called indirectly via alloc_etherdev(). > > > > Yawn... > > Same here, getting tiered. Should have seen that. Here goes, > attatched as well as explained earlier. I'll apply this, thanks. Please provide a proper "Signed-off-by:" line with future patch submissions, thanks. From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH] ucc_geth: Convert to net_device_ops Date: Tue, 24 Mar 2009 16:29:13 -0700 (PDT) Message-ID: <20090324.162913.226453070.davem@davemloft.net> References: <20090324.154902.206324978.davem@davemloft.net> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: avorontsov@ru.mvista.com, leoli@freescale.com, linuxppc-dev@ozlabs.org, netdev@vger.kernel.org To: Joakim.Tjernlund@transmode.se Return-path: Received: from 74-93-104-97-Washington.hfc.comcastbusiness.net ([74.93.104.97]:35143 "EHLO sunset.davemloft.net" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1753837AbZCXX30 (ORCPT ); Tue, 24 Mar 2009 19:29:26 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: From: Joakim Tjernlund Date: Wed, 25 Mar 2009 00:08:53 +0100 > David Miller wrote on 24/03/2009 23:49:02: > > > > From: Joakim Tjernlund > > Date: Tue, 24 Mar 2009 23:45:13 +0100 > > > > > I don't follow. Are these mandatory now? They were not in the old > > > impl. either. > > > > See ether_setup() which is called indirectly via alloc_etherdev(). > > > > Yawn... > > Same here, getting tiered. Should have seen that. Here goes, > attatched as well as explained earlier. I'll apply this, thanks. Please provide a proper "Signed-off-by:" line with future patch submissions, thanks.