Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751377AbWEDGwY (ORCPT ); Thu, 4 May 2006 02:52:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751410AbWEDGwY (ORCPT ); Thu, 4 May 2006 02:52:24 -0400 Received: from rutherford.zen.co.uk ([212.23.3.142]:21384 "EHLO rutherford.zen.co.uk") by vger.kernel.org with ESMTP id S1751377AbWEDGwX (ORCPT ); Thu, 4 May 2006 02:52:23 -0400 Message-ID: <4459A4A6.1080207@cantab.net> Date: Thu, 04 May 2006 07:52:22 +0100 From: David Vrabel User-Agent: Mozilla Thunderbird 1.0.2 (Windows/20050317) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Francois Romieu CC: Pekka J Enberg , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, david@pleyades.net Subject: Re: [PATCH 2/2] ipg: redundancy with mii.h References: <1146342905.11271.3.camel@localhost> <1146389171.11524.1.camel@localhost> <44554ADE.8030200@cantab.net> <4455F1D8.5030102@cantab.net> <1146506939.23931.2.camel@localhost> <20060501231206.GD7419@electric-eye.fr.zoreil.com> <20060502214520.GC26357@electric-eye.fr.zoreil.com> <20060502215559.GA1119@electric-eye.fr.zoreil.com> <20060503233558.GA27232@electric-eye.fr.zoreil.com> In-Reply-To: <20060503233558.GA27232@electric-eye.fr.zoreil.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Originating-Rutherford-IP: [82.70.146.41] Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1166 Lines: 41 Francois Romieu wrote: > > ipg: remove forward declarations > > It makes no sense in a new driver. > > Signed-off-by: Francois Romieu Ack. > ipg: replace #define with enum > > Added some underscores to improve readability. > > Signed-off-by: Francois Romieu Nack. Register names in code should match those used in the documentation (even if they are a bit unreadable). Though I will conceed that the available datasheet doesn't actually describe the majority of the registers. > ipg: removal of useless #defines > > IPG_TX_NOTBUSY apart (one occurence in ipg.c), the #defines appear > nowhere in the sources. Ack. > ipg: redundancy with mii.h - take II > > Replace a bunch of #define with their counterpart from mii.h > > It is applied to the usual MII registers this time. Ack. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/