Re: [PATCH] staging: rdma: Fix braces around if/else

From: Martin Kletzander
Date: Sun Oct 04 2015 - 05:06:33 EST


On Sun, Oct 04, 2015 at 08:20:09AM +0100, Greg Kroah-Hartman wrote:
On Sun, Oct 04, 2015 at 09:47:52AM +0300, Or Gerlitz wrote:
On 10/3/2015 11:55 PM, Martin Kletzander wrote:
>Get rid of all ELSE_AFTER_BRACE type errors reported by checkpatch.pl.

Hi Greg,

Is there a way to signal people/tools that a certain driver parks in staging
on their way **out** of the kernel
and not the other way around? I guess you (nor Doug) don't want to spend
time on fixing such drivers, right?

I'm not spending the time :)

But no, there isn't a way to really show that that I know of, and it's
fine, we end up getting a number of "cleanup" patches for a kernel
release or two before the drivers get deleted. It gets people involved
in kernel development which is the key thing here, it's not like they
are actually using these drivers...

So it's fine, I take the patches, the people writing the patches are
happy, and the code gets a bit nicer and feels better about itself
before we kick it out the door.


I didn't mean to waste anyone's time on this. I'm just trying to get
more familiar with the patch submission process into the kernel. And
I'm doing this in my free time despite the address used. I probably
should've noted that below to commit message.

Martin

Attachment: signature.asc
Description: PGP signature