From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTP id AE854DDEE1 for ; Thu, 23 Aug 2007 07:08:42 +1000 (EST) Date: Thu, 23 Aug 2007 01:08:26 +0400 From: Vitaly Bordug To: Scott Wood Subject: Re: [PATCH 4/7] fs_enet: mac-fcc: Eliminate __fcc-* macros. Message-ID: <20070823010826.6ae3452d@localhost.localdomain> In-Reply-To: <20070817175402.GD9218@ld0162-tx32.am.freescale.net> References: <20070817175320.GA8259@ld0162-tx32.am.freescale.net> <20070817175402.GD9218@ld0162-tx32.am.freescale.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Cc: netdev@vger.kernel.org, jgarzik@pobox.com, linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, 17 Aug 2007 12:54:02 -0500 Scott Wood wrote: > These macros accomplish nothing other than defeating type checking. > > This patch also fixes one instance of the wrong register size being > used that was revealed by enabling type checking. > > Signed-off-by: Scott Wood > --- > drivers/net/fs_enet/mac-fcc.c | 25 ++++++++----------------- > 1 files changed, 8 insertions(+), 17 deletions(-) > > diff --git a/drivers/net/fs_enet/mac-fcc.c > b/drivers/net/fs_enet/mac-fcc.c index ad3c5fa..8b30361 100644 > --- a/drivers/net/fs_enet/mac-fcc.c > +++ b/drivers/net/fs_enet/mac-fcc.c > @@ -48,28 +48,19 @@ > > /* FCC access macros */ > > -#define __fcc_out32(addr, x) out_be32((unsigned *)addr, x) > -#define __fcc_out16(addr, x) out_be16((unsigned short *)addr, > x) -#define __fcc_out8(addr, x) out_8((unsigned char *)addr, x) > -#define __fcc_in32(addr) in_be32((unsigned *)addr) > -#define __fcc_in16(addr) in_be16((unsigned short *)addr) > -#define __fcc_in8(addr) in_8((unsigned char *)addr) > - > -/* parameter space */ > - > /* write, read, set bits, clear bits */ > -#define W32(_p, _m, _v) __fcc_out32(&(_p)->_m, (_v)) > -#define R32(_p, _m) __fcc_in32(&(_p)->_m) > +#define W32(_p, _m, _v) out_be32(&(_p)->_m, (_v)) > +#define R32(_p, _m) in_be32(&(_p)->_m) > #define S32(_p, _m, _v) W32(_p, _m, R32(_p, _m) | (_v)) > #define C32(_p, _m, _v) W32(_p, _m, R32(_p, _m) & ~(_v)) > > -#define W16(_p, _m, _v) __fcc_out16(&(_p)->_m, (_v)) > -#define R16(_p, _m) __fcc_in16(&(_p)->_m) > +#define W16(_p, _m, _v) out_be16(&(_p)->_m, (_v)) > +#define R16(_p, _m) in_be16(&(_p)->_m) > #define S16(_p, _m, _v) W16(_p, _m, R16(_p, _m) | (_v)) > #define C16(_p, _m, _v) W16(_p, _m, R16(_p, _m) & ~(_v)) > > -#define W8(_p, _m, _v) __fcc_out8(&(_p)->_m, (_v)) > -#define R8(_p, _m) __fcc_in8(&(_p)->_m) > +#define W8(_p, _m, _v) out_8(&(_p)->_m, (_v)) > +#define R8(_p, _m) in_8(&(_p)->_m) > #define S8(_p, _m, _v) W8(_p, _m, R8(_p, _m) | (_v)) > #define C8(_p, _m, _v) W8(_p, _m, R8(_p, _m) & ~(_v)) > > @@ -290,7 +281,7 @@ static void restart(struct net_device *dev) > > /* clear everything (slow & steady does it) */ > for (i = 0; i < sizeof(*ep); i++) > - __fcc_out8((char *)ep + i, 0); > + out_8((char *)ep + i, 0); > Perhaps W8() here, to keep consistency? > /* get physical address */ > rx_bd_base_phys = fep->ring_mem_addr; > @@ -495,7 +486,7 @@ static void tx_kickstart(struct net_device *dev) > struct fs_enet_private *fep = netdev_priv(dev); > fcc_t *fccp = fep->fcc.fccp; > > - S32(fccp, fcc_ftodr, 0x80); > + S16(fccp, fcc_ftodr, 0x8000); > } > > static u32 get_int_events(struct net_device *dev) -- Sincerely, Vitaly