Return-Path: Received: from buildserver.ru.mvista.com (unknown [85.21.88.6]) by ozlabs.org (Postfix) with ESMTP id E8B5ADDE08 for ; Sun, 21 Dec 2008 14:37:42 +1100 (EST) Date: Sun, 21 Dec 2008 06:37:41 +0300 From: Anton Vorontsov To: Timur Tabi Subject: Re: [PATCH 2/6] ucc_geth: Fix TX watchdog timeout handling Message-ID: <20081221033741.GB3885@oksana.dev.rtsoft.ru> References: <20081218182326.GB30682@oksana.dev.rtsoft.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=windows-1251 In-Reply-To: Cc: netdev@vger.kernel.org, linuxppc-dev@ozlabs.org, Andy Fleming , Li Yang , Jeff Garzik , David Miller Reply-To: avorontsov@ru.mvista.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sat, Dec 20, 2008 at 09:19:23PM -0600, Timur Tabi wrote: > On Thu, Dec 18, 2008 at 12:23 PM, Anton Vorontsov > wrote: > > +static int ucc_geth_close(struct net_device *dev); > > +static int ucc_geth_open(struct net_device *dev); > > I would prefer that you reorder the functions to eliminate the need > for forward references. I'll post a cleanup patch (0/6 series is already in the netdev tree). Thanks! -- Anton Vorontsov email: cbouatmailru@gmail.com irc://irc.freenode.net/bd2