Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756594Ab0KQNEn (ORCPT ); Wed, 17 Nov 2010 08:04:43 -0500 Received: from metis.ext.pengutronix.de ([92.198.50.35]:56664 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755052Ab0KQNEl (ORCPT ); Wed, 17 Nov 2010 08:04:41 -0500 Message-ID: <4CE3D2D5.3030301@pengutronix.de> Date: Wed, 17 Nov 2010 14:04:21 +0100 From: Marc Kleine-Budde Organization: Pengutronix User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101027 Thunderbird/3.0.10 MIME-Version: 1.0 To: Tomoya MORINAGA CC: Wolfgang Grandegger , Wolfram Sang , Christian Pellegrin , Barry Song <21cnbao@gmail.com>, Samuel Ortiz , socketcan-core@lists.berlios.de, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, "David S. Miller" , andrew.chih.howe.khor@intel.com, qi.wang@intel.com, margie.foster@intel.com, yong.y.wang@intel.com, kok.howg.ewe@intel.com, joel.clark@intel.com Subject: Re: [PATCH net-next-2.6] can: EG20T PCH: use BIT(X) References: <4CE3C90D.6010705@dsn.okisemi.com> In-Reply-To: <4CE3C90D.6010705@dsn.okisemi.com> X-Enigmail-Version: 1.0.1 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enigD78484C5CF4FA3153FBA1420" X-SA-Exim-Connect-IP: 2001:6f8:1178:4:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: mkl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5280 Lines: 143 This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enigD78484C5CF4FA3153FBA1420 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 11/17/2010 01:22 PM, Tomoya MORINAGA wrote: > Replace bit assignment value to BIT(X). > For easy to readable/identifiable, replace all bit assigned macros to B= IT(X) >=20 > Signed-off-by: Tomoya MORINAGA IMHO you can squash this into the "add prefix to macro" patch. You have my Acked-by for both solutions. cheers, Marc > --- > drivers/net/can/pch_can.c | 73 +++++++++++++++++++++++--------------= -------- > 1 files changed, 37 insertions(+), 36 deletions(-) >=20 > diff --git a/drivers/net/can/pch_can.c b/drivers/net/can/pch_can.c > index c523e3d..238622a 100644 > --- a/drivers/net/can/pch_can.c > +++ b/drivers/net/can/pch_can.c > @@ -38,50 +38,51 @@ > =20 > #define PCH_ENABLE 1 /* The enable flag */ > #define PCH_DISABLE 0 /* The disable flag */ > -#define PCH_CTRL_INIT 0x0001 /* The INIT bit of CANCONT register. */ > -#define PCH_CTRL_IE 0x0002 /* The IE bit of CAN control register */ > -#define PCH_CTRL_IE_SIE_EIE 0x000e > -#define PCH_CTRL_CCE 0x0040 > -#define PCH_CTRL_OPT 0x0080 /* The OPT bit of CANCONT register. */ > -#define PCH_OPT_SILENT 0x0008 /* The Silent bit of CANOPT reg. */ > -#define PCH_OPT_LBACK 0x0010 /* The LoopBack bit of CANOPT reg. */ > +#define PCH_CTRL_INIT BIT(0) /* The INIT bit of CANCONT register. */ > +#define PCH_CTRL_IE BIT(1) /* The IE bit of CAN control register */ > +#define PCH_CTRL_IE_SIE_EIE (BIT(3) | BIT(2) | BIT(1)) > +#define PCH_CTRL_CCE BIT(6) > +#define PCH_CTRL_OPT BIT(7) /* The OPT bit of CANCONT register. */ > +#define PCH_OPT_SILENT BIT(3) /* The Silent bit of CANOPT reg. */ > +#define PCH_OPT_LBACK BIT(4) /* The LoopBack bit of CANOPT reg. */ > + > #define PCH_CMASK_RX_TX_SET 0x00f3 > #define PCH_CMASK_RX_TX_GET 0x0073 > #define PCH_CMASK_ALL 0xff > -#define PCH_CMASK_RDWR 0x80 > -#define PCH_CMASK_ARB 0x20 > -#define PCH_CMASK_CTRL 0x10 > -#define PCH_CMASK_MASK 0x40 > -#define PCH_CMASK_NEWDAT 0x04 > -#define PCH_CMASK_CLRINTPND 0x08 > -#define PCH_IF_MCONT_NEWDAT 0x8000 > -#define PCH_IF_MCONT_INTPND 0x2000 > -#define PCH_IF_MCONT_UMASK 0x1000 > -#define PCH_IF_MCONT_TXIE 0x0800 > -#define PCH_IF_MCONT_RXIE 0x0400 > -#define PCH_IF_MCONT_RMTEN 0x0200 > -#define PCH_IF_MCONT_TXRQXT 0x0100 > -#define PCH_IF_MCONT_EOB 0x0080 > -#define PCH_IF_MCONT_DLC 0x000f > -#define PCH_IF_MCONT_MSGLOST 0x4000 > -#define PCH_MASK2_MDIR_MXTD 0xc000 > -#define PCH_ID2_DIR 0x2000 > -#define PCH_ID2_XTD 0x4000 > -#define PCH_ID_MSGVAL 0x8000 > -#define PCH_IF_CREQ_BUSY 0x8000 > +#define PCH_CMASK_NEWDAT BIT(2) > +#define PCH_CMASK_CLRINTPND BIT(3) > +#define PCH_CMASK_CTRL BIT(4) > +#define PCH_CMASK_ARB BIT(5) > +#define PCH_CMASK_MASK BIT(6) > +#define PCH_CMASK_RDWR BIT(7) > +#define PCH_IF_MCONT_NEWDAT BIT(15) > +#define PCH_IF_MCONT_MSGLOST BIT(14) > +#define PCH_IF_MCONT_INTPND BIT(13) > +#define PCH_IF_MCONT_UMASK BIT(12) > +#define PCH_IF_MCONT_TXIE BIT(11) > +#define PCH_IF_MCONT_RXIE BIT(10) > +#define PCH_IF_MCONT_RMTEN BIT(9) > +#define PCH_IF_MCONT_TXRQXT BIT(8) > +#define PCH_IF_MCONT_EOB BIT(7) > +#define PCH_IF_MCONT_DLC (BIT(0) | BIT(1) | BIT(2) | BIT(3)) > +#define PCH_MASK2_MDIR_MXTD (BIT(14) | BIT(15)) > +#define PCH_ID2_DIR BIT(13) > +#define PCH_ID2_XTD BIT(14) > +#define PCH_ID_MSGVAL BIT(15) > +#define PCH_IF_CREQ_BUSY BIT(15) > =20 > #define PCH_STATUS_INT 0x8000 > #define PCH_REC 0x00007f00 > #define PCH_TEC 0x000000ff > =20 > -#define PCH_RX_OK 0x00000010 > -#define PCH_TX_OK 0x00000008 > -#define PCH_BUS_OFF 0x00000080 > -#define PCH_EWARN 0x00000040 > -#define PCH_EPASSIV 0x00000020 > -#define PCH_LEC0 0x00000001 > -#define PCH_LEC1 0x00000002 > -#define PCH_LEC2 0x00000004 > +#define PCH_TX_OK BIT(3) > +#define PCH_RX_OK BIT(4) > +#define PCH_EPASSIV BIT(5) > +#define PCH_EWARN BIT(6) > +#define PCH_BUS_OFF BIT(7) > +#define PCH_LEC0 BIT(0) > +#define PCH_LEC1 BIT(1) > +#define PCH_LEC2 BIT(2) > #define PCH_LEC_ALL (PCH_LEC0 | PCH_LEC1 | PCH_LEC2) > #define PCH_STUF_ERR PCH_LEC0 > #define PCH_FORM_ERR PCH_LEC1 --=20 Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions | Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | --------------enigD78484C5CF4FA3153FBA1420 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAkzj0tkACgkQjTAFq1RaXHMbGQCfVxoVtwlXphw44uODAlEaqM8J 06AAoIsRtLK+ZzCx9C9vP9a9TeJVTR+v =XkfX -----END PGP SIGNATURE----- --------------enigD78484C5CF4FA3153FBA1420-- -- 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/