Re: [PATCH v2] staging: skein: Remove all do{} while (0) loop from single statement.

From: Jason Cooper <jason@lakedaemon.net>
To: shirish gajera <gajerashirish@gmail.com>
Cc: gregkh@linuxfoundation.org, Eric Rost <eric.rost@mybabylon.net>,
	jake@lwn.net, Anton Saraev <antonysaraev@gmail.com>,
	devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] staging: skein: Remove all do{} while (0) loop from single statement.
Date: Wed, 14 Jan 2015 13:37:56 -0500	[thread overview]
Message-ID: <20150114183756.GA24989@titan.lakedaemon.net> (raw)
In-Reply-To: <CAG77vrojhVij2154vDxCxE9vPDPfP2QuJaKFcAz4pErW_pLRVQ@mail.gmail.com>

Shirish,

Please don't toppost, I've corrected it below.

On Mon, Jan 12, 2015 at 11:14:57AM -0800, shirish gajera wrote:
> On Mon, Jan 12, 2015 at 6:15 AM, Jason Cooper <jason@lakedaemon.net> wrote:
> > On Sun, Jan 11, 2015 at 03:41:05PM -0800, Shirish Gajera wrote:
> > > This patch fixes the checkpatch.pl warning:
> > >
> > > WARNING: Single statement macros should not use a do {} while (0) loop
> > >
> > > I remove do while from the single statement macro.
> > >
> > > Signed-off-by: Shirish Gajera <gajerashirish@gmail.com>
> > > ---
> > >  drivers/staging/skein/skein_block.c | 31 +++++++++----------------------
> > >  1 file changed, 9 insertions(+), 22 deletions(-)
> >
> > Several people have submitted patches for this checkpatch warning.
> > Please check the mailinglist archives and deconflict with the other
> > submitters.
> 
> It did't see any other submitter in script.
> 
> Is there any other way to check ?

The mailing list for the linux-staging tree is rather arcane to find,
it's not linux-staging@...  Here's the link for joining:

  http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

And here's the archives:

  http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/

hth,

Jason.