[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20131023220245.GF9378@mwanda>
Date: Thu, 24 Oct 2013 01:02:45 +0300
From: Dan Carpenter <dan.carpenter@...cle.com>
To: Chuong Ngo <cngo.github@...il.com>
Cc: gregkh@...uxfoundation.org, klmckinney1@...il.com,
devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drivers: staging: bcm: Removed a developer debug
statement.
On Wed, Oct 23, 2013 at 03:11:20PM -0400, Chuong Ngo wrote:
> Removed a developer debug statement per the TODO list. Additionally, removed braces for the if-statement to match coding style.
Line wrap the changelog at 72 characters. You don't really need to
mention the removed braces because that's too obvious to bother
commenting on.
My guess is that Greg will probably be willing to apply this as is so
don't worry about resending unless he requests it. (He normally seems
to let newbies get by with one line wrapping warning before he starts
making you redo it).
regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists