[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20131029155040.GA3169@kroah.com>
Date: Tue, 29 Oct 2013 08:50:40 -0700
From: Greg KH <gregkh@...uxfoundation.org>
To: Dan Carpenter <dan.carpenter@...cle.com>
Cc: Chuong Ngo <cngo.github@...il.com>, devel@...verdev.osuosl.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drivers: staging: bcm: Removed a developer debug
statement.
On Thu, Oct 24, 2013 at 01:02:45AM +0300, Dan Carpenter wrote:
> 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).
Yes, I'll fix it this time, but Chuong, please be more careful next
time.
greg k-h
--
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