From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anton Blanchard Subject: Re: [PATCH 4/6] ehea: header files Date: Sat, 12 Aug 2006 07:40:21 +1000 Message-ID: <20060811214020.GG479@krispykreme> References: <44D99F56.7010201@de.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev , linux-ppc , linux-kernel , Marcus Eder , Christoph Raisch , Thomas Klein Return-path: Received: from ozlabs.tip.net.au ([203.10.76.45]:11648 "EHLO ozlabs.org") by vger.kernel.org with ESMTP id S1750907AbWHKVox (ORCPT ); Fri, 11 Aug 2006 17:44:53 -0400 To: Jan-Bernd Themann Content-Disposition: inline In-Reply-To: <44D99F56.7010201@de.ibm.com> Sender: netdev-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Hi, > drivers/net/ehea/ehea.h | 452 > +#define EHEA_DRIVER_NAME "IBM eHEA" You are using this for ethtool get_drvinfo. Im not sure if it should match the module name, and I worry about having a space in the name. Any ideas on what we should be doing here? > +#define NET_IP_ALIGN 0 Shouldnt override this in your driver. > +#define EDEB_P_GENERIC(level, idstring, format, args...) \ > +#define EDEB_P_GENERIC(level,idstring,format,args...) \ > +#define EDEB(level, format, args...) \ > +#define EDEB_ERR(level, format, args...) \ > +#define EDEB_EN(level, format, args...) \ > +#define EDEB_EX(level, format, args...) \ > +#define EDEB_DMP(level, adr, len, format, args...) \ There are a lot of debug statements in the driver. When doing a review I stripped them all out to make it easier to read. As suggested by others, using the standard debug macros (where still required) would be a good idea. Anton