From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Egor Uleyskiy <egor.ulieiskii@gmail.com>,
Manohar Vanga <manohar.vanga@gmail.com>,
Martyn Welch <martyn@welchs.me.uk>,
Trivial Patch Monkey <trivial@kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] drivers: staging: vme: Changed (1 << n) to BIT(n)
Date: Tue, 24 Nov 2015 15:28:07 -0800 [thread overview]
Message-ID: <20151124232807.GB17422@kroah.com> (raw)
In-Reply-To: <CAHp75VdyiQ50N44-TRV4k+Mo-dG94tJBpHBt_xnMU3x-DN_Cmw@mail.gmail.com>
On Sun, Nov 08, 2015 at 11:43:28PM +0200, Andy Shevchenko wrote:
> On Sun, Nov 8, 2015 at 10:39 PM, Egor Uleyskiy <egor.ulieiskii@gmail.com> wrote:
> > From: Egor Uleyskiy <egor.ulieiskii@gmail.com>
> >
> > Signed-off-by: Egor Uleyskiy <egor.ulieiskii@gmail.com>
> > ---
> > drivers/staging/vme/devices/vme_pio2.h | 93 ++++++++++++++++------------------
> > 1 file changed, 45 insertions(+), 48 deletions(-)
> >
> > diff --git a/drivers/staging/vme/devices/vme_pio2.h b/drivers/staging/vme/devices/vme_pio2.h
> > index d5d94c4..29d7a58 100644
> > --- a/drivers/staging/vme/devices/vme_pio2.h
> > +++ b/drivers/staging/vme/devices/vme_pio2.h
> > @@ -1,6 +1,8 @@
> > #ifndef _VME_PIO2_H_
> > #define _VME_PIO2_H_
> >
> > +#include <linux/bitops.h>
> > +
> > #define PIO2_CARDS_MAX 32
> >
> > #define PIO2_VARIANT_LENGTH 5
> > @@ -48,8 +50,6 @@ static const int PIO2_REGS_INT_MASK[8] = { PIO2_REGS_INT_MASK0,
> > PIO2_REGS_INT_MASK6,
> > PIO2_REGS_INT_MASK7 };
> >
> > -
> > -
> > #define PIO2_REGS_CTRL 0x18
> > #define PIO2_REGS_VME_VECTOR 0x19
> > #define PIO2_REGS_CNTR0 0x20
> > @@ -63,7 +63,6 @@ static const int PIO2_REGS_INT_MASK[8] = { PIO2_REGS_INT_MASK0,
> >
> > #define PIO2_REGS_ID 0x30
> >
> > -
> > /* PIO2_REGS_DATAx (0x0 - 0x3) */
> >
> > static const int PIO2_CHANNEL_BANK[32] = { 0, 0, 0, 0, 0, 0, 0, 0,
> > @@ -71,38 +70,38 @@ static const int PIO2_CHANNEL_BANK[32] = { 0, 0, 0, 0, 0, 0, 0, 0,
> > 2, 2, 2, 2, 2, 2, 2, 2,
> > 3, 3, 3, 3, 3, 3, 3, 3 };
> >
> > -#define PIO2_CHANNEL0_BIT (1 << 0)
> > -#define PIO2_CHANNEL1_BIT (1 << 1)
> > -#define PIO2_CHANNEL2_BIT (1 << 2)
> > -#define PIO2_CHANNEL3_BIT (1 << 3)
> > -#define PIO2_CHANNEL4_BIT (1 << 4)
> > -#define PIO2_CHANNEL5_BIT (1 << 5)
> > -#define PIO2_CHANNEL6_BIT (1 << 6)
> > -#define PIO2_CHANNEL7_BIT (1 << 7)
> > -#define PIO2_CHANNEL8_BIT (1 << 0)
> > -#define PIO2_CHANNEL9_BIT (1 << 1)
> > -#define PIO2_CHANNEL10_BIT (1 << 2)
> > -#define PIO2_CHANNEL11_BIT (1 << 3)
> > -#define PIO2_CHANNEL12_BIT (1 << 4)
> > -#define PIO2_CHANNEL13_BIT (1 << 5)
> > -#define PIO2_CHANNEL14_BIT (1 << 6)
> > -#define PIO2_CHANNEL15_BIT (1 << 7)
> > -#define PIO2_CHANNEL16_BIT (1 << 0)
> > -#define PIO2_CHANNEL17_BIT (1 << 1)
> > -#define PIO2_CHANNEL18_BIT (1 << 2)
> > -#define PIO2_CHANNEL19_BIT (1 << 3)
> > -#define PIO2_CHANNEL20_BIT (1 << 4)
> > -#define PIO2_CHANNEL21_BIT (1 << 5)
> > -#define PIO2_CHANNEL22_BIT (1 << 6)
> > -#define PIO2_CHANNEL23_BIT (1 << 7)
> > -#define PIO2_CHANNEL24_BIT (1 << 0)
> > -#define PIO2_CHANNEL25_BIT (1 << 1)
> > -#define PIO2_CHANNEL26_BIT (1 << 2)
> > -#define PIO2_CHANNEL27_BIT (1 << 3)
> > -#define PIO2_CHANNEL28_BIT (1 << 4)
> > -#define PIO2_CHANNEL29_BIT (1 << 5)
> > -#define PIO2_CHANNEL30_BIT (1 << 6)
> > -#define PIO2_CHANNEL31_BIT (1 << 7)
> > +#define PIO2_CHANNEL0_BIT BIT(0)
> > +#define PIO2_CHANNEL1_BIT BIT(1)
> > +#define PIO2_CHANNEL2_BIT BIT(2)
> > +#define PIO2_CHANNEL3_BIT BIT(3)
> > +#define PIO2_CHANNEL4_BIT BIT(4)
> > +#define PIO2_CHANNEL5_BIT BIT(5)
> > +#define PIO2_CHANNEL6_BIT BIT(6)
> > +#define PIO2_CHANNEL7_BIT BIT(7)
> > +#define PIO2_CHANNEL8_BIT BIT(0)
> > +#define PIO2_CHANNEL9_BIT BIT(1)
> > +#define PIO2_CHANNEL10_BIT BIT(2)
> > +#define PIO2_CHANNEL11_BIT BIT(3)
> > +#define PIO2_CHANNEL12_BIT BIT(4)
> > +#define PIO2_CHANNEL13_BIT BIT(5)
> > +#define PIO2_CHANNEL14_BIT BIT(6)
> > +#define PIO2_CHANNEL15_BIT BIT(7)
> > +#define PIO2_CHANNEL16_BIT BIT(0)
> > +#define PIO2_CHANNEL17_BIT BIT(1)
> > +#define PIO2_CHANNEL18_BIT BIT(2)
> > +#define PIO2_CHANNEL19_BIT BIT(3)
> > +#define PIO2_CHANNEL20_BIT BIT(4)
> > +#define PIO2_CHANNEL21_BIT BIT(5)
> > +#define PIO2_CHANNEL22_BIT BIT(6)
> > +#define PIO2_CHANNEL23_BIT BIT(7)
> > +#define PIO2_CHANNEL24_BIT BIT(0)
> > +#define PIO2_CHANNEL25_BIT BIT(1)
> > +#define PIO2_CHANNEL26_BIT BIT(2)
> > +#define PIO2_CHANNEL27_BIT BIT(3)
> > +#define PIO2_CHANNEL28_BIT BIT(4)
> > +#define PIO2_CHANNEL29_BIT BIT(5)
> > +#define PIO2_CHANNEL30_BIT BIT(6)
> > +#define PIO2_CHANNEL31_BIT BIT(7)
> >
> > static const int PIO2_CHANNEL_BIT[32] = { PIO2_CHANNEL0_BIT, PIO2_CHANNEL1_BIT,
> > PIO2_CHANNEL2_BIT, PIO2_CHANNEL3_BIT,
> > @@ -123,12 +122,12 @@ static const int PIO2_CHANNEL_BIT[32] = { PIO2_CHANNEL0_BIT, PIO2_CHANNEL1_BIT,
> > };
> >
> > /* PIO2_REGS_INT_STAT_CNTR (0xc) */
> > -#define PIO2_COUNTER0 (1 << 0)
> > -#define PIO2_COUNTER1 (1 << 1)
> > -#define PIO2_COUNTER2 (1 << 2)
> > -#define PIO2_COUNTER3 (1 << 3)
> > -#define PIO2_COUNTER4 (1 << 4)
> > -#define PIO2_COUNTER5 (1 << 5)
> > +#define PIO2_COUNTER0 BIT(0)
> > +#define PIO2_COUNTER1 BIT(1)
> > +#define PIO2_COUNTER2 BIT(2)
> > +#define PIO2_COUNTER3 BIT(3)
> > +#define PIO2_COUNTER4 BIT(4)
> > +#define PIO2_COUNTER5 BIT(5)
> >
> > static const int PIO2_COUNTER[6] = { PIO2_COUNTER0, PIO2_COUNTER1,
> > PIO2_COUNTER2, PIO2_COUNTER3,
> > @@ -136,8 +135,8 @@ static const int PIO2_COUNTER[6] = { PIO2_COUNTER0, PIO2_COUNTER1,
> >
> > /* PIO2_REGS_CTRL (0x18) */
> > #define PIO2_VME_INT_MASK 0x7
> > -#define PIO2_LED (1 << 6)
> > -#define PIO2_LOOP (1 << 7)
> > +#define PIO2_LED BIT(6)
> > +#define PIO2_LOOP BIT(7)
> >
> > /* PIO2_REGS_VME_VECTOR (0x19) */
> > #define PIO2_VME_VECTOR_SPUR 0x0
> > @@ -182,7 +181,7 @@ static const int PIO2_CNTR_CTRL[6] = { PIO2_REGS_CTRL_WRD0,
> > PIO2_REGS_CTRL_WRD1 };
> >
> > #define PIO2_CNTR_SC_DEV0 0
> > -#define PIO2_CNTR_SC_DEV1 (1 << 6)
> > +#define PIO2_CNTR_SC_DEV1 BIT(6)
> > #define PIO2_CNTR_SC_DEV2 (2 << 6)
> > #define PIO2_CNTR_SC_RDBACK (3 << 6)
>
> With the first parts which are an excellent clean up, this one makes
> two styles out of one.
> Greg, what would you suggest to do? For my opinion in such cases
> direct values or previous syntax looks better.
Yes, this should not be changed.
thanks,
greg k-h
next prev parent reply other threads:[~2015-11-24 23:28 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-08 20:39 [PATCH] drivers: staging: vme: Changed (1 << n) to BIT(n) Egor Uleyskiy
2015-11-08 21:43 ` Andy Shevchenko
2015-11-24 23:28 ` Greg Kroah-Hartman [this message]
[not found] ` <CAEccXefJK9PE73B2njX-on0RWKZbiOk-ARwJe8V1xLHP1i0FAQ@mail.gmail.com>
2015-11-24 23:28 ` Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151124232807.GB17422@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=andy.shevchenko@gmail.com \
--cc=egor.ulieiskii@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=manohar.vanga@gmail.com \
--cc=martyn@welchs.me.uk \
--cc=trivial@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox